Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Removed mention of css_select supporting substitution values. It is not ↵ | Timm | 2014-06-16 | 1 | -7/+1 |
| | | | | tested anywhere. | ||||
* | Updated documentation to state more things about css selectors with ↵ | Timm | 2014-06-16 | 1 | -3/+11 |
| | | | | substitution values. | ||||
* | Reworked the wrapping root in NodeSet implementation in css_select. | Timm | 2014-06-16 | 1 | -3/+5 |
| | |||||
* | Wrapped element to search in NodeSet. Changed selectors to selector. | Timm | 2014-06-16 | 1 | -3/+5 |
| | |||||
* | Moved around alias line. | Timm | 2014-06-16 | 1 | -2/+2 |
| | |||||
* | Returning from filter if matches are empty. | Timm | 2014-06-16 | 1 | -1/+1 |
| | |||||
* | Fixed: no longer wrapped @selected in fragment, since .css works fine ↵ | Timm | 2014-06-16 | 1 | -2/+1 |
| | | | | without it. | ||||
* | Reverted to using documents instead of document fragments, since searching ↵ | Timm | 2014-06-16 | 1 | -3/+6 |
| | | | | via default xml namespaces didn't work. | ||||
* | add_regex returns inspected value for non Regexp objects. Workaround, so ↵ | Timm | 2014-06-16 | 1 | -1/+2 |
| | | | | users don't have to care about enclosing values in double quotes. | ||||
* | Fixed: inadvertently called message method in MiniTest instead of ↵ | Timm | 2014-06-16 | 1 | -1/+1 |
| | | | | selector.message. | ||||
* | Cleaned up SubstitutionContext class. | Timm | 2014-06-16 | 1 | -10/+8 |
| | |||||
* | Simplified assert_select further by moving match filtering into HTMLSelector ↵ | Timm | 2014-06-16 | 1 | -32/+29 |
| | | | | select. | ||||
* | Fixed: now only compares html of children in filter_matches. | Timm | 2014-06-16 | 1 | -1/+1 |
| | |||||
* | Added NodeSet comparison to possible root element in determine_root_from. | Timm | 2014-06-16 | 1 | -1/+1 |
| | |||||
* | Changed html_document to use fragments. Changed response_from_page to be an ↵ | Timm | 2014-06-16 | 1 | -6/+3 |
| | | | | alias of html_document. | ||||
* | Fixed bug by switching to Loofah fragment instead of document. | Timm | 2014-06-16 | 1 | -2/+2 |
| | |||||
* | Changed css_select to not count on multiple selectors. Fixed bug in ↵ | Timm | 2014-06-16 | 1 | -8/+10 |
| | | | | determine_root_from where @selected was an Array. Changed assert_select_encoded to use a fragment instead of a document. | ||||
* | Added a proper substitution context class. Changed ArgumentFilter to be a ↵ | Timm | 2014-06-16 | 1 | -34/+47 |
| | | | | selector. It is now called HTMLSelector. | ||||
* | Fixed nested assert_select bug. Trying to create a full document for a ↵ | Timm | 2014-06-16 | 1 | -1/+1 |
| | | | | nested call that already had a document. | ||||
* | Added implementation for substitution values via Nokogiri's custom pseudo ↵ | Timm | 2014-06-16 | 1 | -5/+35 |
| | | | | classes. | ||||
* | Readded some documentation about substitution values. | Timm | 2014-06-16 | 1 | -2/+12 |
| | |||||
* | Updated documentation to remove mention of HTML::Selector and clarify what ↵ | Timm | 2014-06-15 | 1 | -21/+9 |
| | | | | is expected of a selector now. | ||||
* | Readded html_document method since it is used integration tests. | Timm | 2014-06-15 | 1 | -1/+5 |
| | |||||
* | Changed filter_matches to return a new NodeSet instead of Array. | Timm | 2014-06-15 | 1 | -1/+2 |
| | |||||
* | Initialized @css_selector_is_second_argument in determine_root_from. | Timm | 2014-06-15 | 1 | -0/+1 |
| | |||||
* | Changed early return for filter_matches as well as reassigning matches. ↵ | Timm | 2014-06-15 | 1 | -6/+6 |
| | | | | Meddled with initialize in ArgumentFilter. | ||||
* | Removed redundant comments from assert_select. | Timm | 2014-06-15 | 1 | -3/+1 |
| | | | | Cleaned up a comment. | ||||
* | Added assert_size_match! with the assertions for assert_select. | Timm | 2014-06-15 | 1 | -14/+18 |
| | |||||
* | Changed some documentation for css_select. | Timm | 2014-06-15 | 1 | -1/+1 |
| | |||||
* | Replaced fragment calls with document, since we assume the responses are ↵ | Timm | 2014-06-15 | 1 | -9/+8 |
| | | | | complete documents. | ||||
* | Added filter_matches to reduce line count in assert_select. | Timm | 2014-06-15 | 1 | -24/+19 |
| | |||||
* | Renamed Selector to ArgumentFilter. Put code from HTMLSelector to ↵ | Timm | 2014-06-15 | 1 | -59/+39 |
| | | | | ArgumentFilter. | ||||
* | Removed the custom selected proc. It's no longer needed. | Timm | 2014-06-15 | 1 | -10/+8 |
| | |||||
* | Updated selector to not have reponse_from_page. | Timm | 2014-06-15 | 1 | -5/+6 |
| | |||||
* | Changed css_select and pulled out response_from_page from Selector. | Timm | 2014-06-15 | 1 | -23/+17 |
| | |||||
* | Changed comparison from HTML::Node to Nokogiri::XML::Node in ↵ | Timm | 2014-06-15 | 1 | -1/+1 |
| | | | | assert_select_encoded. | ||||
* | Removed more redundant lines. Removed fix_content block that circumvented a ↵ | Timm | 2014-06-15 | 1 | -7/+1 |
| | | | | bug in html-scanner. | ||||
* | Removed more lines of code in assert_select. | Timm | 2014-06-15 | 1 | -7/+6 |
| | |||||
* | Simplified the first delete_if loop in assert_select to use Loofah's text ↵ | Timm | 2014-06-15 | 1 | -14/+13 |
| | | | | method. | ||||
* | Removed argument and root variable in assert_select proc. | Timm | 2014-06-15 | 1 | -4/+2 |
| | |||||
* | Changed name to selector. And a bunch of other things. | Timm | 2014-06-15 | 1 | -60/+51 |
| | |||||
* | The first attempt at abstracting argument parsing from selection methods. | Timm | 2014-06-15 | 1 | -81/+114 |
| | |||||
* | Removed tag.rb since it has been deprecated. | Timm | 2014-06-15 | 2 | -137/+0 |
| | |||||
* | Corrected documentation bug. | Timm | 2014-06-15 | 1 | -10/+57 |
| | |||||
* | Added Loofah as a dependency in actionview.gemspec. | Timm | 2014-06-11 | 1 | -9/+9 |
| | | | | | | 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. | ||||
* | Handle client disconnect during live streaming | Matthew Draper | 2014-06-08 | 2 | -13/+95 |
| | | | | .. even when the producer is blocked for a write. | ||||
* | adds some details to the rationale of converted_arrays [ci skip] | Xavier Noria | 2014-06-07 | 1 | -0/+4 |
| | |||||
* | Revert "Convert StrongParameters cache to a hash. This fixes an unbounded" | Xavier Noria | 2014-06-07 | 1 | -6/+6 |
| | | | | | | | | | | | | | | | | | | | | We cannot cache keys because arrays are mutable. We rather want to cache the arrays. This behaviour is tailor-made for the usage pattern strongs params is designed for. In a forthcoming commit I am going to add a test that covers why we need to cache by value. Every strong params instance has a live span of a request, the cache goes away with the object. Since strong params have such a concrete intention, it would be interesting to see if there are actually any real-world use cases that are an actual leak, one that practically may matter. I am not convinced that the theoretical leak has any practical consequences, but if it can be shown there are, then I believe we should either get rid of the cache (which is an optimization), or else wipe it in the mutating API. This reverts commit e63be2769c039e4e9ada523a8497ce3206cc8a9b. | ||||
* | [ci skip] Fix capitalization | Akshay Vishnoi | 2014-06-07 | 2 | -3/+3 |
| | |||||
* | remove another wasteful AS::SafeBuffer allocation | Aaron Patterson | 2014-06-06 | 1 | -1/+1 |
| |