aboutsummaryrefslogtreecommitdiffstats
path: root/actionview/test/template
Commit message (Collapse)AuthorAgeFilesLines
* Added ability to pass a custom scrubber to sanitize. Includes test coverage.Timm2014-06-151-0/+36
|
* Marked tests in sanitize_helper_test.rb as pending.Timm2014-06-151-8/+29
|
* Marked a test in form_helper_test.rb as pending because of unknown encoding ↵Timm2014-06-151-0/+2
| | | | ASCII-8BIT output error.
* Marked some tests as pending in date_helper_test.rb.Timm2014-06-151-0/+4
|
* Added some tests for ActionView::Sanitizer.Timm2014-06-151-0/+32
|
* Changed the description of some pending tests. Changed the expected output ↵Timm2014-06-151-3/+4
| | | | of a script test.
* Extracted one highlight test method and marked it as pending.Timm2014-06-151-0/+3
|
* Renamed the SanitizerTest class to SanitersTest, to remove the conflict with ↵Timm2014-06-151-1/+1
| | | | the old SanitizerTest for html-scanner.
* Added video poster sanitization testing (from @vipulnsward).Timm2014-06-151-0/+5
|
* Extracted failing tests in santiizers_test.rb into their own methods and ↵Timm2014-06-151-7/+71
| | | | marked them as pending.
* Changed expected value from '<b>' to empty string.Timm2014-06-151-1/+1
|
* Removed the contains_bad_protocols? method as well as the tests for it. ↵Timm2014-06-151-33/+0
| | | | Loofah already deals with this.
* Reordered form removal with stripping.Timm2014-06-151-1/+1
|
* Added Loofah as a dependency in actionview.gemspec.Timm2014-06-111-0/+330
| | | | | | Implemented ActionView: FullSanitizer, LinkSanitizer and WhiteListSanitizer in sanitizers.rb. Deprecated protocol_separator and bad_tags. Added new tests in sanitizers_test.rb and reimplemented assert_dom_equal with Loofah.
* Removed CaptureHelper#flush_output_buffer as it is only used in tests.Ryan Davis2014-06-042-82/+0
| | | | reviewed: @tenderlove
* Fix AS::NumberHelper results with rationalsJuanjo Bazán2014-05-311-0/+1
| | | | | | | | | | :precision was incorrectly being applied to Rationals before: ActiveSupport::NumberHelper.number_to_rounded Rational(10, 3), precision: 2 => "3.3" after: ActiveSupport::NumberHelper.number_to_rounded Rational(10, 3), precision: 2 => "3.33"
* Merge pull request #15021 from hubertlepicki/allow_custom_host_in_asset_urlRafael Mendonça França2014-05-161-0/+15
|\ | | | | | | Allow custom asset host to be passed in asset_url
| * Allow custom asset host to be passed in asset_urlHubert Łępicki2014-05-081-0/+15
| |
* | minor: point to the right test suite locationazul2014-05-141-1/+1
| |
* | Fix assertion order and :scissors: extra spacesCarlos Antonio da Silva2014-05-131-2/+2
| |
* | Merge pull request #15068 from josepjaume/patch-1Aaron Patterson2014-05-131-0/+6
|\ \ | | | | | | Dup options hash to prevent modifications
| * | Dup options hash to prevent modificationsJosep Jaume Rey2014-05-131-0/+6
| |/ | | | | | | `options[:default]` and `options[:raise]` can be mistakenly added to the `options` hash. This can be a problem if you're reusing the same object.
* / simplified route method name generationCoraline Ada Ehmke + Aaron Patterson2014-05-121-0/+2
|/
* Include label value in i18n attribute lookupJoshua Cody2014-05-061-0/+9
| | | | | | | | | | | | | | | | | | | | | Previously, only the object and method name from the label tag were used when looking up the translation for a label. If a value is given for the label, this ought to be additionally used. The following: # form.html.erb <%= form_for @post do |f| %> <%= f.label :type, value: "long" %> <% end %> # en.yml en: activerecord: attributes: post/long: "Long-form Post" Used to simply return "long", but now it will return "Long-form Post".
* always pass options to the _url methodAaron Patterson2014-05-011-3/+13
|
* always use File.joinphoet2014-05-011-0/+8
|
* Merge pull request #13335 from glorieux/favicon_link_tag_mimetypeRafael Mendonça França2014-04-221-3/+3
|\ | | | | Change favicon_link_tag helper mimetype from image/vnd.microsoft.icon to image/x-icon.
| * Change favicon_link_tag helper mimetype from image/vnd.microsoft.icon to ↵glorieux2014-04-211-3/+3
| | | | | | | | | | | | | | | | image/x-icon. Although the official IANA-registered MIME type for ICO files is image/vnd.microsoft.icon, registered in 2003, it was submitted to IANA by a third party and is not recognized by Microsoft products. The MIME type image/x-icon should be used since is the one recognized by the major browsers on the market.
* | Add test for using ActionView::Helpers::FormHelper.label with block and htmlZachary Scott2014-04-221-0/+7
|/
* Remove wrapper div for inputs in button_toRafael Mendonça França2014-04-171-20/+20
| | | | Related with cbb917455f306cf5818644b162f22be09f77d4b2
* Merge pull request #14738 from tilsammans/pull/11407Rafael Mendonça França2014-04-172-8/+12
|\ | | | | | | | | | | | | Remove wrapping div with inline styles for hidden form fields. Conflicts: actionview/CHANGELOG.md
| * Remove wrapping div with inline styles for hidden form fields.Joost Baaij2014-04-142-8/+12
| | | | | | | | | | | | We are dropping HTML 4.01 and XHTML strict compliance since input tags directly inside a form are valid HTML5, and the absense of inline styles help in validating for Content Security Policy.
* | Use the index on hidden fieldRafael Mendonça França2014-04-141-1/+1
| |
* | `collection_check_boxes` respects `:index` option for the hidden filed name.Vasiliy Ermolovich2014-04-141-0/+7
|/ | | | closes #14147
* CollectionHelpers now accepts a readonly optionMauro George2014-04-121-0/+44
|
* Add test for selected and disabled custom attributes in options_for_selectLaura Paredes2014-03-311-1/+21
|
* Fix date_select option overwriting html classesIzumi Wong-Horiuchi2014-03-241-0/+16
| | | | | with_css_classes: true option overwrites other html classes. Concatenate day month and year classes rather than overwriting.
* Digestor should just rely on the finder to know about the format and the ↵David Heinemeier Hansson2014-03-211-10/+7
| | | | variant -- trying to pass it back in makes a mess of things (oh, and doesnt work)
* fix `number_to_percentage` with `Float::NAN`, `Float::INFINITY`.Yves Senn2014-03-171-0/+3
| | | | | | | Closes #14405. This is a follow-up to 9e997e9039435617b6a844158f5437e97f6bc107 to restore the documented behavior.
* Merge pull request #12662 from nashby/include-hidden-collectionRafael Mendonça França2014-03-151-0/+7
|\ | | | | | | | | | | | | | | add include_hidden option to collection_check_boxes helper Conflicts: actionview/CHANGELOG.md actionview/test/template/form_collections_helper_test.rb
| * add include_hidden option to collection_check_boxes helperVasiliy Ermolovich2013-10-271-0/+7
| |
* | Clarify AV::Digestor.digest method signature docs and deprecation warningJeremy Kemper2014-03-151-2/+2
| |
* | Fix the resolver cache and stop mutating the lookup_contextRafael Mendonça França2014-03-142-21/+0
| | | | | | | | | | Before we had a bug in the resolver cache so the disable_cache were not working when passing options to find
* | Introduce #with_formats_and_variants to prevent problems with mutating ↵Łukasz Strzałkowski2014-03-142-0/+21
| | | | | | | | finder object
* | Add mocked disable_cache for FixtureFinderŁukasz Strzałkowski2014-03-141-0/+4
| |
* | Set format in finderŁukasz Strzałkowski2014-03-141-3/+8
| |
* | Ensure LookupContext in Digestor selects correct variantPiotr Chmolowski2014-03-092-3/+19
| | | | | | | | | | | | | | | | | | | | Related to: #14242 #14243 14293 Variants passed to LookupContext#find() seem to be ignored, so I've used the setter instead: `finder.variants = [ variant ]`. I've also added some more test cases for variants. Hopefully this time passing tests will mean it actually works.
* | Variants in ActionView::DigestorPiotr Chmolowski2014-03-041-8/+27
| | | | | | | | | | | | | | | | | | | | | | Take variants into account when calculating template digests in ActionView::Digest. Digestor#digest now takes a hash as an argument to support variants and allow more flexibility in the future. Old-style arguments have been deprecated. Fixes #14242
* | fixes default attributes for button_tagSergey Prikhodko2014-03-031-0/+5
| |
* | refactor, with_locale is not needed because I18n is mocked.Yves Senn2014-02-241-7/+5
| | | | | | | | | | This is a follow up to #14170. While backporting I recognized that this call is not needed at all.