aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/controller
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | config.action_view.embed_authenticity_token_in_remote_forms is true by defaultPiotr Sarnacki2012-03-281-19/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changed default value for `config.action_view.embed_authenticity_token_in_remote_forms` to `false`. This change breaks remote forms that need to work also without javascript, so if you need such behavior, you can either set it to `true` or explicitly pass `:authenticity_token => true` in form options
* | | | | Added config.action_view.embed_authenticity_token_in_remote_formsPiotr Sarnacki2012-03-281-2/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is a regression introduced in 16ee611fa, which breaks remote forms that should also work without javascript. This commit introduces config option that allows to configure this behavior defaulting to the old behavior (ie. include authenticity token in remote forms by default) Conflicts: actionpack/CHANGELOG.md
* | | | | Add test for default_url_options[:port]Aaron Suggs2012-03-281-0/+12
| | | | |
* | | | | If partial is rendered in controller, grab format from templatePiotr Sarnacki2012-03-271-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously `rendered_format` was set only based on mime types passed in Accept header, which was wrong if first type from Accept was different than rendered partial. The fix is to simply move setting rendered_format to the place where template is available and grab format from the template. If it fails we can fallback to formats passed by Accept header.
* | | | | Merge pull request #5480 from drogus/rendering-issuesJosé Valim2012-03-171-0/+39
| | | | | | | | | | | | | | | | | | | | Fix for #5440
* | | | | Should use an argument in http_digest_authentication_test.rbkennyj2012-03-171-1/+1
| | | | |
* | | | | fixed - warning: ambiguous first argument; put parentheses or even spacesSandeep2012-03-161-1/+1
| | | | |
* | | | | fix typo in redirect testBrian Lopez2012-03-151-1/+1
| | | | |
* | | | | strip null bytes from Location header as wellBrian Lopez2012-03-151-0/+20
| | | | | | | | | | | | | | | | | | | | add tests for stripping \r\n chars since that's already happening
* | | | | Remove ActionController::TestCase#rescue_action_in_public!Piotr Sarnacki2012-03-151-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This method has no effect since exception handling was moved to middlewares and ActionController tests do not use any middlewares.
* | | | | Allow you to force the authenticity_token to be rendered even on remote ↵David Heinemeier Hansson2012-03-141-0/+11
| | | | | | | | | | | | | | | | | | | | forms if you pass true
* | | | | Do not include the authenticity token in forms where remote: true as ajax ↵David Heinemeier Hansson2012-03-141-0/+13
| | | | | | | | | | | | | | | | | | | | forms use the meta-tag value
* | | | | Remove --http.José Valim2012-03-147-197/+0
| | | | |
* | | | | Merge pull request #5357 from bagilevi/sweeper-undefined-method-ignoredPiotr Sarnacki2012-03-141-0/+16
|\ \ \ \ \ | | | | | | | | | | | | Don't ignore call to undefined method in Sweeper
| * | | | | Don't ignore call to undefined method in SweeperLevente Bagi2012-03-061-0/+16
| | | | | |
* | | | | | Add ActionController::HTTPSantiago Pastorino2012-03-147-0/+197
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | More info http://edgeguides.rubyonrails.org/api_app.html [Carlos Antonio da Silva & Santiago Pastorino]
* | | | | | Remove unused methodSantiago Pastorino2012-03-121-4/+0
| | | | | |
* | | | | | Merge pull request #5326 from lest/patch-2Michael Koziarski2012-03-101-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | configure how unverified request will be handled
| * | | | | | configure how unverified request will be handledSergey Nartimov2012-03-091-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | can be configured using `:with` option in `protect_from_forgery` method or `request_forgery_protection_method` config option possible values: - :reset_session (default) - :exception new applications are generated with: protect_from_forgery :with => :exception
* | | | | | | @controller already includes url_helpers in with_routing methodSantiago Pastorino2012-03-101-1/+0
|/ / / / / /
* / / / / / Add test case for #5307Carlos Antonio da Silva2012-03-071-2/+6
|/ / / / /
* | | | | Failing test for mime responder respond_with using a block.Mario Visic2012-03-051-0/+36
| | | | |
* | | | | force response body to be read in assert_templateSergey Nartimov2012-03-051-0/+13
| | | | |
* | | | | Route root helper shortcutBrian Cardarella2012-03-031-0/+9
| | | | | | | | | | | | | | | | | | | | Allow the root route helper to accept just a string
* | | | | 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