aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
Commit message (Collapse)AuthorAgeFilesLines
...
* | tiny refactoringVasiliy Ermolovich2012-01-291-3/+3
| | | | | | | | | | use :default option with translate method instead of condition simplify separator method
* | Remove not used env[] call in routing_test. Carlos Antonio da Silva2012-01-271-1/+0
| | | | | | The commit 4c321c6d42b6e35f9ead12eb1dccdead03c5abf4 removes the path_params variable assignment, actually the entire line is not used at all.
* | Fix warning. assigned but unused variable - path_paramskennyj2012-01-281-1/+1
| |
* | Do not generate label for attribute when giving nilCarlos Antonio da Silva2012-01-261-0/+4
| |
* | reuse common video/audio tags code and do not modify optionsSergey Nartimov2012-01-261-0/+8
| |
* | do not modify options in image_tagSergey Nartimov2012-01-261-0/+6
| |
* | allow to pass multiple sources to audio/video tags without using arraySergey Nartimov2012-01-251-0/+2
| |
* | 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 #4658 from pwim/unicode-pathsAaron Patterson2012-01-251-0/+20
| | | | | | | | Test for unicode path support
* | Do not mutate given options hash in number helpersCarlos Antonio da Silva2012-01-251-0/+25
| |
* | Make audio_tag able to receive multiple sourcesJanko Marohnić2012-01-251-0/+2
| |
* | Make path_to_video in video_tag apply to each sourceJanko Marohnić2012-01-251-2/+2
| |
* | Remove unused variables to avoid warningsRafael Mendonça França2012-01-241-3/+0
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | Merge pull request #4641 from rafaelfranca/date_select-fix-3-2José Valim2012-01-241-1/+38
| | | | | | | | date_select fixes [3-2-stable]
* | 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-232-11/+17
| | |
* | | Deprecated multi args to http route methodsAaron Patterson2012-01-231-1/+2
| | |
* | | ActionDispatch::Cookies::CookieJar#deleted? predicate method.Paul Annesley2012-01-241-0/+11
| | | | | | | | | | | | | | | Necessary in controller tests to determine if the CookieJar will delete the given cookie.
* | | Raise a better error if anyone wants to create your own ↵Rafael Mendonça França2012-01-231-0/+8
| | | | | | | | | | | | ActionView::Helpers::Tags::Base child and do not implement the render method
* | | Raise exception when date.order includes invalid elementsIndrek Juhkam2012-01-221-0/+8
| | |
* | | Merge pull request #4574 from ihid/masterJosé Valim2012-01-211-0/+16
|\ \ \ | | | | | | | | Fixed regression - unable to use a range as choices for form.select.
| * | | Fix inconsistent ability to use a range as choices for select_tag.Jeremy Walker2012-01-211-0/+16
| | | |
* | | | 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-204-6/+6
| | | | | | | | | | | | | | | They existed since initial rails commit by DHH but lost use a long time ago
* | | Added format to percentageRodrigo Flores2012-01-201-0/+1
| | |
* | | 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
| | |
* | | Refactor button_to helper to use token_tag methodRafael Mendonça França2012-01-192-5/+24
| |/ |/|
* | ActionView now has its own loggerRafael Mendonça França2012-01-182-4/+3
| |
* | Fix warning: assigned but unused variable - resultkennyj2012-01-191-1/+1
| |
* | 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.
* | Fix errors when running entire suite due to class name collisionCarlos Antonio da Silva2012-01-181-23/+23
| | | | | | | | | | | | | | | | | | The Post class is created everywhere in the test suite, and due to that when applying the Array() logic to refactor content_tag_for, some other change to the Post class was breaking record tag tests. The solution is to rename the class to not collide with others already defined in the test suite.
* | Cleanup Post model, no need to require fake_modelsCarlos Antonio da Silva2012-01-181-17/+6
| |
* | Do not mutate options hashCarlos Antonio da Silva2012-01-181-0/+6
| |
* | Cleanup persisted setup for modelCarlos Antonio da Silva2012-01-181-7/+7
| |
* | Mimic AR models yielding when building new records, avoid using tapCarlos Antonio da Silva2012-01-181-6/+8
| |
* | No need for concat as wellCarlos Antonio da Silva2012-01-181-4/+4
| |
* | Add test for content tag with prefix and extra html optionsCarlos Antonio da Silva2012-01-181-4/+10
| |
* | Merge pull request #4488 from rafaelfranca/av-refactorXavier Noria2012-01-171-0/+4
|\ \ | | | | | | ActionView::Helpers::FormHelper refactoring
| * | Adding one test case for label with blockRafael Mendonça França2012-01-171-0/+4
| | |
* | | 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-1722-83/+7
| | |