Merge pull request rails#242 from juank-pa/add_support_for_multiple_matches

Add support for multiple matches

Author: rafaelfranca

CHANGELOG.md

@@ -1,3 +1,5 @@
+- Fix matching mutiple calls for the same selector/function exception
+
 ## 4.3.1
 
 - update jquery to 3.2.1

lib/jquery/assert_select.rb

@@ -91,14 +91,14 @@ def assert_select_jquery(*args, &block)
 
     if block_given?
       @selected ||= nil
-      fragments = Nokogiri::HTML::Document.new
+      fragments = Nokogiri::HTML::Document.new.fragment
 
       if matched_pattern
         response.body.scan(Regexp.new(matched_pattern)).each do |match|
           flunk 'This function can\'t have HTML argument' if match.is_a?(String)
 
-          doc = Nokogiri::HTML::Document.parse(unescape_js(match.first))
-          doc.root.children.each do |child|
+          doc = Nokogiri::HTML::DocumentFragment.parse(unescape_js(match.first))
+          doc.children.each do |child|
             fragments << child if child.element?
           end
         end

test/assert_select_jquery_test.rb

@@ -11,6 +11,7 @@ class AssertSelectJQueryTest < ActiveSupport::TestCase
     $('#card').html('<div><p>something else</p></div>');
     jQuery("#id").replaceWith("<div><p>something</p></div>");
     $("<div><p>something</p></div>").appendTo("#id");
+    $("<div><p>something else</p></div>").appendTo("#id");
     jQuery("<div><p>something</p></div>").prependTo("#id");
     $('#id').remove();
     jQuery("#id").hide();
@@ -48,6 +49,7 @@ def test_target_as_argument
     assert_nothing_raised do
       assert_select_jquery :appendTo, '#id' do
         assert_select 'p', 'something'
+        assert_select 'p', 'something else'
       end
       assert_select_jquery :prependTo, '#id' do
         assert_select 'p', 'something'