Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Include routes.mounted_helpers into integration tests | Piotr Sarnacki | 2012-06-01 | 1 | -2/+5 |
| | | | | | | | | | | | | | | | In integration tests, you might want to use helpers from engines that you mounted in your application. It's not hard to add it by yourself, but it's unneeded boilerplate. mounted_helpers are now included by default. That means that given engine mounted like: mount Foo::Engine => "/foo", :as => "foo" you will be able to use paths from this engine in tests this way: foo.root_path #=> "/foo" (closes #6573) | ||||
* | Refactor the handling of default_url_options in integration tests | Andrew White | 2012-05-10 | 1 | -7/+24 |
| | | | | | | | | | | | | This commit improves the handling of default_url_options in integration tests by making behave closer to how a real application operates. Specifically the following issues have been addressed: * Options specified in routes.rb are used (fixes #546) * Options specified in controllers are used * Request parameters are recalled correctly * Tests can override default_url_options directly | ||||
* | Remove the leading \n added by textarea on assert_select | Santiago Pastorino | 2012-03-30 | 1 | -0/+1 |
| | |||||
* | Merge pull request #5456 from brianmario/redirect-sanitization | Aaron Patterson | 2012-03-15 | 1 | -1/+1 |
| | | | | Strip null bytes from Location header | ||||
* | TestCase should respect the view_assigns API instead of pulling variables on ↵ | José Valim | 2011-10-02 | 1 | -6/+1 |
| | | | | its own. | ||||
* | Revert "Make process reuse the env var passed as argument" | Santiago Pastorino | 2011-09-24 | 1 | -4/+4 |
| | | | | This reverts commit 0e4748cd415660eb91e63d50aa15cdd027c612dd. | ||||
* | fix assert_select_email to work on non-multipart emails as well as ↵ | Andrew Kaspick | 2011-08-11 | 1 | -2/+2 |
| | | | | converting the Mail::Body to a string to prevent errors. | ||||
* | Remove blank line | Santiago Pastorino | 2011-07-26 | 1 | -1/+0 |
| | |||||
* | refactored 'assert_redirected_to': local call to validate_request! will be ↵ | thoefer | 2011-07-26 | 1 | -5/+4 |
| | | | | called in assert_response already. changed names of local variables in order to recognize the semantics a bit easier. | ||||
* | TODO fix explicitly loading exceptations, autoload removed | Vishnu Atrai | 2011-07-11 | 1 | -0/+1 |
| | |||||
* | Get the fixture_path from self.class instead of ActiveSupport::TestCase. | David Chelimsky | 2011-06-25 | 1 | -1/+1 |
| | | | | | | 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. | ||||
* | remove warning: assigned but unused variable | Santiago Pastorino | 2011-06-08 | 1 | -3/+0 |
| | |||||
* | Add backward compatibility for testing cookies | Andrew White | 2011-06-05 | 1 | -1/+7 |
| | | | | | | | | | | | | | | | | | | | 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. | ||||
* | Add missing require for cookies middleware | Andrew White | 2011-06-04 | 1 | -0/+1 |
| | |||||
* | Refactor ActionController::TestCase cookies | Andrew White | 2011-06-04 | 2 | -26/+1 |
| | | | | | | | | | | | | | | | | | | 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. | ||||
* | Corrected some typos and American vs. Queen's English issues | Lee Reilly | 2011-05-29 | 1 | -4/+4 |
| | |||||
* | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-05-25 | 2 | -10/+10 |
|\ | | | | | | | | | | | Conflicts: actionmailer/lib/action_mailer/base.rb activesupport/lib/active_support/core_ext/kernel/requires.rb | ||||
| * | Remove extra white spaces on ActionPack docs. | Sebastian Martinez | 2011-05-23 | 2 | -10/+10 |
| | | |||||
* | | Restructure TemplateAssertions-related code to eliminate circular requires. | wycats | 2011-05-22 | 2 | -14/+6 |
|/ | | | Also, no need to include dependencies in AS::Concerns inside included blocks. | ||||
* | Merge pull request #247 from goncalossilva/performance_test | Jeremy Kemper | 2011-05-17 | 1 | -13/+6 |
|\ | | | | | Performance tests improved | ||||
| * | Merge branch 'master' of https://github.com/rails/rails into performance_test | Gonçalo Silva | 2011-05-08 | 1 | -1/+1 |
| |\ | | | | | | | | | | | | | Conflicts: activesupport/lib/active_support/testing/performance.rb | ||||
| * \ | Merge branch 'master' of https://github.com/rails/rails into performance_test | Gonçalo Silva | 2011-04-17 | 3 | -212/+24 |
| |\ \ | |||||
| * \ \ | Merge branch 'master' of https://github.com/rails/rails into performance_test | Gonçalo Silva | 2011-04-03 | 1 | -3/+3 |
| |\ \ \ | |||||
| * | | | | remove deprecated documentation | Gonçalo Silva | 2011-03-28 | 1 | -3/+0 |
| | | | | | |||||
| * | | | | move "check for ruby-prof" from AD/testing/performance_test to ↵ | Gonçalo Silva | 2011-03-24 | 1 | -13/+9 |
| | | | | | | | | | | | | | | | | | | | | AS/testing/performance, where it is actually required | ||||
* | | | | | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-05-14 | 1 | -1/+1 |
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/date_helper.rb railties/lib/rails/generators/rails/app/templates/config/initializers/wrap_parameters.rb.tt | ||||
| * | | | | | Follow code conventions on docs | Sebastian Martinez | 2011-05-10 | 1 | -1/+1 |
| | |_|_|/ | |/| | | | |||||
* / | | | | typo | Jason Dew | 2011-05-13 | 1 | -1/+1 |
|/ / / / | |||||
* | | | | Flunk makes a lot more sense, doesnt it (hat tip @tenderlove) | David Heinemeier Hansson | 2011-05-02 | 1 | -1/+1 |
| | | | | |||||
* | | | | We cant use assert_block because its buggy in MiniTest and wont actually ↵ | David Heinemeier Hansson | 2011-04-22 | 1 | -1/+1 |
| |_|/ |/| | | | | | | | | show you the failure message you provide -- instead you just always get a "Expected block to return true" | ||||
* | | | Remove `#among?` from Active Support | Prem Sichanugrist | 2011-04-13 | 2 | -2/+2 |
| | | | | | | | | | | | | | | | | | | 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?`. | ||||
* | | | renames response_from_page_or_rjs -> response_from_page, and extracts the ↵ | Xavier Noria | 2011-04-13 | 1 | -58/+5 |
| | | | | | | | | | | | | RJS in it | ||||
* | | | removes assert_select_rjs | Xavier Noria | 2011-04-13 | 1 | -139/+1 |
| | | | |||||
* | | | Change Object#either? to Object#among? -- thanks to @jamesarosen for the ↵ | David Heinemeier Hansson | 2011-04-12 | 3 | -3/+3 |
| | | | | | | | | | | | | suggestion! | ||||
* | | | Using Object#in? and Object#either? in various places | Prem Sichanugrist | 2011-04-11 | 3 | -3/+7 |
| | | | | | | | | | | | | There're a lot of places in Rails source code which make a lot of sense to switching to Object#in? or Object#either? instead of using [].include?. | ||||
* | | | Make process reuse the env var passed as argument | Santiago Pastorino | 2011-04-06 | 1 | -5/+4 |
| | | | |||||
* | | | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-04-03 | 1 | -8/+8 |
|\ \ \ | |_|/ |/| | | |||||
| * | | Fix grammar, formatting, and cross references | John Firebaugh | 2011-03-28 | 1 | -8/+8 |
| |/ | |||||
* / | use assert_equal so we get normal error messages along with our custom ↵ | Aaron Patterson | 2011-03-30 | 1 | -3/+3 |
|/ | | | | failure message | ||||
* | Improve testing of cookies in functional tests: | Andrew White | 2011-03-06 | 2 | -2/+7 |
| | | | | | | | | | - cookies can be set using string or symbol keys - cookies are preserved across calls to get, post, etc. - cookie names and values are escaped - cookies can be cleared using @request.cookies.clear [#6272 state:resolved] | ||||
* | Remove incorrect assert_recognizes example | Andrew White | 2011-02-13 | 1 | -3/+0 |
| | |||||
* | Fix assert_recognizes with block constraints [#5805 state:resolved] | Andrew White | 2011-02-13 | 1 | -1/+2 |
| | |||||
* | corrected the location of status code symbols | misfo | 2011-01-29 | 1 | -1/+1 |
| | |||||
* | just use alias | Aaron Patterson | 2010-11-30 | 1 | -3/+1 |
| | |||||
* | Remove bazillion warnings from AP suite. | José Valim | 2010-11-23 | 1 | -1/+1 |
| | |||||
* | add respond_to? to ActionDispatch::Integration::Runner | Sven Fuchs | 2010-11-22 | 1 | -0/+4 |
| | | | | since Runner uses method_missing to delegate to the integration session it also should define respond_to? accordingly | ||||
* | Make cookies hash in ActionDispatch::TestProcess indifferent access [#5761 ↵ | David Trasbo | 2010-11-07 | 1 | -1/+1 |
| | | | | | | state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | Remove this require since active_support/testing/default doesn't exist anymore | Santiago Pastorino | 2010-10-20 | 1 | -1/+0 |
| | |||||
* | removing AS::Testing::Default in favor of just undefing default_test | Aaron Patterson | 2010-10-01 | 1 | -2/+1 |
| | |||||
* | _ is not a valid scheme name character, \w includes it and also is redundant ↵ | Emilio Tagua | 2010-09-29 | 1 | -1/+1 |
| | | | | | | with \d. 'The scheme name consists of a letter followed by any combination of letters, digits, and the plus ("+"), period ("."), or hyphen ("-") characters; and is terminated by a colon (":").' |