aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/controller
Commit message (Collapse)AuthorAgeFilesLines
* remove meaningless assert trueNeeraj Singh2011-06-111-1/+0
|
* expected message should come firstNeeraj Singh2011-06-111-2/+2
|
* try not to use assert_blank when nil will wouldNeeraj Singh2011-06-111-2/+2
| | | | | | be considered as failure Test for specific value to the extent possible
* add missing ensureNeeraj Singh2011-06-101-0/+1
|
* replaced and with && as per coding guidelinesNeeraj Singh2011-06-101-1/+1
|
* The test name is misleading. It saysNeeraj Singh2011-06-101-1/+1
| | | | | | | | | | | | test_should_cache_with_trailing_slash_on_url A trailing slash is removed when a page is cached. What the test meant to say was : a url with trailing slash should be cached and the cached page should not have any trailing slash. This patch clarifies the name a bit.
* remove unused user_controller from testNeeraj Singh2011-06-081-3/+0
|
* Refactor ActionController::TestCase cookiesAndrew White2011-06-041-2/+2
| | | | | | | | | | | | | | | | | | Assigning cookies for test cases should now use cookies[], e.g: cookies[:email] = 'user@example.com' get :index assert_equal 'user@example.com', cookies[:email] To clear the cookies, use clear, e.g: cookies.clear get :index assert_nil cookies[:email] We now no longer write out HTTP_COOKIE and the cookie jar is persistent between requests so if you need to manipulate the environment for your test you need to do it before the cookie jar is created.
* fix creating an empty route on 1.8. Closes #1210Damien Mathieu2011-06-011-0/+6
|
* Corrected some typos and American vs. Queen's English issuesLee Reilly2011-05-291-2/+2
|
* Merge pull request #1203 from dchelimsky/stringify-parameter-values-in-testsJosé Valim2011-05-281-0/+10
|\ | | | | Stringify param values in controller tests.
| * Stringify param values in controller tests.David Chelimsky2011-05-221-0/+10
| | | | | | | | | | | | | | | | This reduces false positives that come from using ints in params in tests, which do not get converted to strings in the tests. In implementations going through rack, they do get converted to strings. - David Chelimsky and Sam Umbach
* | don't raise an exception if the format isn't recognizeddmathieu2011-05-261-0/+6
| | | | | | | | Fixed while traveling to heuruko
* | removed deprecated methods, and related tests, from ActionPackJosh Kalderimis2011-05-244-40/+3
| |
* | Replace references to ActiveSupport::SecureRandom with just SecureRandom, ↵Jon Leighton2011-05-231-3/+3
| | | | | | | | and require 'securerandom' from the stdlib when active support is required.
* | Always initialize @title in caching test to avoid uninitialized instance ↵wycats2011-05-221-0/+3
|/ | | | variable warning
* renamed the wrap_parameters :only and :except options to :include and ↵Josh Kalderimis2011-05-192-9/+9
| | | | :exclude to make it consistent with controller filters
* add more robust test for wrapping params with anonymous classDavid Chelimsky2011-05-171-25/+45
|
* better test nameDavid Chelimsky2011-05-171-1/+1
|
* add failing test for https://github.com/rails/rails/issues/1089David Chelimsky2011-05-171-0/+7
|
* Make ParamsWrapper calling newly introduced `Model.attribute_names` instead ↵Prem Sichanugrist2011-05-151-10/+8
| | | | of `.column_names`
* Do not try to call `column_names` on the abstract class.Prem Sichanugrist2011-05-151-1/+14
| | | Normally the table for abstract class won't be existed, so we should not trying to call `#column_names` on it.
* Test csrf token param name customizationDavid Lee2011-05-101-7/+18
|
* Get around weird missing constant error caused by AS instead of simply ↵José Valim2011-05-111-21/+24
| | | | raising NameError, closes #477.
* Revert to old semantics, use available_action? instead of action_method?.José Valim2011-05-061-4/+4
|
* controller's prefixes are now added to LookupContext.Nick Sutterer2011-05-031-0/+4
|
* Fix typos in test method namesVijay Dev2011-05-031-3/+3
|
* raise an error if the old router draw method is used, along with a message ↵Josh Kalderimis2011-05-031-0/+6
| | | | advising them to either upgrade their routes or add rails_legacy_mapper to their Gemfile
* Fix broken params_wrapper_test on 1.8.7 :bomb:Prem Sichanugrist2011-05-031-36/+80
| | | | | I have to keep in mind that 1.8.7 does *not* preserve the hash order. Guys, let's move to use 1.9.2 in production!
* Improve performance for filtered parameters and add tests.José Valim2011-05-031-0/+11
|
* Move most processing to load time for performance and improve test suite.José Valim2011-05-031-12/+27
|
* Add `ActionController::ParamsWrapper` to wrap parameters into a nested hashPrem Sichanugrist2011-05-031-0/+187
| | | This will allow us to do a rootless JSON/XML request to server.
* Tidy up pending TODOs after discussion with Mr. Gatoz (@wycats).José Valim2011-05-011-7/+7
|
* log errors when an exception happens when streaming.José Valim2011-05-011-0/+13
|
* Add a test for rendering from the controller context.José Valim2011-05-011-0/+54
|
* prevent errors when passing a frozen string as a param to ↵misfo2011-04-281-0/+12
| | | | | | | ActionController::TestCase#process since ActionDispatch::Http::Parameters#encode_params will force encoding on all params strings (when using an encoding aware Ruby), dup all strings passed into process. This prevents modification of params passed in and, more importantly, doesn't barf when a frozen string is passed thanks and high fives to kinsteronline
* rename test case to better describe use caseDavid Chelimsky2011-04-251-6/+6
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Don't reuse a closed flash when using nowFlorent Piteau2011-04-201-0/+8
|
* Do not stream on HTTP/1.0.José Valim2011-04-191-0/+8
|
* Output a redirect to the 500 page if something happens when streaming.José Valim2011-04-191-1/+23
| | | | | | Currently, we output: "><script type="text/javascript">window.location = "/500.html"</script></html>
* Be sure to not store the closed flash in the session.José Valim2011-04-191-1/+8
|
* Test explicit skip.José Valim2011-04-181-1/+10
|
* Just define the controller if fibers are defined.José Valim2011-04-181-2/+2
|
* Body... wanna *stream* my body? Body... such a thrill my body!José Valim2011-04-181-0/+62
| | | | | Added stream as class level method to make it explicit when to stream. Render also accepts :stream as option.
* switch to using comments to comment thingsAaron Patterson2011-04-151-2/+1
|
* render :once, YAGNI.José Valim2011-04-151-86/+0
|
* ActionView::PathSet# accepts String or ArrayDavid Chelimsky2011-04-131-0/+29
| | | | | | - Closes #6692 Signed-off-by: José Valim <jose.valim@gmail.com>
* Remove `#among?` from Active SupportPrem Sichanugrist2011-04-131-1/+1
| | | | | | After a long list of discussion about the performance problem from using varargs and the reason that we can't find a great pair for it, it would be best to remove support for it for now. It will come back if we can find a good pair for it. For now, Bon Voyage, `#among?`.
* removes the RJS template handlerXavier Noria2011-04-135-66/+11
|
* removes support for render :updateXavier Noria2011-04-135-389/+0
|