Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix AC::TemplateAssertions instance variables not resetting. | Guo Xiang Tan | 2014-07-21 | 1 | -0/+1 |
| | | | | Fixes https://github.com/rails/rails/issues/16119. | ||||
* | routed applications will respond to these methods | Aaron Patterson | 2014-07-15 | 1 | -1/+1 |
| | |||||
* | remove the mounted_helpers respond_to check | Aaron Patterson | 2014-07-07 | 1 | -1/+1 |
| | | | | It always responds to mounted_helpers now | ||||
* | always test against a routed rack app so there are always url_helpers | Aaron Patterson | 2014-07-07 | 1 | -1/+1 |
| | |||||
* | Remove symbolized_path_parameters. | Guo Xiang Tan | 2014-07-02 | 1 | -1/+1 |
| | | | | This pull request is a continuation of https://github.com/rails/rails/commit/925bd975 and https://github.com/rails/rails/commit/8d8ebe3d. | ||||
* | Remove unused parameter. | Guo Xiang Tan | 2014-06-15 | 1 | -1/+1 |
| | |||||
* | Merge pull request #15061 from tgxworld/remove_unnecessary_include | Matthew Draper | 2014-05-29 | 1 | -6/+0 |
|\ | | | | | Remove unnecessary include for integration tests. | ||||
| * | Remove unnecessary include for integration tests. | Guo Xiang Tan | 2014-05-11 | 1 | -6/+0 |
| | | |||||
* | | Remove unnecessary require of Minitest. | Guo Xiang Tan | 2014-05-24 | 1 | -1/+0 |
| | | | | | | | | Minitest has already been required when calling Minitest.autorun. | ||||
* | | Form full URI as string to be parsed in Rack::Test. | Guo Xiang Tan | 2014-05-21 | 1 | -7/+5 |
| | | | | | | | | There are performance gains to be made by avoiding URI setter methods. | ||||
* | | Fix regression on `assert_redirected_to`. | Arthur Neves | 2014-05-13 | 1 | -1/+7 |
|/ | | | | | | | | | | `assert_redirected_to` would fail if there is no controller set on a `ActionDispatch::IntegrationTest`, as _compute_redirect_to_location would be called on the controller to build the url. This regression was introduced after 1dacfbabf3bb1e0a9057dd2a016b1804e7fa38c0. [fixes #14691] | ||||
* | Remove surplus period from assertion messages | Marcus Stollsteimer | 2014-04-13 | 1 | -3/+3 |
| | |||||
* | Swapped parameters of assert_equal in assert_select | Vishal Lal | 2014-03-22 | 1 | -1/+1 |
| | |||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-12-20 | 1 | -1/+1 |
|\ | |||||
| * | Typos. return -> returns. [ci skip] | Lauro Caetano | 2013-12-03 | 1 | -1/+1 |
| | | |||||
* | | Change all `MiniTest` to `Minitest` since, `MiniTest` namespace has been ↵ | Vipul A M | 2013-12-18 | 2 | -2/+2 |
| | | | | | | | | | | | | renamed to `Minitest` Ref: https://github.com/seattlerb/minitest/blob/master/History.txt | ||||
* | | Better error message for typos in assert_response argument. | Victor Costan | 2013-11-25 | 1 | -0/+3 |
|/ | | | | | This commit makes it really easy to debug errors due to typos like "assert_response :succezz". | ||||
* | Fix incorrect assert_redirected_to failure message | Derek Prior | 2013-09-19 | 1 | -15/+5 |
| | | | | | | | | | | | | | | | In some instances, `assert_redirected_to` assertion was returning an incorrect and misleading failure message when the assertion failed. This was due to a disconnect in how the assertion computes the redirect string for the failure message and how `redirect_to` computes the string that is actually used for redirection. I made the `_compute_redirect_to_loaction` method used by `redirect_to` public and call that from the method `assert_redirect_to` uses to calculate the URL. The reveals a new test failure due to the regex used by `_compute_redirect_to_location` allow `_` in the URL scheme. | ||||
* | Pass assert_dom_equal message arg to underlying assertion | Ryan McGeary | 2013-08-04 | 1 | -4/+4 |
| | | | | | #assert_dom_equal and #assert_dom_not_equal both take a "failure" message argument, but this argument was not utilized. | ||||
* | Allow overriding of all headers from passed environment hash | Andrew White | 2013-07-25 | 1 | -5/+5 |
| | | | | | | | Allow REMOTE_ADDR, HTTP_HOST and HTTP_USER_AGENT to be overridden from the environment passed into `ActionDispatch::TestRequest.new`. Fixes #11590 | ||||
* | remove useless `env.merge!(env)` from ActionDispatch::IntegrationTest` | Yves Senn | 2013-07-04 | 1 | -2/+0 |
| | |||||
* | Remove deprecated Rails application fallback for integration testing | Carlos Antonio da Silva | 2013-07-02 | 1 | -4/+0 |
| | | | | Set ActionDispatch.test_app instead. | ||||
* | Require only minitest. | Rafael Mendonça França | 2013-05-22 | 1 | -1/+1 |
| | | | | | minitest/autorun required minitest/spec and we are avoiding to require it. | ||||
* | Eliminate minitest warnings | Sam Ruby | 2013-05-17 | 1 | -1/+1 |
| | | | | https://github.com/seattlerb/minitest/commit/9a57c520ceac76abfe6105866f8548a94eb357b6#L15R8 | ||||
* | Revert "Integration tests support the OPTIONS http method" | Rafael Mendonça França | 2013-05-16 | 1 | -7/+1 |
| | | | | | | | | | | | | | | This reverts commit ad46884af567d6f8d6d8d777f372c39e81a560ba. Conflicts: actionpack/CHANGELOG.md actionpack/lib/action_dispatch/testing/integration.rb actionpack/test/controller/integration_test.rb Reason: It will conflict with a lot of test cases. Better to call `process` directly since this is a very uncommon HTTP method. Fixes #10638. | ||||
* | Mark unused variables and make some style fixes | Agis Anastasopoulos | 2013-04-08 | 3 | -5/+5 |
| | | | | It'd be a nice convention to mark the unused variables like this, now that Ruby 2 will issue no warnings for such vars being unused. | ||||
* | `Http::Headers` directly modifies the passed environment. | Yves Senn | 2013-03-15 | 1 | -2/+3 |
| | | | | | | | | | The env hash passed to `Http::Headers#new` must be in env format. Also be aware that the passed hash is modified directly. docs and test-cases for setting headers/env in functional tests. Follow up to #9700. | ||||
* | allow headers and env to be passed in `IntegrationTest`. | Yves Senn | 2013-03-13 | 1 | -34/+34 |
| | | | | Closes #6513. | ||||
* | Ensure port is set when passed via the process method | Andrew White | 2013-01-15 | 1 | -1/+1 |
| | |||||
* | extract PerformanceTest into rails-performance_tests gem | Yves Senn | 2013-01-10 | 1 | -10/+0 |
| | |||||
* | Missing requires | Akira Matsuda | 2013-01-06 | 1 | -0/+1 |
| | |||||
* | Remove unnecessary begin..rescue..end, use only rescue | Akira Matsuda | 2013-01-06 | 1 | -5/+3 |
| | |||||
* | Alias refute methods to assert_not and perfer assert_not on tests | Rafael Mendonça França | 2012-12-31 | 1 | -1/+1 |
| | |||||
* | Inherit from MiniTest::Unit::TestCase instead of MiniTest::Spec | Rafael Mendonça França | 2012-12-31 | 1 | -3/+0 |
| | |||||
* | Cleans and removes 'Examples' tag [ci skip] | Alvaro Pereyra | 2012-12-01 | 1 | -2/+0 |
| | |||||
* | Use File.join to better integrate fixture_path in fixture_file_upload. | Ben Woosley | 2012-11-26 | 1 | -5/+7 |
| | |||||
* | Removed extra call to #diff in #assert_recognizes. assert_equal calls it for ↵ | Ryan Davis | 2012-11-17 | 1 | -4/+7 |
| | | | | you and shows the diff. Also delayed message calculation so the cost of the diff on success is now gone. | ||||
* | Deprecate Hash#diff. | Steve Klabnik | 2012-11-09 | 1 | -3/+1 |
| | | | | | | It's no longer used in Rails any more. See https://github.com/rails/rails/pull/8142\#issuecomment-10227297 for more | ||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-11-03 | 6 | -46/+46 |
|\ | | | | | | | | | | | | | Conflicts: actionpack/lib/action_controller/metal/mime_responds.rb activerecord/lib/active_record/attribute_methods.rb guides/source/working_with_javascript_in_rails.md | ||||
| * | 1.9 hash syntax changes to docs | AvnerCohen | 2012-10-31 | 6 | -46/+46 |
| | | |||||
* | | Make caller attribute in deprecation methods optional | Alexey Gaziev | 2012-10-30 | 1 | -2/+1 |
| | | |||||
* | | Add missing require for minitest assertions. Breaks calling 'app' in console ↵ | Jeremy Kemper | 2012-10-25 | 1 | -0/+1 |
|/ | | | | otherwise. See 94248ffe1e6f815b8ffab4eb752b60c1e4459d25. | ||||
* | Register AD::IntegrationTest for minitest's spec DSL | Mike Moore | 2012-09-24 | 1 | -0/+3 |
| | |||||
* | Merge pull request #7530 from robin850/master | Xavier Noria | 2012-09-05 | 1 | -1/+1 |
|\ | | | | | Fix a typo in AD | ||||
| * | Fix a typo | Robin Dupret | 2012-09-05 | 1 | -1/+1 |
| | | |||||
* | | Move action_controller/vendor/html-scanner to action_view | Piotr Sarnacki | 2012-08-28 | 3 | -3/+4 |
| | | | | | | | | | | | | This is another step in moving Action View's dependencies in Action Pack to Action View itself. Also, HtmlScanner seems to be better suited for views rather than controllers. | ||||
* | | removes usage of Object#in? from the code base (the method remains defined ↵ | Xavier Noria | 2012-08-06 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | by Active Support) Selecting which key extensions to include in active_support/rails made apparent the systematic usage of Object#in? in the code base. After some discussion in https://github.com/rails/rails/commit/5ea6b0df9a36d033f21b52049426257a4637028d we decided to remove it and use plain Ruby, which seems enough for this particular idiom. In this commit the refactor has been made case by case. Sometimes include? is the natural alternative, others a simple || is the way you actually spell the condition in your head, others a case statement seems more appropriate. I have chosen the one I liked the most in each case. | ||||
* | | load active_support/core_ext/object/inclusion in active_support/rails | Xavier Noria | 2012-08-02 | 3 | -3/+0 |
| | | |||||
* | | load active_support/core_ext/object/blank in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 |
| | | |||||
* | | Updated with_routing example to 3.x syntax [ci skip] | Wojciech Wnętrzak | 2012-07-13 | 1 | -7/+4 |
|/ |