aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/controller
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge pull request #5242 from rails/opt_routesJosé Valim2012-03-022-7/+7
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Optimize routes generation in simple cases. If you pass to the route helper the same amount of arguments as the required segments, route generation will be optimized as a string interpolation. After this commit, `post_path(post)` is about 6.5 times faster, `post_url(post)` is about 5 times.
| * | | | | Optimize url helpers.Sergey Nartimov + José Valim2012-03-022-7/+7
| | |_|/ / | |/| | |
* / | | | Fix #5238, rendered_format is not set when template is not renderedPiotr Sarnacki2012-03-021-2/+6
|/ / / /
* | | | Adding tests for non-optional glob parametersAndrew White2012-02-261-2/+27
| | | |
* | | | SSL should not be disabled by default in any environment.Pat Allan2012-02-231-14/+0
| | | |
* | | | Remove unused global variable in controller filters testAlexey Vakhov2012-02-231-2/+0
| | | |
* | | | Add config.default_method_for_update to support PATCHDavid Lee2012-02-226-13/+103
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PATCH is the correct HTML verb to map to the #update action. The semantics for PATCH allows for partial updates, whereas PUT requires a complete replacement. Changes: * adds config.default_method_for_update you can set to :patch * optionally use PATCH instead of PUT in resource routes and forms * adds the #patch verb to routes to detect PATCH requests * adds #patch? to Request * changes documentation and comments to indicate support for PATCH This change maintains complete backwards compatibility by keeping :put as the default for config.default_method_for_update.
* | | | Merge pull request #5082 from ↵Aaron Patterson2012-02-201-0/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | willbryant/assigns_should_not_stringify_values_master assigns(:foo) should not convert @foo's keys to strings if it happens to be a hash
| * | | | assigns(:foo) should not convert @foo's keys to strings if it happens to be ↵Will Bryant2012-02-181-0/+5
| | | | | | | | | | | | | | | | | | | | a hash
* | | | | Integration tests support the OPTIONS http methodJeremy Kemper2012-02-191-1/+17
|/ / / /
* | | | Merge pull request #3479 from arvida/ensure-date-header-on-expires-inJosé Valim2012-02-181-0/+7
|\ \ \ \ | | | | | | | | | | Ensure Date header on expires_in
| * | | | Added test for setting of HTTP Date header when calling #expires_inarvida2011-10-311-0/+7
| | | | |
* | | | | let expires_in accept a must_revalidate flagXavier Noria2012-02-171-0/+20
| | | | |
* | | | | adding tests to document behavior for #4817Aaron Patterson2012-02-141-0/+26
| | | | |
* | | | | Rack body respond to each and not to joinSantiago Pastorino2012-02-141-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes undef `to_str' for Rack::Chunked::Body when using caches_action + streaming on an action Closes #5027
* | | | | Merge pull request #3305 from cjolly/request_filtered_envJeremy Kemper2012-02-131-0/+33
|\ \ \ \ \ | | | | | | | | | | | | Exclude rack.request.form_vars from request.filtered_env
| * | | | | Exclude rack.request.form_vars from request.filtered_envChad Jolly2012-01-181-0/+33
| | | | | |
* | | | | | Removing old Controller testThiago Pradi2012-02-121-37/+0
| | | | | |
* | | | | | Cleaning Route generation testsThiago Pradi2012-02-121-30/+0
| | | | | |
* | | | | | replacing the orderhash with hash for ruby-1.9ganesh2012-02-091-3/+2
| | | | | |
* | | | | | Added unit test to cover changes to RouteSet.url_forRyan McGeary2012-02-071-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ActionDispatch::Routing::RouteSet.url_for now handles passing params through to ActionDispatch::Http::Url.url_for Conflicts: actionpack/test/controller/base_test.rb
* | | | | | Fixed force_ssl redirects to include original query paramsRyan McGeary2012-02-071-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `ActionController.force_ssl` redirects http URLs to their https equivalent; however, when a URL contains a query string, the resulting redirect lacked the original query string. Conflicts: actionpack/lib/action_controller/metal/force_ssl.rb
* | | | | | Fix override API response bug in respond_withPrem Sichanugrist2012-02-031-0/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Default responder was only using the given respond block when user requested for HTML format, or JSON/XML format with valid resource. This fix the responder so that it will use the given block regardless of the validity of the resource. Note that in this case you'll have to check for object's validity by yourself in the controller. Fixes #4796
* | | | | | Merge pull request #4529 from ask4prasath/refactorVijay Dev2012-01-251-8/+10
|\ \ \ \ \ \ | | | | | | | | | | | | | | Refactored class methods on address render test
| * | | | | | Refactored class methods on address render testprasath2012-01-191-8/+10
| |/ / / / /
* | | | | | Merge pull request #3775 from karevn/masterAaron Patterson2012-01-241-1/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | Please pull my changes - they fix a rare problem with tests framework
| * | | | | | Fix: when using subdomains and constraints, request params were not passed ↵karevn2011-11-281-1/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | to constraints callback
* | | | | | | Added custom regexps to ASTs that have literal nodes on either side ofAaron Patterson2012-01-231-0/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | symbol nodes. Fixes #4585
* | | | | | | moved the `get` testing method to a moduleAaron Patterson2012-01-231-11/+2
| | | | | | |
* | | | | | | Remove not used requires from csrf helper file and testCarlos Antonio da Silva2012-01-211-7/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These requires were added in a87b92d and the implementation changed in 2cdc1f0, removing the need for them.
* | | | | | | Removed unused assigns from ActionView::Template::Errorbrainopia2012-01-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They existed since initial rails commit by DHH but lost use a long time ago
* | | | | | | Use performed? instead of checking for response_bodyCarlos Antonio da Silva2012-01-191-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Check for performed? instead of response_body * Change performed? to return a boolean * Refactor AC::Metal#response_body= to reuse variable
* | | | | | | Do not deprecate performed.José Valim2012-01-191-0/+6
| |/ / / / / |/| | | | |
* | | | | | ActionView now has its own loggerRafael Mendonça França2012-01-181-1/+2
| | | | | |
* | | | | | Refactor FormBuilder arguments and default configCarlos Antonio da Silva2012-01-181-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Do not reopen AV::Base to define default form builder Inside the load hook we are already in AV::Base context. * Do not pass the given block to the form builder The block is evaluated in fields_for context using capture, with the builder as argument. This means we do not need to give the block to the FormBuilder itself.
* | | | | | Bring back rendering templates that start with / in nested structuresCarlos Antonio da Silva2012-01-171-0/+22
| | | | | |
* | | | | | Remove deprecated default_charset= from AC::BaseCarlos Antonio da Silva2012-01-171-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should be set globally as a configuration, using `config.action_dispatch.default_charset` instead
* | | | | | Move render_to_body logic to return a spaced string to AC::RenderingCarlos Antonio da Silva2012-01-171-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This seems to be required only when calling render :partial with an empty collection from a controller. This call happens to return no content, letting the response body empty, which means to Rails that it should go on and try to find a template to render based on the current action name, thus failing hard. Although tests keep all green, we need to check a better way to fix this.
* | | | | | Remove method missing handling when action is not found, use action missing ↵Carlos Antonio da Silva2012-01-171-45/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | instead Do not create a method_missing method to handle not found actions, use the action_missing method provided by Rails instead.
* | | | | | Rename test class and fix tests to keep consistencyCarlos Antonio da Silva2012-01-171-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | Based on 50d23bc2bd3653b3c66e480c22ae97c5f7fd7f62.
* | | | | | Remove deprecated logic to render templates starting with /Carlos Antonio da Silva2012-01-171-22/+0
| | | | | | | | | | | | | | | | | | | | | | | | render :template => "/foo/bar"
* | | | | | Remove rescue_action from compatibility module and testsCarlos Antonio da Silva2012-01-1716-56/+1
| | | | | |
* | | | | | Remove other old compatibility constantsCarlos Antonio da Silva2012-01-172-8/+8
| | | | | |
* | | | | | Remove old compatibility methods not being usedCarlos Antonio da Silva2012-01-172-9/+0
| | | | | |
* | | | | | When force redirecting to SSL, make sure that the session is kept. As we're ↵Marcin Bunsch and Przemek Dąbek2012-01-121-1/+37
| | | | | | | | | | | | | | | | | | | | | | | | moving from a non-secure to secure environment, it's safe
* | | | | | AC:TestCase::Behavior#head only accepts parameters as second argumentRafael Mendonça França2012-01-101-0/+4
| | | | | |
* | | | | | Whitespaces :scissors:Rafael Mendonça França2012-01-101-4/+4
| | | | | |
* | | | | | Fix ActionController::TestCase::Behavior.headRafael Mendonça França2012-01-101-0/+5
| | | | | |
* | | | | | Merge pull request #4371 from rafaelfranca/test_unit_cleanupAaron Patterson2012-01-091-3/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Test unit cleanup
| * | | | | | Test constant is not defined anymoreRafael Mendonça França2012-01-061-3/+5
| | | | | | |
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-06 04:30:15 +0000