Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Running isolated tests | Arun Agrawal | 2013-05-28 | 2 | -18/+7 | |
| | | | | These stopped running don't know when. But these should be running now. | |||||
* | 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. | |||||
* | Merge pull request #10667 from rubys/minitest-autorun | Rafael Mendonça França | 2013-05-22 | 1 | -1/+1 | |
|\ | | | | | Eliminate minitest warnings | |||||
| * | Eliminate minitest warnings | Sam Ruby | 2013-05-17 | 1 | -1/+1 | |
| | | | | | | | | https://github.com/seattlerb/minitest/commit/9a57c520ceac76abfe6105866f8548a94eb357b6#L15R8 | |||||
* | | Add has_named_route? to the mapper API | José Valim | 2013-05-20 | 2 | -0/+18 | |
|/ | ||||||
* | Fix named routing regression from 3.2.13 | schneems | 2013-05-16 | 2 | -0/+12 | |
| | | | | | | | | | | | | | | | | | | | | | | When named route that is nested is used in 3.2.13 Example `routes.rb`: ``` resources :nested do resources :builder, :controller => 'nested/builder' end ``` In 3.2.12 and 3.2.12 this named route would work: ``` nested_builder_path(:last_step, :nested_id => "foo") ``` Generating a url that looks like `/nested/foo/builder/last_step`. This PR fixes the regression when building urls via the optimized helper. Any explicit keys set in the options are removed from the list of implicitly mapped keys. Not sure if this is exactly how the original version worked, but this fixes this use case regression. | |||||
* | Merge pull request #10527 from zenspider/squishy_minitest5 | Aaron Patterson | 2013-05-16 | 1 | -1/+3 | |
|\ | | | | | Squishy minitest5 | |||||
| * | Updates to make rails 4 happy with minitest 5: | Ryan Davis | 2013-05-06 | 1 | -1/+3 | |
| | | | | | | | | | | | | | | | | | | | | + Namespace changes, overhaul of runners. + Internal ivar name changes - Removed a logger globally applied to tests that spew everywhere?!? + Override Minitest#__run to sort tests by name. + Reworked testing isolation to work with the new cleaner architecture. - Removed a bunch of tests that just test minitest straight up. I think these changes were all merged to minitest 4 a long time ago. - Minor report output differences. | |||||
* | | Revert "Integration tests support the OPTIONS http method" | Rafael Mendonça França | 2013-05-16 | 2 | -24/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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. | |||||
* | | Merge pull request #10636 from seanwalbran/fix-asset-host-proc-returning-nil | Rafael Mendonça França | 2013-05-16 | 2 | -1/+13 | |
|\ \ | | | | | | | fix rails4 regression with asset_host Proc returning nil | |||||
| * | | restore ability to return nil from asset_host proc, add test | Sean Walbran | 2013-05-15 | 2 | -1/+13 | |
| | | | ||||||
* | | | Revert "Add the options method to action_controller testcase." | Rafael Mendonça França | 2013-05-16 | 2 | -11/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 0303c2325fab253adf5e4a0b738cb469c048f008. Conflicts: actionpack/lib/action_controller/test_case.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. | |||||
* | | | Merge pull request #10591 from acapilleri/cond_callback_test | Aaron Patterson | 2013-05-15 | 1 | -0/+13 | |
|\ \ \ | |/ / |/| | | add test for skip_before_filter with condition | |||||
| * | | add test for skip_before_filter with condition | Angelo capilleri | 2013-05-13 | 1 | -0/+13 | |
| | | | ||||||
* | | | deprecating string based terminators | Aaron Patterson | 2013-05-14 | 1 | -1/+3 | |
| | | | ||||||
* | | | inspect the filter when displaying error messages | Aaron Patterson | 2013-05-14 | 1 | -1/+1 | |
| | | | ||||||
* | | | use public api for testing rather than grabbing instance variables | Aaron Patterson | 2013-05-14 | 1 | -1/+1 | |
| | | | ||||||
* | | | Fixing build broken by this change | Arun Agrawal | 2013-05-13 | 1 | -2/+2 | |
|/ / | | | | | | | c43ca06ca091fc09e2c86bb051ac92b648f12b64 | |||||
* | | Code cleanup for ActionDispatch::Flash#call | Julian Vargas | 2013-05-12 | 1 | -9/+3 | |
| | | | | | | | | | | | | | | The nested `if` was replaced by using `presence` which takes account for the given hash when it is `nil` or when it is empty. The `else` was removed because what it was doing was to assign to `env[KEY]` the value it already had. | |||||
* | | Replace multi_json with json | Erik Michaels-Ober | 2013-05-11 | 1 | -1/+1 | |
| | | ||||||
* | | Merge pull request #10543 from vipulnsward/to_sort | Carlos Antonio da Silva | 2013-05-10 | 1 | -1/+1 | |
|\ \ | | | | | | | Can you improve the commit message a little bit? It's good to describe why the change is happening. | |||||
| * | | sort => sort! on new array | Vipul A M | 2013-05-10 | 1 | -1/+1 | |
| | | | ||||||
* | | | Merge pull request #10510 from think602/master | Xavier Noria | 2013-05-09 | 1 | -0/+1 | |
|\ \ \ | |/ / |/| | | [ci skip] document protect_against_forgery? method | |||||
| * | | [ci skip] document protect_against_forgery? method | Weston Platter | 2013-05-10 | 1 | -0/+1 | |
| | | | ||||||
* | | | Merge pull request #10536 from vipulnsward/fix_exp_warning | Carlos Antonio da Silva | 2013-05-09 | 1 | -1/+0 | |
|\ \ \ | | | | | | | | | remove variable and fix warning | |||||
| * | | | remove variable and fix warning | Vipul A M | 2013-05-09 | 1 | -1/+0 | |
| | | | | ||||||
* | | | | rails -> Rails [ci skip] | Prathamesh Sonpatki | 2013-05-09 | 1 | -1/+1 | |
|/ / / | ||||||
* / / | Fix that JSON and XML exception responses should give the HTTP error message ↵ | Jeremy Kemper | 2013-05-08 | 2 | -4/+4 | |
|/ / | | | | | | | for their status, by default, not the message from the underlying exception | |||||
* | | Test that #fresh_when accepts an array | Brandon Keepers | 2013-05-07 | 1 | -2/+19 | |
| | | | | | | | | https://github.com/rails/etagger/pull/3 | |||||
* | | Merge pull request #10377 from bricker/template_digest_ending_with_number | Rafael Mendonça França | 2013-05-06 | 4 | -3/+12 | |
|\ \ | | | | | | | Allow numbers in partial name for digesting | |||||
| * | | Allow numbers in partial name for digesting | Bryan Ricker | 2013-05-06 | 4 | -3/+12 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add failing test for template with number at the end Use \w for RENDER_DEPENDENCY regex Spacing Add CHANGELOG entry | |||||
* | | | Merge pull request #10446 from dasch/instrument-template-compilation | Rafael Mendonça França | 2013-05-06 | 1 | -2/+9 | |
|\ \ \ | |/ / |/| | | Instrument template compilation | |||||
| * | | Instrument template compilation | Daniel Schierbeck | 2013-05-06 | 1 | -2/+9 | |
| | | | ||||||
* | | | Merge pull request #10478 from cainlevy/patch-1 | Rafael Mendonça França | 2013-05-06 | 1 | -1/+1 | |
|\ \ \ | |_|/ |/| | | use canonical #controller_path logic in controller test cases | |||||
| * | | use canonical #controller_path logic in controller test cases | Lance Ivy | 2013-05-05 | 1 | -1/+1 | |
| | | | ||||||
* | | | remove unused variable | Vipul A M | 2013-05-06 | 1 | -1/+0 | |
| | | | ||||||
* | | | Fix typo in documentation comment | Junya Ogura | 2013-05-06 | 1 | -1/+1 | |
|/ / | ||||||
* | | Added missing `require` to ActionView::LookupContext | Vladimir Strakhov | 2013-05-05 | 1 | -0/+1 | |
| | | ||||||
* | | Fix generating route from engine to other engine | Piotr Sarnacki | 2013-05-03 | 1 | -1/+2 | |
|/ | | | | | | | | | | | | A regression was introduced in 5b3bb6, generating route from within an engine to an another engine resulted in prefixing a path with the SCRIPT_NAME value. The regression was caused by the fact that SCRIPT_NAME should be appended only if it's the SCRIPT_NAME for the application, not if it's SCRIPT_NAME from the current engine. closes #10409 | |||||
* | Merge pull request #10426 from tkrajcar/rescue-h1-fix | Carlos Antonio da Silva | 2013-05-02 | 1 | -0/+6 | |
|\ | | | | | Add styling to h1 for default rescue layout | |||||
| * | Add styling to h1 | Tim Krajcar | 2013-05-02 | 1 | -0/+6 | |
| | | ||||||
* | | use constant for encoding | Nihad Abbasov | 2013-05-02 | 1 | -1/+1 | |
|/ | ||||||
* | Fix failing AP test | Carlos Antonio da Silva | 2013-05-01 | 1 | -1/+1 | |
| | ||||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-05-01 | 1 | -1/+1 | |
|\ | ||||||
| * | fixed typo in select_tag docs | Rncrtr | 2013-04-18 | 1 | -1/+1 | |
| | | ||||||
* | | Merge pull request #9857 from yyyc514/bad_params_should_400 | Aaron Patterson | 2013-04-30 | 3 | -0/+8 | |
|\ \ | | | | | | | failure to parse params should trigger a 400 Bad Request | |||||
| * | | failure to parse params should trigger a 400 Bad Request | Josh Goebel | 2013-03-21 | 3 | -0/+8 | |
| | | | ||||||
* | | | add missing semicolon to journey parser.y | Ben Holley | 2013-04-30 | 1 | -0/+1 | |
| | | | ||||||
* | | | Rack::Mount was replaced by Journey, Fixed comment | Gaurish Sharma | 2013-04-30 | 1 | -1/+1 | |
| | | | ||||||
* | | | rails/master is now 4.1.0.beta | Rafael Mendonça França | 2013-04-29 | 2 | -1193/+3 | |
| | | |