aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
Commit message (Collapse)AuthorAgeFilesLines
* Added missing assert to test checking form with namespaced isolated model ↵Piotr Sarnacki2011-06-071-5/+5
| | | | and fixed helpers to work correctly in such case
* Add backward compatibility for testing cookiesAndrew White2011-06-051-2/+49
| | | | | | | | | | | | | | | | | | | This commit restores the ability to assign cookies for testing via @request.env['HTTP_COOKIE'] and @request.cookies, e.g: @request.env['HTTP_COOKIE'] = 'user_name=david' get :index assert_equal 'david', cookies[:user_name] and @request.cookies[:user_name] = 'david' get :index assert_equal 'david', cookies[:user_name] Assigning via cookies[] is the preferred method and will take precedence over the other two methods. This is so that cookies set in controller actions have precedence and are carried over between calls to get, post, etc.
* Ensure cookie keys are stringsAndrew White2011-06-051-0/+9
|
* Allow multiple sources in Sprockets helpersGuillermo Iguaran2011-06-053-0/+6
|
* Refactor ActionController::TestCase cookiesAndrew White2011-06-043-26/+23
| | | | | | | | | | | | | | | | | | 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
* | Fix typo in test.Andrés Botero2011-05-281-1/+1
| |
* | fixed asset_paths when called from sprockets for relative_url_rootjacott2011-05-281-0/+6
| |
* | 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-248-105/+3
| |
* | Move the last pockets of Sprockets stuff into the Sprockets directorywycats2011-05-231-2/+1
| |
* | Move javascript/css helpers into the Sprockets helper and just have them ↵wycats2011-05-231-12/+12
| | | | | | | | override the old behavior
* | Start moving some logic from being embedded in AV into the Rails Sprockets ↵wycats2011-05-231-0/+2
| | | | | | | | extensions
* | Replace references to ActiveSupport::SecureRandom with just SecureRandom, ↵Jon Leighton2011-05-232-4/+4
| | | | | | | | and require 'securerandom' from the stdlib when active support is required.
* | Merge pull request #1228 from fcheung/db_time_fix2José Valim2011-05-231-0/+26
|\ \ | | | | | | Take into account time spent in AR even if a redirect occurs or if it is after the render
| * | Take into account time spent in AR even if a redirect occurs or if it is ↵Frederick Cheung2011-05-231-0/+26
| | | | | | | | | | | | after the render
* | | Eliminate uninitialized instance variable warningwycats2011-05-221-2/+2
| | |
* | | Prepare the context in URLHelperTest so that there are no required ↵wycats2011-05-221-0/+2
| | | | | | | | | | | | uninitialized instance variables
* | | Merge pull request #1222 from arunagw/sprockets-fixesJosé Valim2011-05-222-8/+0
|\ \ \ | | | | | | | | We don't need to load version any more as Sprockets.beta8 is included.
| * | | We don't need to load version any more as Sprockets.beta8 is included. Arun Agrawal2011-05-232-8/+0
| | | |
* | | | Always initialize @title in caching test to avoid uninitialized instance ↵wycats2011-05-221-0/+3
|/ / / | | | | | | | | | variable warning
* | | Merge branch '3-1-stable'Joshua Peek2011-05-223-2/+11
|\ \ \ | | | | | | | | | | | | | | | | | | | | Conflicts: actionpack/actionpack.gemspec activemodel/activemodel.gemspec
| * | | Add debug_asset supportJoshua Peek2011-05-223-2/+11
| | |/ | |/|
* | | Revert "Take into account time spent in AR even if a redirect occurs or if ↵David Heinemeier Hansson2011-05-221-26/+0
| | | | | | | | | | | | | | | | | | it is after the render" because it's causing test failures about missing db_during_render. This reverts commit 60b55baca9e381deb3a0a85580f089f4caf551d4.
* | | Make sure that we're able to run Action Pack test standalone.Prem Sichanugrist2011-05-222-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | Sprocket was trying to look for these two things: * `Rails::VERSION::STRING` * `.initializer(str)` in `Rails::Engine` This commit added require to version.rb into Abstract Unit, and mocking the `Rails::Engine.initializer` in `sprockets_helper_test.rb`
* | | Take into account time spent in AR even if a redirect occurs or if it is ↵Frederick Cheung2011-05-221-0/+26
| |/ |/| | | | | after the render
* | Add method fields_for_with_index to FormHelperJorge Bejar2011-05-211-0/+125
|/
* Merge pull request #1151 from joshk/wrap_params_options_changeJosé Valim2011-05-192-9/+9
|\ | | | | renamed the wrap_parameters :only and :except options to :include and :ex
| * 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
* | adding teststeve2011-05-191-4/+8
|/
* Example Usage updated for actionpack and actionmailer.Arun Agrawal2011-05-191-9/+9
|
* Dump and load rack-cache stuff.José Valim2011-05-171-0/+21
|
* 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
|
* Merge pull request #570 from sikachu/decouple_actionpackJosé Valim2011-05-151-10/+8
|\ | | | | Make ParamsWrapper use a well-defined API and not rely on AR methods
| * Make ParamsWrapper calling newly introduced `Model.attribute_names` instead ↵Prem Sichanugrist2011-05-151-10/+8
| | | | | | | | of `.column_names`
* | Merge pull request #568 from carlosantoniodasilva/fields_for_fixesJosé Valim2011-05-151-0/+14
|\ \ | |/ |/| Fields for fixes for non nested attributes and without object
| * Refactor fields_for to make the api more clear, and fix usage with non ↵Carlos Antonio da Silva2011-05-141-0/+14
| | | | | | | | nested attributes and without object
* | 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.
* Reset ActiveRecord::LogSubscriber runtime at the start of each requestTom Ward2011-05-141-0/+13
| | | | Previously the runtime was reset implicitly when #cleanup_view_runtime was called at the end of most requests. However, this doesn't happen when the request redirects, or send_file is called. Consequently, the ActiveRecord runtime recorded in the logs included the time taken for both the current request and the previous redirect. Explicitly resetting at the start of each request ensures that this can't happen, no matter what occurs previously.
* Escape regex in controller_runtime_test to actually check that the ↵Tom Ward2011-05-141-1/+1
| | | | ActiveRecord message appears
* 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.
* minor cleaningVishnu Atrai2011-05-101-1/+1
|
* Fix previous commit by allowing a proc to be given as response_body. This is ↵José Valim2011-05-101-5/+4
| | | | deprecated and is going to be removed in future releases.
* Merge pull request #446 from danielstutzman/response_body_is_procJosé Valim2011-05-101-0/+38
|\ | | | | New test: setting response_body to a Proc should be supported.