fix: ensure expected and actual query strings are parsed consistently

Author: bethesque

lib/pact/consumer_contract/query.rb

@@ -18,7 +18,7 @@ def self.parse_string query_string
       parsed_query = parse_query(query_string)
 
       # If Rails nested params...
-      if parsed_query.keys.any?{ | key| key.include?("[") }
+      if parsed_query.keys.any?{ | key| key =~ /\[.*\]/ }
         parse_nested_query(query_string)
       else
         parsed_query.each_with_object({}) do | (key, value), new_hash |

lib/pact/consumer_contract/query_hash.rb

@@ -32,7 +32,7 @@ def ==(other)
     # from the actual query string.
     def difference(other)
       require 'pact/matchers' # avoid recursive loop between this file, pact/reification and pact/matchers
-      Pact::Matchers.diff(query, symbolize_keys(CGI::parse(other.query)), allow_unexpected_keys: false)
+      Pact::Matchers.diff(query, symbolize_keys(convert_to_hash_of_arrays(Query.parse_string(other.query))), allow_unexpected_keys: false)
     end
 
     def query

lib/pact/shared/request.rb

@@ -3,9 +3,7 @@
 require 'pact/consumer_contract/query'
 
 module Pact
-
   module Request
-
     class Base
       include Pact::SymbolizeKeys
 
@@ -91,7 +89,6 @@ def display_path
       def display_query
         (query.nil? || query.empty?) ? '' : "?#{Pact::Reification.from_term(query)}"
       end
-
     end
   end
-end
+end

spec/lib/pact/consumer_contract/query_hash_spec.rb

@@ -73,6 +73,32 @@ module Pact
           end
         end
 
+        context "with a real example" do
+          let(:other) { QueryString.new('q%5B%5D%5Bpacticipant%5D=Foo&q%5B%5D%5Bversion%5D=1.2.3&q%5B%5D%5Bpacticipant%5D=Bar&q%5B%5D%5Bversion%5D=4.5.6&latestby=cvpv') }
+
+          let(:query) do
+            {
+              "q" => [
+                {
+                  "pacticipant" => "Foo",
+                  "version" => "1.2.3"
+                },
+                {
+                  "pacticipant" => "Bar",
+                  "version" => "4.5.6"
+                }
+              ],
+              "latestby" => [
+                "cvpv"
+              ]
+            }
+          end
+
+          it "matches" do
+            expect(subject.difference(other)).to be_empty
+          end
+        end
+
         context "when there is an ArrayLike" do
           let(:query) { { param: Pact.each_like("1") } }
           let(:other) { QueryString.new('param=1&param=2') }
@@ -149,6 +175,5 @@ module Pact
         expect(subject.to_json).to eq query_with_array.to_json
       end
     end
-
   end
 end