aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
Commit message (Collapse)AuthorAgeFilesLines
* Add test for warning and CHANGELOG entryMike Dillon2011-09-101-0/+16
|
* Pull up a method we only use once.Aaron Patterson2011-09-081-5/+1
|
* Conditions must never be equalAaron Patterson2011-09-081-6/+1
|
* Merge pull request #2499 from akaspick/assert_select_email_fixJon Leighton2011-09-071-0/+19
|\ | | | | Fix assert_select_email to work on non-multipart emails as well as converting the Mail::Body to a string to prevent errors.
| * fix assert_select_email to work on non-multipart emails as well as ↵Andrew Kaspick2011-08-111-0/+19
| | | | | | | | converting the Mail::Body to a string to prevent errors.
* | Eliminate newlines in basic auth. fixes #2882Aaron Patterson2011-09-061-0/+8
| |
* | Merge pull request #2872 from sikachu/content_tag_html_safeSantiago Pastorino2011-09-051-0/+12
|\ \ | | | | | | Make sure that result from content_tag_for with collection is html_safe
| * | Make sure that result from content_tag_for with collection is html_safePrem Sichanugrist2011-09-051-0/+12
| | | | | | | | | Thank you @spastorino for catching the empty test in b84cee0, as I totally forgot that I still have to make it html_safe.
* | | Merge pull request #2815 from avakhov/current-page-non-get-requestsSantiago Pastorino2011-09-051-2/+8
|\ \ \ | | | | | | | | current_page? returns false for non-GET requests
| * | | current_page? returns false for non-GET requestsAlexey Vakhov2011-09-021-2/+8
| | | |
* | | | Merge pull request #2799 from tomstuart/3-1-stableSantiago Pastorino2011-09-051-0/+36
| | | | | | | | | | | | | | | | Never return stored content from content_for when a block is given
* | | | add blanks between methodsVijay Dev2011-09-051-0/+3
| |/ / |/| |
* | | Remove empty testSantiago Pastorino2011-09-051-3/+0
| | |
* | | Merge pull request #2863 from ↵Santiago Pastorino2011-09-041-0/+5
|\ \ \ | | | | | | | | | | | | | | | | raybaxter/silence_unitialized_instance_variable_warnings Silence unitialized instance variable warnings
| * | | Tests shouldn't produce warningsRay Baxter2011-09-041-0/+5
| | | |
* | | | * is not allowed in windows file names. Closes #2574 #2847Santiago Pastorino2011-09-042-1/+1
|/ / /
* / / Make `content_tag_for` and `div_for` accepts the array of recordsPrem Sichanugrist2011-09-041-2/+22
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | So instead of having to do this: @items.each do |item| content_tag_for(:li, item) do Title: <%= item.title %> end end You can now do this: content_tag_for(:li, @items) do |item| Title: <%= item.title %> end
* | CookieJar is enumerable. fixes #2795Aaron Patterson2011-09-011-0/+16
| |
* | Backport f443f9cb0c64 to masterGuillermo Iguaran2011-08-311-2/+4
| |
* | Revert "Merge pull request #2647 from dmathieu/no_rescue"Santiago Pastorino2011-08-301-5/+1
| | | | | | | | This reverts commit 125b1b0b9180ec8f0135da185e77250d2b8a4bea.
* | Removed Unused Variable. Arun Agrawal2011-08-261-1/+1
| |
* | Merge pull request #2491 from waynn/fix_symbolSantiago Pastorino2011-08-251-0/+6
|\ \ | | | | | | fix destructive stringify_keys for label_tag
| * | fix destructive stringify_keys for label_tagWaynn Lue2011-08-111-0/+6
| | |
* | | Merge pull request #2644 from guilleiguaran/allow-assets-debuggingSantiago Pastorino2011-08-241-0/+2
|\ \ \ | | | | | | | | Add config.allow_debugging option
| * | | Add config.allow_debugging option to determine if the debug_assets query ↵Guillermo Iguaran2011-08-241-0/+2
| | | | | | | | | | | | | | | | param can be passed by user
* | | | Merge pull request #2668 from guilleiguaran/debug-assets-media-typeSantiago Pastorino2011-08-241-0/+3
|\ \ \ \ | |/ / / |/| | | Debug assets shouldn't ignore media type for stylesheets. Closes #2625
| * | | Debug assets shouldn't ignore media type for stylesheets. Closes #2625Guillermo Iguaran2011-08-231-0/+3
| | | |
* | | | Merge pull request #2647 from dmathieu/no_rescueSantiago Pastorino2011-08-241-1/+5
|\ \ \ \ | |/ / / |/| | | remove useless rescue
| * | | remove useless rescueDamien Mathieu2011-08-241-1/+5
| | | | | | | | | | | | | | | | | | | | params is a method, defined in every controller, which always returns a hash. If it raises a NoMethodError, it means there's a bug somewhere else, which we want to know about.
* | | | Add failing test case for #2654Andrew White2011-08-231-0/+17
| | | |
* | | | Ensure regexp and hash key are UTF-8Andrew White2011-08-231-1/+6
| | | |
* | | | Don't modify params in place - fixes #2624Andrew White2011-08-231-0/+6
|/ / /
* | | Merge pull request #2411 from ai/debug_assets_by_configSantiago Pastorino2011-08-221-14/+6
|\ \ \ | | | | | | | | Debug assets by config
| * | | Allow to debug assets by config.assets.debugAndrey A.I. Sitnik2011-08-221-14/+6
| | | |
* | | | fix escape_javascript for unicode character \u2028.Teng Siong Ong2011-08-211-0/+1
|/ / /
* | | Debug assets by default in development and test environmentsGuillermo Iguaran2011-08-181-6/+22
| | |
* | | Properly escape glob characters.Aaron Patterson2011-08-162-0/+15
| | |
* | | Tags with invalid names should also be stripped in order to preventAaron Patterson2011-08-161-0/+7
| | | | | | | | | | | | XSS attacks. Thanks Sascha Depold for the report.
* | | Use lazy load hooks to set parameter wrapping configuration. This means that ↵Jon Leighton2011-08-161-7/+0
| | | | | | | | | | | | it doesn't force Action Controller / Active Record to load, but it doesn't fail if they have already loaded. Thanks @josevalim for the hint.
* | | Don't refer to ActionController::Base in the wrap_parameters initializer - ↵Jon Leighton2011-08-161-0/+7
| | | | | | | | | | | | use config object instead. Cuts about 15% off the load time. (#734)
* | | Unused variable removed.Arun Agrawal2011-08-151-4/+4
| | |
* | | Methods like status and location are interfering with redirect_to [Closes #2511]Franck Verrot2011-08-141-0/+5
| | |
* | | moving test_generate to an integration test with one assert per testAaron Patterson2011-08-122-108/+183
| | |
* | | when calling url_for with a hash, additional (likely unwanted) values (such ↵Andrew Kaspick2011-08-111-0/+12
| |/ |/| | | | | as :host) would be returned in the hash... calling #dup on the hash prevents this
* | deprecating process_view_pathsAaron Patterson2011-08-092-2/+2
| |
* | generate environment dependent asset digestsIlya Grigorik2011-08-041-0/+13
| | | | | | | | | | | | If two different environments are configured to use the pipeline, but one has an extra step (such as compression) then without taking the environment into account you may end up serving wrong assets
* | Merge pull request #2421 from pivotal-casebook/masterSantiago Pastorino2011-08-041-2/+2
|\ \ | | | | | | Simple_format should not edit it in place. (Fixes https://github.com/rails/rails/issues/1980)
| * | ActionView::Helpers::TextHelper#simple_format should not change the text in ↵Casebook Developer2011-08-041-2/+2
| | | | | | | | | | | | place. Now it duplicates it.
* | | Merge pull request #2356 from waynn/fix_symbolSantiago Pastorino2011-08-041-0/+24
|\ \ \ | |/ / |/| / | |/ fix stringify_keys destructive behavior for most FormTagHelper functions
| * fix stringify_keys destructive behavior for most FormTagHelper functionsWaynn Lue2011-08-031-0/+24
| | | | | | | | add four new tests to verify that the other three methods that called stringify_keys! are fixed. verified that the tests break in master without the code patch. Closes #2355