diff options
author | Timm <kaspth@gmail.com> | 2013-08-16 19:41:24 +0200 |
---|---|---|
committer | Timm <kaspth@gmail.com> | 2014-06-16 21:04:07 +0200 |
commit | 71aaddbd1464c4ad02bea9bbaada622da0b1feb1 (patch) | |
tree | 931445f07e19416cc1329552cb3da3d467e9d104 /actionview/lib/action_view | |
parent | 75789d5326cdd4718976480dc64d8c5e95f7d069 (diff) | |
download | rails-71aaddbd1464c4ad02bea9bbaada622da0b1feb1.tar.gz rails-71aaddbd1464c4ad02bea9bbaada622da0b1feb1.tar.bz2 rails-71aaddbd1464c4ad02bea9bbaada622da0b1feb1.zip |
Changed: removed @selected and @page variables from HTMLSelector since one method used them. Passed the values directly to there instead.
Diffstat (limited to 'actionview/lib/action_view')
-rw-r--r-- | actionview/lib/action_view/testing/assertions/selector.rb | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/actionview/lib/action_view/testing/assertions/selector.rb b/actionview/lib/action_view/testing/assertions/selector.rb index 616d400536..e3aa6bf0f1 100644 --- a/actionview/lib/action_view/testing/assertions/selector.rb +++ b/actionview/lib/action_view/testing/assertions/selector.rb @@ -310,10 +310,8 @@ module ActionView alias :source :css_selector def initialize(selected, page, args) - @selected, @page = selected, page - # Start with optional element followed by mandatory selector. - @root = determine_root_from(args.first) + @root = determine_root_from(args.first, page, selected) # First or second argument is the selector selector = @css_selector_is_second_argument ? args.shift(2).last : args.shift @@ -361,7 +359,7 @@ module ActionView Nokogiri::XML::NodeSet.new(matches.document, remaining) end - def determine_root_from(root_or_selector) + def determine_root_from(root_or_selector, page, previous_selection = nil) @css_selector_is_second_argument = false if root_or_selector == nil raise ArgumentError, "First argument is either selector or element to select, but nil found. Perhaps you called assert_select with an element that does not exist?" @@ -371,14 +369,14 @@ module ActionView @css_selector_is_second_argument = true root_or_selector - elsif @selected - if @selected.is_a?(Array) - doc = @selected.empty? ? @page.document : @selected[0].document - @selected = Nokogiri::XML::NodeSet.new(doc, @selected) + elsif previous_selection + if previous_selection.is_a?(Array) + Nokogiri::XML::NodeSet.new(previous_selection[0].document, previous_selection) + else + previous_selection end - @selected else - @page + page end end |