Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | fix the undefined method content_tag #15245 | Rajarshi Das | 2014-09-06 | 1 | -1/+0 | |
| | | | | not required include ActionView::Helpers::TagHelper in test as well | |||||
* | Add support for ARIA attributes in tags | PaoMar | 2014-09-03 | 1 | -0/+7 | |
| | ||||||
* | Ship with rails-html-sanitizer instead. | Kasper Timm Hansen | 2014-09-03 | 1 | -1/+1 | |
| | ||||||
* | Leave all our tests as order_dependent! for now | Matthew Draper | 2014-09-02 | 1 | -0/+5 | |
| | | | | | | | | | We're seeing too many failures to believe otherwise. This reverts commits bc116a55ca3dd9f63a1f1ca7ade3623885adcc57, cbde413df3839e06dd14e3c220e9800af91e83ab, bf0a67931dd8e58f6f878b9510ae818ae1f29a3a, and 2440933fe2c27b27bcafcd9019717800db2641aa. | |||||
* | Remove warning | Rafael Mendonça França | 2014-09-01 | 1 | -2/+2 | |
| | ||||||
* | Allow polymorphic routes with nil when a route can still be drawn | Sammy Larbi | 2014-08-31 | 1 | -2/+34 | |
| | | | | | | | | | | | | | | | | Suppose you have two resources routed in the following manner: ```ruby resources :blogs do resources :posts end resources :posts ``` When using polymorphic resource routing like `url_for([@blog, @post])`, and `@blog` is `nil` Rails should still try to match the route to the top-level posts resource. Fixes #16754 | |||||
* | Merge pull request #16488 from agrobbin/form-label-builder | Jeremy Kemper | 2014-08-25 | 1 | -0/+9 | |
|\ | | | | | Provide a builder for form labels to customize wrapping around I18n content | |||||
| * | provide a builder for form labels to customize wrapping around I18n content | Alex Robbin | 2014-08-20 | 1 | -0/+9 | |
| | | ||||||
* | | just use the placeholder tag value if it is passed as a String | Alex Robbin | 2014-08-22 | 1 | -4/+19 | |
| | | ||||||
* | | add I18n support for `:placeholder` HTML option is passed to form fields | Alex Robbin | 2014-08-22 | 1 | -0/+168 | |
| | | ||||||
* | | Revert "Add I18n support for `:placeholder` HTML option is passed to form ↵ | Jeremy Kemper | 2014-08-21 | 1 | -168/+0 | |
|/ | | | | fields" | |||||
* | 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 | |
| | | | |