aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'master' into sessionAaron Patterson2012-05-045-1/+126
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (55 commits) extract deprecated dynamic methods Add some docs and changelog entry Allow overriding exception handling in threaded consumer Allow configuring a different queue consumer actually don't need to expand the aggregates at all #to_sym is unnecessary de-globalise method extract code from AR::Base clean up implementation of dynamic methods. use method compilation etc. Fix ActiveModel README example mention database mapping in getting started guide Remove vestiges of the http_only! config from configuring guide Remove content-length as well Make ActionController#head pass rack-link RouteSet: optimize routes generation when globbing is used Allows assert_redirected_to to accept a regular expression use extract_options! No need to force conversion to Symbol since case ensures it's already one. No need to work around 1.8 warnings anymore. Update command line guide ...
| * Merge pull request #6148 from twinturbo/head-fixJosé Valim2012-05-031-0/+66
| |\ | | | | | | Make ActionController#head pass rack-link
| | * Remove content-length as welltwinturbo2012-05-031-0/+6
| | |
| | * Make ActionController#head pass rack-linktwinturbo2012-05-031-0/+60
| | |
| * | Allows assert_redirected_to to accept a regular expressionAndy Lindeman2012-05-031-0/+4
| | |
| * | Reset the request parameters after a constraints checkAndrew White2012-05-021-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A callable object passed as a constraint for a route may access the request parameters as part of its check. This causes the combined parameters hash to be cached in the environment hash. If the constraint fails then any subsequent access of the request parameters will be against that stale hash. To fix this we delete the cache after every call to `matches?`. This may have a negative performance impact if the contraint wraps a large number of routes as the parameters hash is built by merging GET, POST and path parameters. Fixes #2510.
| * | Merge pull request #6125 from ncri/masterJosé Valim2012-05-021-1/+32
| |\ \ | | | | | | | | Adding proc evaluation for action caching layout parameter
| | * | added proc evaluation for action cache's layout parameterNico2012-05-021-1/+32
| | | |
| * | | Enable ActionDispatch::Http::Headers to support fetchMark Turner2012-05-021-0/+5
| |/ /
* / / testing session store behaviorAaron Patterson2012-05-022-0/+104
|/ /
* | ActionPack: remove tests for hash_for_* methodsBogdan Gusiev2012-05-021-18/+0
| |
* | distance_of_time_in_words: Add/tweak comments to understand time intervals ↵Elliot Winkler2012-04-301-11/+20
| | | | | | | | better
* | distance_of_time_in_words: 45-60 days is about 2 months, not about 1 monthElliot Winkler2012-04-301-1/+3
|/
* Merge pull request #5705 from lest/patch-3Jeremy Kemper2012-04-301-0/+5
|\ | | | | split CDATA end token in cdata_section helper
| * split CDATA end token in cdata_section helperSergey Nartimov2012-04-301-0/+5
| |
* | Merge pull request #6084 from brainopia/support_for_magic_domain_on_all_storesJosé Valim2012-04-301-0/+7
|\ \ | | | | | | Support cookie jar options for all cookie stores
| * | Support cookie jar options for all cookie storesbrainopia2012-04-301-0/+7
| | |
* | | Merge pull request #6082 from brainopia/smarter_cookie_jarJosé Valim2012-04-301-8/+36
|\ \ \ | |_|/ |/| | Stream cookies only if needed
| * | Dont stream back cookie value if it was set to the same valuebrainopia2012-04-301-0/+12
| | |
| * | Dont set cookie header for deletion of unexisting databrainopia2012-04-301-8/+24
| | |
* | | Merge pull request #3726 from JanDupal/fix-date-helper-hiddenJosé Valim2012-04-301-0/+32
|\ \ \ | |/ / |/| | fix Helpers::DateHelper with :use_hidden - hide separators
| * | fix Helpers::DateHelper with :use_hidden - hide separatorsJan Dupal2011-11-221-0/+32
| | |
* | | Merge pull request #6077 from ↵José Valim2012-04-302-37/+61
|\ \ \ | | | | | | | | | | | | | | | | dmitriy-kiriyenko/improve_signature_of_time_in_words_helper Replace boolean argument with an options hash.
| * | | Replace boolean argument with an options hash.Dmitriy Kiriyenko2012-04-302-37/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This replaces `include_seconds` argument with an option key `include_seconds => true` in options hash. Also `time_ago_in_words` now passes options hash, including a `locale` key, which makes in compatible with `distance_of_time_in_words`.
* | | | Merge pull request #6078 from lest/patch-4José Valim2012-04-301-0/+11
|\ \ \ \ | | | | | | | | | | allow send_file/send_data to skip disposition header, closes #2973
| * | | | allow send_file/send_data to skip disposition header, closes #2973Sergey Nartimov2012-04-301-0/+11
| |/ / /
* / / / wrap translate defaults to use translate helper features, closes #1102Sergey Nartimov2012-04-302-1/+25
|/ / /
* | | Remove `button_to_function` and `link_to_function` helpersRafael Mendonça França2012-04-301-30/+0
| | |
* | | Merge pull request #6074 from mark-rushakoff/unused-variablesJeremy Kemper2012-04-291-1/+1
|\ \ \ | | | | | | | | Remove some unused variable assignments
| * | | Remove unused assignment in actionpack date helper testMark Rushakoff2012-04-291-1/+1
| | | |
* | | | Restore interpolation of path option in redirect routesAndrew White2012-04-291-1/+32
| | | |
* | | | Escape interpolated params when redirecting - fixes #5688Andrew White2012-04-291-0/+29
| | | |
* | | | Add failing test case for #6053Andrew White2012-04-291-0/+21
| | | |
* | | | Merge pull request #6059 from rafaelfranca/check_box_invertedJosé Valim2012-04-291-2/+98
|\ \ \ \ | | | | | | | | | | Change check_box to work inverting the checked and unchecked value
| * | | | Change check_box to work inverting the checked and unchecked valueRafael Mendonça França2012-04-291-2/+98
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes: * Boolean with inverted logic * Integer with inverted logic * BigDecimal with inverted logic Fixes #3995
* | | | Allow access to current object_counter variable from layout when rendering ↵Carlos Antonio da Silva2012-04-292-4/+12
| | | | | | | | | | | | | | | | with partial + collection
* | | | Allow layout to access current object being rendered when using render ↵Carlos Antonio da Silva2012-04-291-0/+4
| | | | | | | | | | | | | | | | partial + object
* | | | Allow layout rendering to access current object being rendered when using ↵Carlos Antonio da Silva2012-04-292-2/+7
|/ / / | | | | | | | | | partial + collection
* | / Add changelog entry for jsonp mimetype change, fix failing testCarlos Antonio da Silva2012-04-291-1/+1
| |/ |/| | | | | Fix failing test: Mime::JS generates "text/javascript"
* | Don't convert params if the request isn't HTML - fixes #5341Andrew White2012-04-291-2/+40
| |
* | Merge pull request #2321 from omjokine/masterJosé Valim2012-04-291-1/+1
|\ \ | | | | | | JSONP should use mimetype application/javascript
| * \ Merge remote-tracking branch 'upstream/master'Olli Jokinen2011-12-0174-429/+1708
| |\ \
| * | | fixed test case test_render_json_with_callback to use content_type ↵Olli Jokinen2011-07-281-1/+1
| | | | | | | | | | | | | | | | application/javascript
* | | | Lazy load `default_form_builder` if it's passed as a stringPiotr Sarnacki2012-04-281-0/+17
| | | | | | | | | | | | | | | | closes #3341
* | | | Merge session arg with existing session instead of overwritingAndrew White2012-04-281-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This may break existing tests that are asserting the whole session contents but should not break existing tests that are asserting individual keys - e.g: class SomeControllerTest < ActionController::TestCase setup do session['user_id'] = 1 end test "some test" do get :some_action, nil, { 'another_var' => 2 } # This assertion will now fail assert_equal({ 'another_var' => 2 }, session) # This assertion will still pass assert_equal 2, session['another_var] end end Fixes #1529.
* | | | remove calls to deprecated find(:first) in actionpack test suiteSergey Nartimov2012-04-261-2/+2
| | | |
* | | | Fix the buildYehuda Katz2012-04-251-1/+2
| | | |
* | | | Merge pull request #5980 from gazay/valid_ipsJeremy Kemper2012-04-251-20/+114
|\ \ \ \ | | | | | | | | | | Remote ip logic and validation. IPv6 support.
| * | | | Valid ips v4 and v6. Right logic for working with X-FORWARDED-FOR header and ↵Alexey Gaziev2012-04-251-20/+114
| | | | | | | | | | | | | | | | | | | | tests.
* | | | | Remove default match without specified methodJose and Yehuda2012-04-2439-341/+341
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the current router DSL, using the +match+ DSL method will match all verbs for the path to the specified endpoint. In the vast majority of cases, people are currently using +match+ when they actually mean +get+. This introduces security implications. This commit disallows calling +match+ without an HTTP verb constraint by default. To explicitly match all verbs, this commit also adds a :via => :all option to +match+. Closes #5964