aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| | * | | | | | Added deprecation warning for invalid selectors and skipping assertions.Timm2014-06-162-5/+35
| | | | | | | |
| | * | | | | | Nokogiri leaves '<' unescaped, so the assert_select looking for '&lt;' will ↵Timm2014-06-161-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | never work. Switched to assert_matching the reponse body.
| | * | | | | | Added tests for TargetScrubber.Timm2014-06-161-0/+31
| | | | | | | |
| | * | | | | | Rounded out PermitScrubber tests. Extracted helper methods to a ScrubberTest ↵Timm2014-06-161-8/+80
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | class.
| | * | | | | | Moved some tests to scrubbers_test.rb. Added better testing of accessor ↵Timm2014-06-162-6/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | validation.
| | * | | | | | Added some test coverage for PermitScrubber.Timm2014-06-161-0/+80
| | | | | | | |
| | * | | | | | Changed PermitScrubber's direction to bottom up to align better with ↵Timm2014-06-161-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Loofah's strip scrubber.
| | * | | | | | Now only requiring Loofah in the places where it is needed.Timm2014-06-163-1/+4
| | | | | | | |
| | * | | | | | Minor rewording in TargetScrubber documentation.Timm2014-06-161-3/+3
| | | | | | | |
| | * | | | | | Added deprecation notice to actionpack changelog.Timm2014-06-161-0/+6
| | | | | | | |
| | * | | | | | Updated Action View changelog entries with more information about the ↵Timm2014-06-161-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | changes in the API. Removed mention of mentor (at his request).
| | * | | | | | Stylistic improvements in ActionView::Assertions::DomAssertions.Timm2014-06-161-0/+4
| | | | | | | |
| | * | | | | | Now returning html if html is blank? in FullSanitizer and ↵Timm2014-06-162-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | WhiteListSanitizer. This means it'll return false if called with false, however that is not a valid use case.
| | * | | | | | Stylistic improvements. Some light documentation for remove_xpaths.Timm2014-06-161-6/+8
| | | | | | | |
| | * | | | | | Removed require's for html-scanner.Timm2014-06-163-2/+3
| | | | | | | |
| | * | | | | | Changed test expectation from '<<' to '' with string to sanitize '<<<bad ↵Timm2014-06-161-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | html>' in sanitizers_test.
| | * | | | | | Added deprecation warning to ActionDispatch::Assertions::TagAssertions.Timm2014-06-161-0/+3
| | | | | | | |
| | * | | | | | Removed dom_assertion method since it created bugs.Timm2014-06-161-8/+6
| | | | | | | |
| | * | | | | | Fixed uninitialized constant ActionView::HTML error entered after recent git ↵Timm2014-06-161-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rebase.
| | * | | | | | Removed a bunch of duplicated tests in SanitizeHelperTest.Timm2014-06-161-37/+0
| | | | | | | |
| | * | | | | | Removed unnecessary documentation in DomAssertions.Timm2014-06-161-7/+0
| | | | | | | |
| | * | | | | | Removed case statement in equal_children? used child.element? instead.Timm2014-06-161-2/+1
| | | | | | | |
| | * | | | | | Reworked some internal documentation for equal_attribute_nodes?.Timm2014-06-161-3/+2
| | | | | | | |
| | * | | | | | Changed attributes_are_equal? to equal_attribute_nodes? which takes ↵Timm2014-06-161-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | attribute_nodes instead of nodes.
| | * | | | | | Removed html_strings variable, no splat operator needed.Timm2014-06-161-4/+4
| | | | | | | |
| | * | | | | | Changed back to =~ or == comparison in HTMLSelector filter.Timm2014-06-161-2/+2
| | | | | | | |
| | * | | | | | Simplified the removal of xpaths in remove_xpaths. Added more tests for ↵Timm2014-06-162-9/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | remove_xpaths.
| | * | | | | | Fixed: added apostrophe to possessive noun.Timm2014-06-161-1/+1
| | | | | | | |
| | * | | | | | Extracted: create Regexp from match_with and use =~ to compare instead of ↵Timm2014-06-161-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | checking .is_a? Regexp every time through the loop.
| | * | | | | | Changed conditional check in filter. Removed weird comments.Timm2014-06-161-7/+3
| | | | | | | |
| | * | | | | | Moved: initial assignment of @selector_is_second_argument is now in initialize.Timm2014-06-161-1/+1
| | | | | | | |
| | * | | | | | Changed: using duck typing instead of requiring subclasses of Node and NodeSet.Timm2014-06-161-4/+1
| | | | | | | |
| | * | | | | | Renamed: HTMLSelector css_selector to selector.Timm2014-06-161-7/+7
| | | | | | | |
| | * | | | | | Removed unnecessary lines from HTMLSelector initialize.Timm2014-06-161-3/+1
| | | | | | | |
| | * | | | | | Changed: put selector extraction into selector_from, which is renamed to ↵Timm2014-06-161-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | extract_selector.
| | * | | | | | Changed: HTMLSelector comparisons renamed to equality_tests.Timm2014-06-161-9/+6
| | | | | | | |
| | * | | | | | Changed: removed @selected and @page variables from HTMLSelector since one ↵Timm2014-06-161-10/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | method used them. Passed the values directly to there instead.
| | * | | | | | Changed: return early from compare_doms if the two doms don't have the same ↵Timm2014-06-161-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | number of children.
| | * | | | | | Renamed: remove_xpaths tests no longer prefixed with sanitizer.Timm2014-06-161-6/+6
| | | | | | | |
| | * | | | | | Changed: remove_xpaths called with String returns String, while called with ↵Timm2014-06-162-2/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Loofah fragment returns Loofah fragment. Added tests for this.
| | * | | | | | Fixed: spelling mistake in SanitizeHelperTest.Timm2014-06-161-3/+3
| | | | | | | |
| | * | | | | | Removed duplication in assert_dom_equal and assert_dom_not_equal.Timm2014-06-161-9/+6
| | | | | | | |
| | * | | | | | Changed wording of missing selector argument exception message in css_select.Timm2014-06-161-1/+1
| | | | | | | |
| | * | | | | | Reworked root and selector conditional assignment in css_select.Timm2014-06-161-5/+2
| | | | | | | |
| | * | | | | | Added related Nokogiri issue link to tests that fail with unknown encoding ↵Timm2014-06-162-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ASCII-8BIT.
| | * | | | | | Moved Action Pack changelog message to Action View. Clarified Dom and ↵Timm2014-06-161-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Selector assertions changes in there.
| | * | | | | | Trimmed deprecation message for ActionDispatch::Assertions::SelectorAssertions.Timm2014-06-161-1/+1
| | | | | | | |
| | * | | | | | Removed :nodoc: from PermitScrubber.Timm2014-06-161-1/+0
| | | | | | | |
| | * | | | | | Reworked documentation for PermitScrubber and TargetScrubber.Timm2014-06-161-2/+33
| | | | | | | |
| | * | | | | | Fixed: spelling error.Timm2014-06-161-1/+1
| | | | | | | |