Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Removed warning | Rafael Mendonça França | 2014-08-18 | 1 | -1/+1 |
| | |||||
* | Fix assertion to map the behaviour of the old sanitizer | Rafael Mendonça França | 2014-08-18 | 1 | -1/+1 |
| | |||||
* | Merge pull request #15889 from carnesmedia/model-name | Rafael Mendonça França | 2014-08-17 | 1 | -2/+2 |
|\ | | | | | | | Use #model_name on instances instead of classes | ||||
| * | Use #model_name on instances instead of classes | Amiel Martin | 2014-06-24 | 1 | -2/+2 |
| | | | | | | | | | | | | This allows rails code to be more confdent when asking for a model name, instead of having to ask for the class. Rails core discussion here: https://groups.google.com/forum/#!topic/rubyonrails-core/ThSaXw9y1F8 | ||||
* | | Merge branch 'master' into loofah | Rafael Mendonça França | 2014-08-17 | 3 | -8/+169 |
|\ \ | | | | | | | | | | | | | Conflicts: actionpack/CHANGELOG.md | ||||
| * | | Remove usafe of respond_to in ActionView tests | José Valim | 2014-08-17 | 1 | -3/+1 |
| | | | |||||
| * | | Merge pull request #16438 from agrobbin/input-placeholder-i18n | Jeremy Kemper | 2014-08-14 | 1 | -0/+168 |
| |\ \ | | | | | | | | | Add I18n support for `:placeholder` HTML option is passed to form fields | ||||
| | * | | add I18n support for `:placeholder` HTML option is passed to form fields | Alex Robbin | 2014-08-12 | 1 | -0/+168 |
| | | | | |||||
| * | | | AM, AP, AV, and AMo tests are already order_independent! | Akira Matsuda | 2014-08-13 | 1 | -5/+0 |
| |/ / | |||||
* | | | Merge branch 'master' into loofah | Rafael Mendonça França | 2014-08-12 | 12 | -9/+130 |
|\| | | | | | | | | | | | | | | | | | | | | Conflicts: actionpack/CHANGELOG.md actionpack/test/controller/integration_test.rb actionview/CHANGELOG.md | ||||
| * | | Nobody sucks so nobody should call this awful method name | Rafael Mendonça França | 2014-08-12 | 1 | -1/+1 |
| | | | |||||
| * | | users_dont_suck_but_only_we_suck_and_only_our_tests_are_order_dependent! | Akira Matsuda | 2014-08-12 | 1 | -0/+5 |
| | | | | | | | | | | | | | | | Calling ActiveSupport::TestCase.i_suck_and_my_tests_are_order_dependent! in AS::TestCase makes everyone's tests order dependent, which should never be done by the framework. | ||||
| * | | Fixed #select form builder helper to support block with html output | Bogdan Gusiev | 2014-08-05 | 1 | -0/+13 |
| | | | |||||
| * | | Properly assert for the expected messages | Carlos Antonio da Silva | 2014-07-30 | 1 | -4/+8 |
| | | | | | | | | | | | | | | | | | | The message passed to Minitest's assert_raise is used as output in case the assertion fails, but we can test against the exact message by using the actual exception object that is returned from the assert_raise call. | ||||
| * | | Fix that render layout should also be picked up by the template dependency ↵ | David Heinemeier Hansson | 2014-07-25 | 1 | -0/+15 |
| | | | | | | | | | | | | tracker, but only half-ways. You can add that layout option on the same render call, and both templates should be added to the dependency tree. But thats going to require a more serious rework of the tracker. Please do help fix this part of it too. For now, render layout needs to be on its own line. | ||||
| * | | Merge branch 'rm-remove-mocha' | Rafael Mendonça França | 2014-07-19 | 1 | -0/+2 |
| |\ \ | | | | | | | | | | | | | | | | | Conflicts: actionpack/test/abstract_unit.rb | ||||
| | * | | Stop requiring mocha automatically | Rafael Mendonça França | 2014-07-19 | 1 | -0/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We are planning to remove mocha from our test suite because of performance problems. To make this possible we should stop require mocha on ActionSupport::TestCase. This should not affect applications since users still need to add mocha to Gemfile and this already load mocha. Added FIXME notes to place that still need mocha removal | ||||
| * | | | adding missing test for text area value before type cast | Eugene Gilburg | 2014-07-19 | 1 | -0/+7 |
| | | | | |||||
| * | | | adding missing test coverage | Eugene Gilburg | 2014-07-18 | 1 | -0/+12 |
| | | | | |||||
| * | | | subclass Rails::Engine | Aaron Patterson | 2014-07-16 | 1 | -1/+2 |
| | | | | |||||
| * | | | Build only one PartialIteration object for loop | Rafael Mendonça França | 2014-07-16 | 1 | -5/+8 |
| | | | | |||||
| * | | | No need to have a file to PartialIteration class | Rafael Mendonça França | 2014-07-16 | 1 | -3/+2 |
| | | | | | | | | | | | | | | | | This class is only used on the PartialRenderer. | ||||
| * | | | Added PartialIteration class used when rendering collections | Joel Junström | 2014-07-16 | 5 | -1/+52 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The iteration object is available as the local variable "template_name_iteration" when rendering partials with collections. It gives access to the +size+ of the collection beeing iterated over, the current +index+ and two convinicence methods +first?+ and +last?+ "template_name_counter" variable is kept but is deprecated. [Joel Junström + Lucas Uyezu] | ||||
| * | | | Return an absolute instead of relative path from an asset url in the case of ↵ | Jolyon Pawlyn | 2014-07-15 | 1 | -0/+8 |
| | | | | | | | | | | | | | | | | the `asset_host` proc returning nil | ||||
| * | | | Assert the nature of the original exception | Matthew Draper | 2014-07-12 | 1 | -0/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | Just so it's clearer what's going on in the following assertion. /cc #11993 @robin850 | ||||
| * | | | Merge pull request #11993 from razielgn/actionview-use-name-to-inspect-error | Guillermo Iguaran | 2014-07-11 | 1 | -1/+1 |
| |\ \ \ | | | | | | | | | | | Use NameError#name to assert raised error. | ||||
| | * | | | Use NameError#name to assert raised error. | Federico Ravasio | 2013-11-24 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | This makes the test compatible with other Ruby implementations, which may implement error messages differently. | ||||
| * | | | | Fixed borken tests in AV caused by 013c74d | Godfrey Chan | 2014-07-10 | 1 | -2/+2 |
| | |/ / | |/| | | |||||
* | | | | Use regexp instead exact match for atom test | Rafael Mendonça França | 2014-07-15 | 1 | -3/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | There are two xml namespaces in the response body and the xhtml namespace is not registered in the root node. This create an invalid XML and nokogiri can't navigate using xpath on that node. | ||||
* | | | | All these tests are passing now | Rafael Mendonça França | 2014-07-15 | 3 | -7/+0 |
| | | | | |||||
* | | | | We don't need loofah for the assertions | Rafael Mendonça França | 2014-07-15 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | We can just use nokogiri | ||||
* | | | | Include the selector assertions on the test case | Rafael Mendonça França | 2014-07-15 | 1 | -2/+0 |
| | | | | | | | | | | | | | | | | | | | | We don't need to require users to include this module on ActionMailer::TestCase | ||||
* | | | | Merge pull request #11218 from kaspth/loofah-integration | Rafael Mendonça França | 2014-07-10 | 15 | -1088/+79 |
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | Loofah-integration Conflicts: actionpack/CHANGELOG.md actionview/CHANGELOG.md | ||||
| * | | | Change date helper tests to expect attributes with double quoted strings. | Timm | 2014-06-16 | 1 | -14/+15 |
| | | | | |||||
| * | | | Fix invalid css selectors in form_collections_helper_test.rb. | Timm | 2014-06-16 | 1 | -11/+11 |
| | | | | |||||
| * | | | Remove html-scanner and its tests. | Timm | 2014-06-16 | 6 | -682/+0 |
| | | | | |||||
| * | | | Migrated test away from escaped quotes. | Timm | 2014-06-16 | 1 | -1/+1 |
| | | | | |||||
| * | | | Fixed deprecated selector in form_collections_helper_test.rb with from ↵ | Timm | 2014-06-16 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | catch_invalid_selector. Sweet. | ||||
| * | | | Removed sanitizers- and scrubbers_test.rb. They are in rails-html-sanitizer. | Timm | 2014-06-16 | 2 | -645/+0 |
| | | | | |||||
| * | | | Included DomAssertions in url_helper- and atom_feed_helper_test.rb. | Timm | 2014-06-16 | 2 | -1/+3 |
| | | | | |||||
| * | | | Removed assert_select test file, since it has been moved to rails-dom-testing. | Timm | 2014-06-16 | 1 | -365/+0 |
| | | | | |||||
| * | | | Changed ActiveSupport::Derprecation.silence to assert_deprecated. | Timm | 2014-06-16 | 1 | -2/+2 |
| | | | | |||||
| * | | | Silenced deprecation warnings in the tests. Documentation uses present ↵ | Timm | 2014-06-16 | 1 | -2/+6 |
| | | | | | | | | | | | | | | | | tense. Changed deprecation message to not use you. Also returning from rescue block in catch_invalid_selector to abort reraising the exception. | ||||
| * | | | Added deprecation warning for invalid selectors and skipping assertions. | Timm | 2014-06-16 | 1 | -0/+11 |
| | | | | |||||
| * | | | Added tests for TargetScrubber. | Timm | 2014-06-16 | 1 | -0/+31 |
| | | | | |||||
| * | | | Rounded out PermitScrubber tests. Extracted helper methods to a ScrubberTest ↵ | Timm | 2014-06-16 | 1 | -8/+80 |
| | | | | | | | | | | | | | | | | class. | ||||
| * | | | Moved some tests to scrubbers_test.rb. Added better testing of accessor ↵ | Timm | 2014-06-16 | 2 | -6/+21 |
| | | | | | | | | | | | | | | | | validation. | ||||
| * | | | Added some test coverage for PermitScrubber. | Timm | 2014-06-16 | 1 | -0/+80 |
| | | | | |||||
| * | | | Now returning html if html is blank? in FullSanitizer and ↵ | Timm | 2014-06-16 | 1 | -4/+4 |
| | | | | | | | | | | | | | | | | WhiteListSanitizer. This means it'll return false if called with false, however that is not a valid use case. | ||||
| * | | | Changed test expectation from '<<' to '' with string to sanitize '<<<bad ↵ | Timm | 2014-06-16 | 1 | -4/+2 |
| | | | | | | | | | | | | | | | | html>' in sanitizers_test. |