Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Revert "Merge pull request #19404 from dmathieu/remove_rack_env" | Jeremy Kemper | 2015-03-20 | 1 | -1/+1 | |
| | | | | | | | Preserving RACK_ENV behavior. This reverts commit 7bdc7635b885e473f6a577264fd8efad1c02174f, reversing changes made to 45786be516e13d55a1fca9a4abaddd5781209103. | |||||
* | don't fallback to RACK_ENV when RAILS_ENV is not present | Damien Mathieu | 2015-03-19 | 1 | -1/+1 | |
| | ||||||
* | Run ActionPack test cases in random order. | Guo Xiang Tan | 2015-03-06 | 1 | -5/+0 | |
| | ||||||
* | Avoid parallelizing Action Pack's tests on Rubinius | Robin Dupret | 2015-03-02 | 1 | -2/+2 | |
| | | | | | | | | | | Since the `ForkingExecutor` class seems to be pretty slow on Rubinius due to DRb (c.f. http://git.io/xIVg), let's avoid running tests with it on this platform. Also, the `parallelize_me!` call make the suite to output a bunch of errors due to rubinius/rubinius#2934 since there are thread-safety problems with autoloading. | |||||
* | Removing old helper that was necessary for compatibility with Ruby 1.8.x | Thiago Pradi | 2015-02-22 | 1 | -13/+0 | |
| | ||||||
* | Removing unused constants from abstract_unit | Thiago Pradi | 2015-02-22 | 1 | -2/+0 | |
| | ||||||
* | Removing unused controller from abstract_unit | Thiago Pradi | 2015-02-22 | 1 | -16/+0 | |
| | ||||||
* | Merge pull request #17862 from lucasmazza/lm-hide-action | Rafael Mendonça França | 2014-12-02 | 1 | -1/+2 | |
| | | | | `_generate_paths_by_default` should always be private. | |||||
* | Pass symbol as an argument instead of a block | Erik Michaels-Ober | 2014-11-29 | 1 | -1/+1 | |
| | ||||||
* | Remove session to allow `with_routing` to be called twice. | Guo Xiang Tan | 2014-11-05 | 1 | -0/+1 | |
| | | | | Fixes: https://github.com/rails/rails/issues/16814 | |||||
* | Default to sorting user's test cases for now | Godfrey Chan | 2014-09-08 | 1 | -1/+1 | |
| | | | | | | | | | | | Goals: 1. Default to :random for newly generated applications 2. Default to :sorted for existing applications with a warning 3. Only show the warning once 4. Only show the warning if the app actually uses AS::TestCase Fixes #16769 | |||||
* | 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 |