Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Methods like status and location are interfering with redirect_to [Closes #2511] | Franck Verrot | 2011-08-14 | 1 | -0/+5 |
| | |||||
* | moving test_generate to an integration test with one assert per test | Aaron Patterson | 2011-08-12 | 2 | -108/+183 |
| | |||||
* | Fix the issue where default_url_options is being cached on test cases. ↵ | thoefer | 2011-08-01 | 1 | -0/+29 |
| | | | | | | Closes #1872. Closes #2031. Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Merge pull request #2379 from arunagw/extra_require_removal | Santiago Pastorino | 2011-07-31 | 1 | -1/+0 |
|\ | | | | | Removing extra requires from the test. Already loaded in abstract_unit. | ||||
| * | Removing extra requires from the test. Already loaded in abstract_unit. | Arun Agrawal | 2011-07-31 | 1 | -1/+0 |
| | | |||||
* | | remove extra require for 'active_support/dependencies' as it is required in ↵ | Vishnu Atrai | 2011-07-31 | 1 | -1/+0 |
|/ | | | | abstract_unit.rb | ||||
* | There is no need to be destructive with the passed-in options. | thedarkone | 2011-07-28 | 1 | -0/+9 |
| | | | | This fixes a bug that is caused by Resource/SingletonResource mangling resource options when using inline "multi"-resource declarations. | ||||
* | - added ActionView::PartialRenderer#merge_path_into_partial(path, partial) | Gaston Ramos | 2011-07-24 | 1 | -1/+1 |
| | | | | fix issues/1951 | ||||
* | - added test case for issue: | Gaston Ramos | 2011-07-24 | 1 | -0/+14 |
| | | | | | https://github.com/rails/rails/issues/1951 Namespaced model partial_path is wrong in namespaced controllers | ||||
* | Ensure that status codes are logged properly | Jesse Storimer | 2011-07-18 | 1 | -0/+19 |
| | | | | | | Needed to move AC::Metal::Instrumentation before AM::Metal::Rescue so that status codes rendered from rescue_from blocks are logged properly. | ||||
* | Remove stream at the class level. | José Valim | 2011-07-06 | 1 | -1/+1 |
| | | | | | | This is because only template rendering works with streaming. Setting it at the class level was also changing the behavior of JSON and XML responses, closes #1337. | ||||
* | provide a more explicit message when using url_for with nil | Damien Mathieu | 2011-07-02 | 1 | -0/+7 |
| | | | This fixes the problem of having a non-explicit message when the :location option is not provided in respond_with. | ||||
* | Make sure respond_with with :js tries to render a template in all cases | José Valim | 2011-06-30 | 1 | -6/+15 |
| | |||||
* | Make send_file guess content-type from file extension, if type wasn't ↵ | Esad Hajdarevic | 2011-06-28 | 1 | -0/+19 |
| | | | | supplied (Issue #1847). Update tests & documentation. | ||||
* | Get the fixture_path from self.class instead of ActiveSupport::TestCase. | David Chelimsky | 2011-06-25 | 1 | -0/+7 |
| | | | | | | This allows test classes that are not subclasses of ActiveSupport::TestCase (like those in rspec-rails) to interact with with this variable without having to reference ActiveSupport::TestCase. | ||||
* | Tests only after filters in cache sweepers | Jeroen Jacobs | 2011-06-22 | 1 | -0/+5 |
| | |||||
* | remove meaningless assert true | Neeraj Singh | 2011-06-11 | 1 | -1/+0 |
| | |||||
* | expected message should come first | Neeraj Singh | 2011-06-11 | 1 | -2/+2 |
| | |||||
* | try not to use assert_blank when nil will would | Neeraj Singh | 2011-06-11 | 1 | -2/+2 |
| | | | | | | be considered as failure Test for specific value to the extent possible | ||||
* | add missing ensure | Neeraj Singh | 2011-06-10 | 1 | -0/+1 |
| | |||||
* | replaced and with && as per coding guidelines | Neeraj Singh | 2011-06-10 | 1 | -1/+1 |
| | |||||
* | The test name is misleading. It says | Neeraj Singh | 2011-06-10 | 1 | -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 test | Neeraj Singh | 2011-06-08 | 1 | -3/+0 |
| | |||||
* | Refactor ActionController::TestCase cookies | Andrew White | 2011-06-04 | 1 | -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 #1210 | Damien Mathieu | 2011-06-01 | 1 | -0/+6 |
| | |||||
* | Corrected some typos and American vs. Queen's English issues | Lee Reilly | 2011-05-29 | 1 | -2/+2 |
| | |||||
* | Merge pull request #1203 from dchelimsky/stringify-parameter-values-in-tests | José Valim | 2011-05-28 | 1 | -0/+10 |
|\ | | | | | Stringify param values in controller tests. | ||||
| * | Stringify param values in controller tests. | David Chelimsky | 2011-05-22 | 1 | -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 recognized | dmathieu | 2011-05-26 | 1 | -0/+6 |
| | | | | | | | | Fixed while traveling to heuruko | ||||
* | | removed deprecated methods, and related tests, from ActionPack | Josh Kalderimis | 2011-05-24 | 4 | -40/+3 |
| | | |||||
* | | Replace references to ActiveSupport::SecureRandom with just SecureRandom, ↵ | Jon Leighton | 2011-05-23 | 1 | -3/+3 |
| | | | | | | | | and require 'securerandom' from the stdlib when active support is required. | ||||
* | | Always initialize @title in caching test to avoid uninitialized instance ↵ | wycats | 2011-05-22 | 1 | -0/+3 |
|/ | | | | variable warning | ||||
* | renamed the wrap_parameters :only and :except options to :include and ↵ | Josh Kalderimis | 2011-05-19 | 2 | -9/+9 |
| | | | | :exclude to make it consistent with controller filters | ||||
* | add more robust test for wrapping params with anonymous class | David Chelimsky | 2011-05-17 | 1 | -25/+45 |
| | |||||
* | better test name | David Chelimsky | 2011-05-17 | 1 | -1/+1 |
| | |||||
* | add failing test for https://github.com/rails/rails/issues/1089 | David Chelimsky | 2011-05-17 | 1 | -0/+7 |
| | |||||
* | Make ParamsWrapper calling newly introduced `Model.attribute_names` instead ↵ | Prem Sichanugrist | 2011-05-15 | 1 | -10/+8 |
| | | | | of `.column_names` | ||||
* | Do not try to call `column_names` on the abstract class. | Prem Sichanugrist | 2011-05-15 | 1 | -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 customization | David Lee | 2011-05-10 | 1 | -7/+18 |
| | |||||
* | Get around weird missing constant error caused by AS instead of simply ↵ | José Valim | 2011-05-11 | 1 | -21/+24 |
| | | | | raising NameError, closes #477. | ||||
* | Revert to old semantics, use available_action? instead of action_method?. | José Valim | 2011-05-06 | 1 | -4/+4 |
| | |||||
* | controller's prefixes are now added to LookupContext. | Nick Sutterer | 2011-05-03 | 1 | -0/+4 |
| | |||||
* | Fix typos in test method names | Vijay Dev | 2011-05-03 | 1 | -3/+3 |
| | |||||
* | raise an error if the old router draw method is used, along with a message ↵ | Josh Kalderimis | 2011-05-03 | 1 | -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 Sichanugrist | 2011-05-03 | 1 | -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é Valim | 2011-05-03 | 1 | -0/+11 |
| | |||||
* | Move most processing to load time for performance and improve test suite. | José Valim | 2011-05-03 | 1 | -12/+27 |
| | |||||
* | Add `ActionController::ParamsWrapper` to wrap parameters into a nested hash | Prem Sichanugrist | 2011-05-03 | 1 | -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é Valim | 2011-05-01 | 1 | -7/+7 |
| | |||||
* | log errors when an exception happens when streaming. | José Valim | 2011-05-01 | 1 | -0/+13 |
| |