Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Leave all our tests as order_dependent! for now | Matthew Draper | 2014-09-02 | 1 | -0/+5 |
| | | | | | | | | | We're seeing too many failures to believe otherwise. This reverts commits bc116a55ca3dd9f63a1f1ca7ade3623885adcc57, cbde413df3839e06dd14e3c220e9800af91e83ab, bf0a67931dd8e58f6f878b9510ae818ae1f29a3a, and 2440933fe2c27b27bcafcd9019717800db2641aa. | ||||
* | Use system /tmp when testing actionpack | Agis- | 2014-08-22 | 1 | -3/+1 |
| | | | | | | | | | | | | | | https://github.com/rails/rails/commit/c64bff2c87ebf363703c63ecd4a96d56a1a78364 added support and enabled parallel execution of the actionpack tests. However it introduced https://github.com/rails/rails/commit/c64bff2c87ebf363703c63ecd4a96d56a1a78364 since one cannot connect to a socket file that's inside a Vagrant synced folder due to security restrictions, and DRb tries to. Also rename the temporary files to make it obvious that they're rails-related, since now they're placed outside the project's directory. Fixes https://github.com/rails/rails/commit/c64bff2c87ebf363703c63ecd4a96d56a1a78364 | ||||
* | UnexpectedErrors may reference exceptions that can't be dumped | Aaron Patterson | 2014-08-13 | 1 | -0/+17 |
| | | | | | | UnexpectedError exceptions wrap the original exception, and the original exception may contain a reference to something that can't be marshal dumped which will cause the process to die. | ||||
* | AM, AP, AV, and AMo tests are already order_independent! | Akira Matsuda | 2014-08-13 | 1 | -5/+0 |
| | |||||
* | Nobody sucks so nobody should call this awful method name | Rafael Mendonça França | 2014-08-12 | 1 | -1/+1 |
| | |||||
* | users_dont_suck_but_only_we_suck_and_only_our_tests_are_order_dependent! | Akira Matsuda | 2014-08-12 | 1 | -0/+5 |
| | | | | | Calling ActiveSupport::TestCase.i_suck_and_my_tests_are_order_dependent! in AS::TestCase makes everyone's tests order dependent, which should never be done by the framework. | ||||
* | Use PROCESS_COUNT to define the number of parallel executors | Rafael Mendonça França | 2014-07-23 | 1 | -1/+1 |
| | |||||
* | specify N=0 to prevent parallel tests (just run tests on the main process) | Aaron Patterson | 2014-07-23 | 1 | -2/+6 |
| | |||||
* | Merge branch 'rm-remove-mocha' | Rafael Mendonça França | 2014-07-19 | 1 | -0/+2 |
|\ | | | | | | | | | Conflicts: actionpack/test/abstract_unit.rb | ||||
| * | Stop requiring mocha automatically | Rafael Mendonça França | 2014-07-19 | 1 | -0/+2 |
| | | | | | | | | | | | | | | | | | | | | | | We are planning to remove mocha from our test suite because of performance problems. To make this possible we should stop require mocha on ActionSupport::TestCase. This should not affect applications since users still need to add mocha to Gemfile and this already load mocha. Added FIXME notes to place that still need mocha removal | ||||
* | | reporting is only done in one thread, so we can safely remove the lock | Aaron Patterson | 2014-07-17 | 1 | -1/+1 |
| | | | | | | | | (I think) | ||||
* | | pass the test reporter by reference | Aaron Patterson | 2014-07-17 | 1 | -1/+4 |
| | | | | | | | | | | | | this prevents the array from being dumped as a DRbObject so we can reduce communication with the server. the reporter should always exist on the server side, so we don't have to worry about GC | ||||
* | | do not restart the service, just stop it | Aaron Patterson | 2014-07-17 | 1 | -1/+0 |
| | | |||||
* | | only parallelize on forking systems | Aaron Patterson | 2014-07-17 | 1 | -1/+1 |
| | | |||||
* | | [EXPERIMENTAL] run actionpack tests in parallel | Aaron Patterson | 2014-07-17 | 1 | -0/+57 |
| | | | | | | | | | | only on forking systems though. Feel free to revert this if it causes problems. | ||||
* | | pass the route name to define_url_helper | Aaron Patterson | 2014-07-17 | 1 | -1/+2 |
| | | | | | | | | | | this allows us to avoid 2 hash allocations per named helper definition, also we can avoid a `merge` and `delete`. | ||||
* | | Fix 1.9. uggghhhhhh get it together @tenderlove :bomb: | Aaron Patterson | 2014-07-15 | 1 | -2/+2 |
| | | |||||
* | | fix for 1.9 kwargs syntax | Aaron Patterson | 2014-07-15 | 1 | -1/+2 |
| | | |||||
* | | stop passing recall to url_for | Aaron Patterson | 2014-07-15 | 1 | -2/+2 |
| | | |||||
* | | fix warnings | Aaron Patterson | 2014-07-15 | 1 | -1/+1 |
| | | |||||
* | | stop calling url_for with recall parameters and actually use a request | Aaron Patterson | 2014-07-15 | 1 | -3/+74 |
|/ | |||||
* | Merge pull request #15061 from tgxworld/remove_unnecessary_include | Matthew Draper | 2014-05-29 | 1 | -1/+0 |
|\ | | | | | Remove unnecessary include for integration tests. | ||||
| * | Remove unnecessary include for integration tests. | Guo Xiang Tan | 2014-05-11 | 1 | -1/+0 |
| | | |||||
* | | _recall should be set to a hash or not set | Aaron Patterson | 2014-05-13 | 1 | -2/+2 |
|/ | | | | Then we can avoid nil checks | ||||
* | standardize on jruby_skip & rbx_skip | Gaurish Sharma | 2014-01-13 | 1 | -0/+9 |
| | | | | | | This Adds helpers(jruby_skip & rbx_skip). In Future, Plan is to use these helpers instead of calls directly to RUBY_ENGINE/RbConfig/JRUBY_VERSION | ||||
* | Unused classes in AP tests | Akira Matsuda | 2013-12-25 | 1 | -3/+0 |
| | |||||
* | Disable available locales checks to avoid warnings running the tests | Carlos Antonio da Silva | 2013-12-17 | 1 | -0/+3 |
| | |||||
* | Remove missing integration points of AV extraction | Carlos Antonio da Silva | 2013-12-05 | 1 | -2/+0 |
| | |||||
* | Remove RenderERBUtils from AP (only AV use it) | Łukasz Strzałkowski | 2013-08-25 | 1 | -22/+0 |
| | |||||
* | Remove AV::TestCase from AP | Łukasz Strzałkowski | 2013-08-25 | 1 | -8/+0 |
| | |||||
* | Load AV::Layouts dynamicly via railties | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+2 |
| | |||||
* | Create AbstractController::Rendering interface | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+1 |
| | | | | This interface should be use when implementing renderers. | ||||
* | Fix AP test suite after moving stuff to AV | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+1 |
| | |||||
* | remove unused variable and assignment | Vipul A M | 2013-03-31 | 1 | -1/+1 |
| | |||||
* | Set Thread.abort_on_exception for the whole AS, AP, and AR tests | Akira Matsuda + Koichi Sasada | 2013-01-24 | 1 | -0/+2 |
| | | | | this would give us some more clues in case a test silently dies inside Thread | ||||
* | Add active_support/testing/autorun | Rafael Mendonça França | 2012-12-31 | 1 | -1/+1 |
| | | | | | minitest/autorun load minitest/spec polluting the global namespace with the DSL that we don't want on Rails | ||||
* | Remove observers and sweepers | Rafael Mendonça França | 2012-11-28 | 1 | -1/+0 |
| | | | | | | | | They was extracted from a plugin. See https://github.com/rails/rails-observers [Rafael Mendonça França + Steve Klabnik] | ||||
* | Define a SynchronousQueue for test in Action Pack. | Rafael Mendonça França | 2012-09-12 | 1 | -6/+0 |
| | | | | We don't need to rely on rails/queueing in Action Pack tests | ||||
* | Fix build Rails.queue | Arun Agrawal | 2012-09-12 | 1 | -0/+6 |
| | | | | see 34b23e7110a3a13cf157608cefc9b5701017bf39 | ||||
* | Allow routing concerns to accept a callable | Ernie Miller | 2012-09-03 | 1 | -0/+1 |
| | | | | | | This allows us to make alterations to the generated routes based on the scope of the current mapper, and otherwise allows us to move larger blocks of concerns out of the routes file, altogether. | ||||
* | Extract common controllers to abstract_unit | Rafael Mendonça França | 2012-08-13 | 1 | -0/+29 |
| | |||||
* | push drawing once to it's own module | Aaron Patterson | 2012-08-09 | 1 | -32/+20 |
| | |||||
* | refactor the before_setup hooks to a module | Aaron Patterson | 2012-08-09 | 1 | -13/+12 |
| | |||||
* | switch callbacks to minitest hooks | Aaron Patterson | 2012-08-09 | 1 | -3/+6 |
| | |||||
* | Renamed _path_segments to _recall | Bogdan Gusiev | 2012-08-04 | 1 | -1/+1 |
| | |||||
* | Remove ActionDispatch::Head middleware in favor of Rack::Head | Santiago Pastorino | 2012-07-23 | 1 | -1/+1 |
| | | | | Closes #7110 there's more work to do on rack-cache issue 69 | ||||
* | Allow to use mounted helpers in ActionView::TestCase | Piotr Sarnacki | 2012-06-01 | 1 | -0/+1 |
| | | | | | Similarly to 6525002, this allows to use routes helpers for mounted helpers, but this time in ActionView::TestCase | ||||
* | Remove useless load path modifications | Santiago Pastorino | 2012-05-11 | 1 | -6/+0 |
| | |||||
* | Remove default match without specified method | Jose and Yehuda | 2012-04-24 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | In the current router DSL, using the +match+ DSL method will match all verbs for the path to the specified endpoint. In the vast majority of cases, people are currently using +match+ when they actually mean +get+. This introduces security implications. This commit disallows calling +match+ without an HTTP verb constraint by default. To explicitly match all verbs, this commit also adds a :via => :all option to +match+. Closes #5964 | ||||
* | Remove --http. | José Valim | 2012-03-14 | 1 | -4/+0 |
| |