Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | remove method redefinition warnings | Rafael Mendonça França | 2012-09-27 | 2 | -2/+2 |
| | | | | | actionpack/test/template/spec_type_test.rb:32: warning: method redefined; discarding old test_spec_type_wont_match_non_space_characters actionpack/test/controller/spec_type_test.rb:30: warning: previous definition of test_spec_type_wont_match_non_space_characters was here | ||||
* | Is not needed to wrap name inside an array here | Santiago Pastorino | 2012-09-27 | 1 | -1/+1 |
| | |||||
* | Change key not found to param not found | Santiago Pastorino | 2012-09-27 | 1 | -6/+6 |
| | |||||
* | Also includes ConstantLookup dependency for controller and mailer tests | Andy Lindeman | 2012-09-26 | 1 | -0/+1 |
| | |||||
* | Adds missing dependency to ActionView::TestCase::Behavior | Andy Lindeman | 2012-09-26 | 1 | -0/+2 |
| | | | | | | | | | * The module is needed for the `determine_constant_from_test_name` method. * Without it, the including class is required to also include `ActiveSupport::Testing::ConstantLookup` or a `NoMethodError` will be raised upon instantiation of that class. * Issue introduced in c0a24555f9e2749fb94efe1967cb9943db0b6a7e | ||||
* | Merge pull request #7444 from szimek/params_parser_raises_parsing_error | Aaron Patterson | 2012-09-26 | 3 | -7/+18 |
|\ | | | | | Raise generic ParseError exception when ParamsParser fails parsing request params | ||||
| * | Make ActionDispatch::ParamsParser::ParseError#original_exception return the ↵ | Szymon Nowak | 2012-08-27 | 3 | -6/+14 |
| | | | | | | | | original exception. | ||||
| * | Fix ActionDispatch::ParamsParser::ParseError message for XML and JSON parsers. | Szymon Nowak | 2012-08-24 | 3 | -6/+6 |
| | | |||||
| * | Raise generic ParseError exception when ActionDispatch::ParamsParser fails ↵ | Szymon Nowak | 2012-08-24 | 3 | -4/+7 |
| | | | | | | | | parsing request params. | ||||
* | | fixing most tests on Ruby 2.0 | Aaron Patterson | 2012-09-25 | 2 | -3/+3 |
| | | |||||
* | | change ^ and $ anchors in regexp to \A and \z respectively | Nihad Abbasov | 2012-09-25 | 1 | -1/+1 |
| | | | | | | | | http://guides.rubyonrails.org/security.html#regular-expressions | ||||
* | | allow to pass numerical value to size option in image_tag | Nihad Abbasov | 2012-09-25 | 3 | -5/+11 |
| | | | | | | | | | | This will set image's both width and height attributes to value passed in size option. | ||||
* | | Merge pull request #7749 from blowmage/minitest | Aaron Patterson | 2012-09-25 | 9 | -8/+397 |
|\ \ | | | | | | | Improve support for minitest's spec DSL | ||||
| * | | Register AD::IntegrationTest for minitest's spec DSL | Mike Moore | 2012-09-24 | 2 | -0/+44 |
| | | | |||||
| * | | Support helper tests using spec DSL | Mike Moore | 2012-09-24 | 3 | -5/+60 |
| | | | | | | | | | | | | | | | Improve how helper tests to resolve the helper class from the test name. Add tests for helper tests using the minitest spec DSL. | ||||
| * | | Register helper and view tests for minitest's spec DSL | Mike Moore | 2012-09-24 | 2 | -0/+42 |
| | | | |||||
| * | | Allow strings in the controller test describe blocks | Mike Moore | 2012-09-24 | 3 | -0/+130 |
| | | | | | | | | | | | | | | | Allow controller tests using the spec DSL to match strings. Add test coverage for the register_spec_type calls. | ||||
| * | | Support controller tests using spec DSL - fixes #7743 | Mike Moore | 2012-09-24 | 2 | -3/+121 |
| | | | | | | | | | | | | Add tests for controller tests using the minitest spec DSL. | ||||
* | | | Merge pull request #7738 from grosser/verified_request | Rafael Mendonça França | 2012-09-25 | 4 | -8/+24 |
|\ \ \ | | | | | | | | | remove unused Mime::Type#verify_request? | ||||
| * | | | depreacte unused Mime::Type#verify_request? and ↵ | grosser | 2012-09-24 | 4 | -8/+24 |
| |/ / | | | | | | | | | | Mime::Type.browser_generated_types | ||||
* | | | some copy-editing I did in passing | Xavier Noria | 2012-09-25 | 1 | -26/+28 |
| | | | |||||
* | | | gives credit to Aaron in the CHANGELOG entry for AC::Live | Xavier Noria | 2012-09-25 | 1 | -2/+4 |
|/ / | |||||
* | | fixes the suite for uploaded files | Xavier Noria | 2012-09-23 | 1 | -6/+6 |
| | | |||||
* | | documents ActionDispatch::Http::UploadedFile | Xavier Noria | 2012-09-22 | 1 | -6/+55 |
| | | |||||
* | | Put the new CHANGELOG entries always in the top [ci skip] | Rafael Mendonça França | 2012-09-22 | 1 | -8/+10 |
| | | |||||
* | | Merge pull request #7713 from ↵ | Xavier Noria | 2012-09-22 | 3 | -6/+16 |
|\ \ | | | | | | | | | | | | | porras/action-dispatch-upload-delegates-close-to-tempfile Delegate ActionDispatch::Http::UploadedFile#close to tempfile | ||||
| * | | Accept parameters in methods delegated to tempfile | Sergio Gil Pérez de la Manga | 2012-09-22 | 2 | -6/+8 |
| | | | |||||
| * | | Update CHANGELOG.md | Sergio Gil Pérez de la Manga | 2012-09-21 | 1 | -0/+2 |
| | | | |||||
| * | | Delegate ActionDispatch::Http::UploadedFile#close to tempfile | Sergio Gil Pérez de la Manga | 2012-09-20 | 2 | -1/+7 |
| | | | |||||
* | | | Fix MIME type in CHANGELOG | Jo Liss | 2012-09-21 | 1 | -1/+1 |
| | | | |||||
* | | | fix typos and improve AC::StrongParameters documentation [ci skip] | Francesco Rodriguez | 2012-09-20 | 1 | -6/+8 |
|/ / | |||||
* | | update AC::StrongParameters documentation | Francesco Rodriguez | 2012-09-19 | 1 | -8/+53 |
| | | |||||
* | | define permitted? method instead of use an alias to fix rdoc | Francesco Rodriguez | 2012-09-19 | 1 | -2/+12 |
| | | |||||
* | | update AC::Parameters documentation [ci skip] | Francesco Rodriguez | 2012-09-19 | 1 | -1/+149 |
| | | |||||
* | | update AC::ParameterMissing documentation [ci skip] | Francesco Rodriguez | 2012-09-19 | 1 | -2/+9 |
| | | |||||
* | | update AC::ParamsWrapper documentation [ci skip] | Francesco Rodriguez | 2012-09-19 | 1 | -3/+4 |
| | | |||||
* | | Update changelogs to add entries about strong_parameters integration | Guillermo Iguaran | 2012-09-19 | 1 | -0/+14 |
| | | |||||
* | | Set AC::Parameters.permit_all_parameters explicitly to false when ↵ | Guillermo Iguaran | 2012-09-19 | 1 | -1/+1 |
| | | | | | | | | config.action_controller.permit_all_parameters is not present | ||||
* | | Removing to_shorthand from default_controller_and_action. Fixes #6497 | Luiz Felipe | 2012-09-19 | 3 | -3/+20 |
| | | | | | | | | | | When using shortcut routes inside an engine the "to_shorthand" variable is set to true, causing the module scope of the route to not be applied. | ||||
* | | Correct parameter access. | Philip Arndt | 2012-09-19 | 1 | -1/+1 |
| | | | | | | * The params as supplied pass born in authors[0] but not authors[1] so it seems like the test isn't covering what it should be covering. | ||||
* | | fix shadowing outer local variable warning | Sergey Nartimov | 2012-09-19 | 1 | -3/+3 |
| | | |||||
* | | Merge pull request #7251 from rails/integrate-strong_parameters | David Heinemeier Hansson | 2012-09-18 | 12 | -48/+385 |
|\ \ | | | | | | | Integrate strong_parameters in Rails 4 | ||||
| * | | AC::ParameterMissing inherits from KeyError since it's more appropiated than ↵ | Guillermo Iguaran | 2012-09-16 | 1 | -1/+1 |
| | | | | | | | | | | | | IndexError | ||||
| * | | Support fields_for attributes, which may have numeric symbols as hash keys | Guillermo Iguaran | 2012-09-16 | 2 | -0/+22 |
| | | | |||||
| * | | Add config.action_controller.permit_all_attributes to bypass ↵ | Guillermo Iguaran | 2012-09-16 | 4 | -1/+21 |
| | | | | | | | | | | | | StrongParameters protection | ||||
| * | | Change tainted/untainted wording to permitted/forbidden | Guillermo Iguaran | 2012-09-16 | 3 | -26/+26 |
| | | | |||||
| * | | Don't use assert_nothing_raised when assert_equal is used | Guillermo Iguaran | 2012-09-16 | 1 | -4/+2 |
| | | | |||||
| * | | require abstract_unit in parameters tests | Guillermo Iguaran | 2012-09-16 | 3 | -0/+3 |
| | | | |||||
| * | | Remove integration between attr_accessible/protected and ↵ | Guillermo Iguaran | 2012-09-16 | 3 | -48/+2 |
| | | | | | | | | | | | | AC::Metal::ParamsWrapper | ||||
| * | | Integrate ActionController::Parameters from StrongParameters gem | Guillermo Iguaran | 2012-09-16 | 8 | -0/+340 |
| | | |