Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Return 204 if render is not called in API controllers | Jorge Bejar | 2015-06-11 | 1 | -0/+15 |
| | |||||
* | Add test coverage for implicit render in empty actions | Jorge Bejar | 2015-06-11 | 1 | -0/+14 |
| | |||||
* | Include ParamsWrapper in AC::API | Jorge Bejar | 2015-06-11 | 1 | -0/+26 |
| | | | | | | | | | ParamsWrapper was initially removed from API controllers according to the following discusision: https://github.com/rails-api/rails-api/issues/33 However, we're including it again so Rails API devs can decide whether to enable or disable it. | ||||
* | Rename test methods in api conditional get controller tests | Jorge Bejar | 2015-06-11 | 1 | -2/+2 |
| | |||||
* | Routes resources avoid :new and :edit endpoints if api_only is enabled | Jorge Bejar | 2015-06-11 | 1 | -0/+75 |
| | |||||
* | Remove extra whitespaces | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 |
| | |||||
* | Use new hash syntax | Santiago Pastorino | 2015-06-11 | 2 | -7/+7 |
| | |||||
* | Remove Unneeded ApiPublicExceptions middleware, PublicExceptions already ↵ | Santiago Pastorino | 2015-06-11 | 1 | -37/+0 |
| | | | | does the work | ||||
* | Use nex hash syntax on tests | Santiago Pastorino | 2015-06-11 | 1 | -5/+5 |
| | |||||
* | Add ApiPublicException middleware | Santiago Pastorino | 2015-06-11 | 1 | -2/+39 |
| | |||||
* | Move Model test class inside RenderersApiController namespace | Santiago Pastorino | 2015-06-11 | 1 | -8/+8 |
| | |||||
* | Add ActionController API functionality | Santiago Pastorino | 2015-06-11 | 7 | -0/+184 |
| | |||||
* | Change the `index` arg of `ActionDispatch::Static#new` to a kwarg | Yuki Nishijima | 2015-06-11 | 1 | -1/+1 |
| | |||||
* | extract required_defaults from the conditions hash before constructing the route | Aaron Patterson | 2015-06-08 | 3 | -48/+48 |
| | | | | | this way we can remove the strange "respond_to?" conditional in the `matches?` loop | ||||
* | Merge pull request #19094 from phoet/have_bearer_be_valid_as_well | Rafael Mendonça França | 2015-06-01 | 1 | -1/+8 |
|\ | | | | | Have Bearer be valid as well | ||||
| * | allow `Bearer` as well as `Token` | phoet | 2015-06-01 | 1 | -0/+7 |
| | | |||||
| * | actually test what the name says | phoet | 2015-06-01 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #20138 from tgxworld/deprecated_assert_template | Rafael Mendonça França | 2015-06-01 | 11 | -492/+101 |
|\ \ | | | | | | | Deprecate `assert_template` and `assigns()`. | ||||
| * | | Remove `assigns` and `assert_template`. | Guo Xiang Tan | 2015-05-30 | 11 | -492/+101 |
| |/ | |||||
* | | Merge pull request #20284 from kaspth/fix-caching-test | Rafael Mendonça França | 2015-06-01 | 3 | -3/+9 |
|\ \ | |/ |/| | Move expectation to instance level. | ||||
| * | Replace expectation with state check. | Kasper Timm Hansen | 2015-05-30 | 3 | -3/+9 |
| | | | | | | | | The tests would still pass if the cache call in the rendered templates were removed. | ||||
* | | Fix warning about ambiguous first argument | Arun Agrawal | 2015-05-29 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #20017 from eliotsykes/configurable-static-index-filename | Rafael Mendonça França | 2015-05-28 | 5 | -0/+18 |
|\ \ | | | | | | | | | | config.static_index configures directory Index "index.html" filename | ||||
| * | | config.static_index configures directory index "index.html" filename | Eliot Sykes | 2015-05-28 | 5 | -0/+18 |
| | | | | | | | | | | | | | | | | | | Set `config.static_index` to serve a static directory index file not named `index`. For example, to serve `main.html` instead of `index.html` for directory requests, set `config.static_index` to `"main"`. | ||||
* | | | Deprecate `:nothing` option for render method | Mehmet Emin İNAÇ | 2015-05-28 | 8 | -13/+23 |
|/ / | | | | | | | `head` method works similar to `render` method with `:nothing` option | ||||
* | | Merge pull request #19808 from byroot/action-parameter | Rafael Mendonça França | 2015-05-27 | 1 | -0/+20 |
|\ \ | | | | | | | [PoC] Stop shadowing parameters named `action` | ||||
| * | | Allow to parameters named `action` or `controller` from AC::TestCase helpers | Jean Boussier | 2015-04-18 | 1 | -0/+20 |
| | | | |||||
* | | | Add assertion for get? method into test cases | Mehmet Emin İNAÇ | 2015-05-27 | 1 | -0/+1 |
| | | | |||||
* | | | Merge pull request #17788 from sivagollapalli/master | Arthur Nogueira Neves | 2015-05-20 | 1 | -0/+5 |
|\ \ \ | | | | | | | | | Issue#17703 Test case for tempfile attribute | ||||
| * | | | Issue#17703 Test case for tempfile attribute | Siva Gollapalli | 2015-04-16 | 1 | -0/+5 |
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | + To avoid regression I am adding this test case on action pack Changed to assert from assert_equal + Added msg to assertion Modified msg | ||||
* | | | Prefer assert_not over refute | Rafael Mendonça França | 2015-05-18 | 2 | -2/+2 |
| | | | |||||
* | | | ActionDispatch::Journey::Routes#empty? test cases | Valentine Valyaeff | 2015-05-19 | 2 | -0/+16 |
| |/ |/| | |||||
* | | Silence ambiguous first argument warning | Prem Sichanugrist | 2015-05-04 | 1 | -1/+1 |
| | | | | | | | | | | | | | | This silences: actionpack/test/journey/route_test.rb:33: warning: ambiguous first argument; put parentheses or a space even after `/' operator | ||||
* | | Give authentication methods the ability to customize response message. | Keenan Brock | 2015-05-03 | 2 | -6/+6 |
| | | | | | | | | Digest allowed the messages. Add the same feature to basic and token | ||||
* | | Merge pull request #18392 from brainopia/fix_route_requirements | Arthur Nogueira Neves | 2015-04-27 | 1 | -0/+8 |
|\ \ | | | | | | | Correct route requirements by overriding defaultls | ||||
| * | | Correct route requirements by overriding defaultls (fixes #18373) | brainopia | 2015-01-08 | 1 | -0/+8 |
| | | | |||||
* | | | Merge pull request #19904 from zzak/rm_route_wrapper_internal_hacks | Rafael Mendonça França | 2015-04-26 | 1 | -0/+16 |
|\ \ \ | | | | | | | | | Rm route wrapper internal hacks | ||||
| * | | | Fix rake routes for api apps | Jorge Bejar | 2015-04-25 | 1 | -0/+16 |
| | | | | | | | | | | | | | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | | | | Removed unused code from request_forgery_protection tests | Prathamesh Sonpatki | 2015-04-26 | 1 | -17/+0 |
|/ / / | |||||
* | | | Why do we add a top-level constant here? | Akira Matsuda | 2015-04-25 | 1 | -5/+3 |
| | | | |||||
* | | | Remove unused WorkshopsController class in redirect_test. | Bryce Boe | 2015-04-23 | 1 | -3/+0 |
| |/ |/| | |||||
* | | Merge pull request #19736 from kmcphillips/master | Rafael Mendonça França | 2015-04-14 | 1 | -0/+17 |
|\ \ | | | | | | | Set default form builder for a controller | ||||
| * | | Override default form builder for a controller | Kevin McPhillips | 2015-04-13 | 1 | -0/+17 |
| | | | |||||
* | | | Use `silence_warnings` on `StaticTests` | eileencodes | 2015-04-14 | 1 | -6/+12 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When 7e504927 was merged setting `Encoding.default_internal` and `Encoding.default_external` would throw a warning when the ActionPack tests were run. Example warning: `actionpack/test/dispatch/static_test.rb:12: warning: setting Encoding.default_external` This patch silences the warnings as other similar tests do for setting default_internal and default_external. | ||||
* | | | Revert "Merge pull request #19682 from ↵ | Santiago Pastorino | 2015-04-12 | 1 | -25/+0 |
|/ / | | | | | | | | | | | | | supercaracal/fix_force_ssl_redirection_flash_error" This reverts commit d215620340be7cb29e2aa87aab22da5ec9e6e6a7, reversing changes made to bbbbfe1ac02162ecb5e9a7b560134a3221f129f3. | ||||
* | | fix a wrong feature test method name | Taishi Kasuga | 2015-04-11 | 1 | -1/+1 |
| | | |||||
* | | [Rails4 regression] prevent thin and puma cause error in Non ASCII URL on ↵ | Toshi MARUYAMA | 2015-04-09 | 1 | -0/+22 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Windows * https://github.com/rails/rails/issues/19187 * https://github.com/rails/rails/pull/19533 * https://github.com/macournoyer/thin/issues/268 These are serious Rails 4 regression for Redmine Bitnami Windows users. https://community.bitnami.com/t/problems-with-3-0-1-installation-see-report-inside/30195/ It is not caused on webrick users. Related: * https://github.com/rack/rack/issues/732#issuecomment-67677272 * https://github.com/phusion/passenger/issues/1328 | ||||
* | | 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. | ||||
* | | fix fails to force_ssl_redirection if session_store is disabled | Taishi Kasuga | 2015-04-09 | 1 | -0/+25 |
| | | |||||
* | | Merge pull request #19633 from y00rb/sort_router_parameters_duplicated_keys | Rafael Mendonça França | 2015-04-08 | 2 | -0/+40 |
|\ \ | | | | | | | avoid error when sort mixture keys in symbol and string |