Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | Merge pull request #21321 from rodzyn/removing_mocha | Kasper Timm Hansen | 2015-08-24 | 1 | -0/+1 | |
|\ \ | |/ |/| | Get rid of mocha tests in actionpack - part 1 | |||||
| * | Get rid of mocha tests - part 1 | Marcin Olichwirowicz | 2015-08-24 | 1 | -0/+1 | |
| | | ||||||
* | | Refactor to remove controller class from route to request | eileencodes | 2015-08-22 | 1 | -1/+1 | |
|/ | | | | | | | | | | This refactoring moves the controller class name that was on the route set to the request. The purpose of this refactoring is for changes we need to move controller tests to integration tests, mainly being able to access the controller on the request instead of having to go through the router. [Eileen M. Uchitelle & Aaron Patterson] | |||||
* | Merge pull request #11352 from xaviershay/dispatcher-api | Rafael Mendonça França | 2015-08-08 | 1 | -8/+4 | |
|\ | | | | | Allow a custom dispatcher to be provided to routing. | |||||
| * | Allow a custom dispatcher to be provided to routing. | Xavier Shay | 2015-08-07 | 1 | -8/+4 | |
| | | ||||||
* | | Refactor to remove DrawOnce module | eileencodes | 2015-08-08 | 1 | -27/+8 | |
|/ | | | | | | We were doing extra work that could be pushed off to Integration test and SharedRoutes. Creating an extra module isn't necessary when those are created by their respective classes. | |||||
* | Using strings or symbols for middleware class names is deprecated. | Aaron Patterson | 2015-08-07 | 1 | -7/+7 | |
| | | | | | | | | | | Convert things like this: middleware.use "Foo::Bar" to this: middleware.use Foo::Bar | |||||
* | Add ActionController API functionality | Santiago Pastorino | 2015-06-11 | 1 | -0/+4 | |
| | ||||||
* | Deprecate `:nothing` option for render method | Mehmet Emin İNAÇ | 2015-05-28 | 1 | -1/+1 | |
| | | | | `head` method works similar to `render` method with `:nothing` option | |||||
* | wrap "require 'drb/unix'" by bgin-end at test/abstract_unit.rb | Toshi MARUYAMA | 2015-04-09 | 1 | -1/+5 | |
| | | | | | | Related: #19617, #19187, #19533, #19689, #19675. 'drb/unix' does not exist on mingw. | |||||
* | 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 | |
| |