do not convert RSpec::Mocks::ArgumentMatchers::HashIncludingMatcher to_s

bblimke · May 1, 2018 · 5122822 · 5122822
1 parent 15fc3f1
commit 5122822
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/lib/webmock/util/values_stringifier.rb b/lib/webmock/util/values_stringifier.rb
@@ -1,8 +1,8 @@
 class WebMock::Util::ValuesStringifier
   def self.stringify_values(value)
     case value
-    when nil
-      value
+    when String, Numeric, TrueClass, FalseClass
+      value.to_s
     when Hash
       Hash[
         value.map do |k, v|
@@ -14,7 +14,7 @@ def self.stringify_values(value)
         stringify_values(v)
       end
     else
-      value.to_s
+      value
     end
   end
 end
diff --git a/spec/acceptance/shared/stubbing_requests.rb b/spec/acceptance/shared/stubbing_requests.rb
@@ -172,6 +172,14 @@
                 :post, "http://www.example.com/", headers: {'Content-Type' => 'application/x-www-form-urlencoded'},
               body: "foo=1").status).to eq("200")
             end
+
+            it "should match if stubbed request body is hash_included" do
+              WebMock.reset!
+              stub_request(:post, "www.example.com").with(body: {"foo" => hash_including("bar" => '1')})
+              expect(http_request(
+                       :post, "http://www.example.com/", headers: {'Content-Type' => 'application/x-www-form-urlencoded'},
+                       body: "foo[bar]=1").status).to eq("200")
+            end
           end
         end