Merge pull request #1066 from c960657/invalid-xml

Ignore parse exceptions
bblimke · Sep 18, 2024 · 4a8a513 · 4a8a513
2 parents a7bb2fc + 69c45b8
commit 4a8a513
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/webmock/request_pattern.rb b/lib/webmock/request_pattern.rb
@@ -309,6 +309,7 @@ def body_as_hash(body, content_type)
       else
         WebMock::Util::QueryMapper.query_to_values(body, notation: Config.instance.query_values_notation)
       end
+    rescue Psych::SyntaxError, REXML::ParseException
     end
 
     def body_format(content_type)

diff --git a/spec/unit/request_pattern_spec.rb b/spec/unit/request_pattern_spec.rb
@@ -562,7 +562,7 @@ def match(request_signature)
             it "should not match when body is not json" do
               expect(WebMock::RequestPattern.new(:post, 'www.example.com', body: body_hash)).
                 not_to match(WebMock::RequestSignature.new(:post, "www.example.com",
-                                                               headers: {content_type: content_type}, body: "foo bar"))
+                                                               headers: {content_type: content_type}, body: "[foo bar"))
             end
 
             it "should not match if request body is different" do
@@ -614,7 +614,7 @@ def match(request_signature)
             it "should not match when body is not xml" do
               expect(WebMock::RequestPattern.new(:post, 'www.example.com', body: body_hash)).
                 not_to match(WebMock::RequestSignature.new(:post, "www.example.com",
-                                                               headers: {content_type: content_type}, body: "foo bar"))
+                                                               headers: {content_type: content_type}, body: "<foo bar"))
                 end
 
             it "matches when the content type include a charset" do