pact-support 1.10.3 → 1.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 22c9bd406afc40bd0c83b91c36eb49f142512b15
-  data.tar.gz: f5f990ba3e1c1b5337283a167346a41411b774c6
+  metadata.gz: 81a9d5549b8fae48099843919ad94693490e9eb9
+  data.tar.gz: 839c24b776dc088241caac15006827c313009bf0
 SHA512:
-  metadata.gz: 7b6cff7af477db0aaba912d79bcfe0293926cffcd77a39b61529decb7e391054a9a2adc1981afe6847b6690d57b464d3e1e8a04c3c82398b53f92d14eb5c659e
-  data.tar.gz: dba22a097b35198446c77bd03f6f305f751406a471adfb5446574662736c344dd599641505fa9b9a022831563026c311687cef2cfdd70ce816093f0d27105c97
+  metadata.gz: 63fb22506c12a51dffbf729ed6b4c652599c581a368b47ad7167bfb2a44761fc26b3ecdcbe4b9f8a7ed7f846b9bd592af510491abce1ece6b167f5066d4d3665
+  data.tar.gz: '055840b001e39360ce459fc1243f56b26d0f19d01023cae27bdecc8a2349a12e0457d96e8c42f6134e50b38a854258dad988c6ebc997c90a8cdb05c80dc3d0fb'

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+<a name="v1.11.0"></a>
+### v1.11.0 (2019-06-18)
+
+
+#### Features
+
+* allow Integers and Floats to be considered equivalent when using type based matching.	 ([d8a70a1](/../../commit/d8a70a1))
+
+
 <a name="v1.10.3"></a>
 ### v1.10.3 (2019-06-07)
 

data/lib/pact/configuration.rb

@@ -51,6 +51,7 @@ module Pact
     attr_accessor :error_stream
     attr_accessor :output_stream
     attr_accessor :pactfile_write_order
+    attr_accessor :treat_all_number_classes_as_equivalent # when using type based matching
 
     def self.default_configuration
       c = Configuration.new
@@ -61,6 +62,7 @@ module Pact
       c.output_stream = $stdout
       c.error_stream = $stderr
       c.pactfile_write_order = :chronological
+      c.treat_all_number_classes_as_equivalent = true
 
       c
     end

data/lib/pact/matchers/matchers.rb

@@ -20,23 +20,30 @@ module Pact
   # maintain backwards compatibility
 
   module Matchers
-
     NO_DIFF_AT_INDEX = NoDiffAtIndex.new
-    DEFAULT_OPTIONS = {allow_unexpected_keys: true, type: false}.freeze
     NO_DIFF = {}.freeze
+    NUMERIC_TYPES = %w[Integer Float Fixnum Bignum BigDecimal].freeze
+    DEFAULT_OPTIONS = {
+      allow_unexpected_keys: true,
+      type: false
+    }.freeze
 
     extend self
 
     def diff expected, actual, opts = {}
-      calculate_diff(expected, actual, DEFAULT_OPTIONS.merge(opts))
+      calculate_diff(expected, actual, DEFAULT_OPTIONS.merge(configurable_options).merge(opts))
     end
 
     def type_diff expected, actual, opts = {}
-      calculate_diff expected, actual, DEFAULT_OPTIONS.merge(opts).merge(type: true)
+      calculate_diff expected, actual, DEFAULT_OPTIONS.merge(configurable_options).merge(opts).merge(type: true)
     end
 
     private
 
+    def configurable_options
+      { treat_all_number_classes_as_equivalent: Pact.configuration.treat_all_number_classes_as_equivalent }
+    end
+
     def calculate_diff expected, actual, opts = {}
       options = DEFAULT_OPTIONS.merge(opts)
       case expected
@@ -156,22 +163,22 @@ module Pact
 
     def object_diff expected, actual, options
       if options[:type]
-        type_difference expected, actual
+        type_difference expected, actual, options
       else
         exact_value_diff expected, actual, options
       end
     end
 
     def exact_value_diff expected, actual, options
-      if expected != actual
-        Difference.new expected, actual, value_difference_message(expected, actual, options)
-      else
+      if expected == actual
         NO_DIFF
+      else
+        Difference.new expected, actual, value_difference_message(expected, actual, options)
       end
     end
 
-    def type_difference expected, actual
-      if types_match? expected, actual
+    def type_difference expected, actual, options
+      if types_match? expected, actual, options
         NO_DIFF
       else
         TypeDifference.new type_diff_expected_display(expected), type_diff_actual_display(actual), type_difference_message(expected, actual)
@@ -186,8 +193,16 @@ module Pact
       actual.is_a?(KeyNotFound) ?  actual : ActualType.new(actual)
     end
 
-    def types_match? expected, actual
-      expected.class == actual.class || (is_boolean(expected) && is_boolean(actual))
+    # Make options optional to support existing monkey patches
+    def types_match? expected, actual, options = {}
+      expected.class == actual.class ||
+        (is_boolean(expected) && is_boolean(actual)) ||
+        (options.fetch(:treat_all_number_classes_as_equivalent, false) && is_number?(expected) && is_number?(actual))
+    end
+
+    def is_number? object
+      # deal with Fixnum and Integer without warnings by using string class names
+      NUMERIC_TYPES.include?(object.class.to_s)
     end
 
     def is_boolean object

data/lib/pact/support/version.rb

@@ -1,5 +1,5 @@
 module Pact
   module Support
-    VERSION = "1.10.3"
+    VERSION = "1.11.0"
   end
 end

data/spec/lib/pact/matchers/matchers_messages_mismatched_value_spec.rb

@@ -32,13 +32,14 @@ module Pact::Matchers
         [INT, 2, "Expected 1 but got 2 at <path>"],
         [INT, nil, "Expected 1 but got nil at <path>"],
         [INT, STRING, "Expected 1 but got \"foo\" at <path>"],
-        [INT, FLOAT, "Expected 1 but got \"foo\" at <path>", {pending: true}],
+        [INT, FLOAT, nil],
         [INT, HASH, "Expected 1 but got a Hash at <path>"],
         [INT, ARRAY, "Expected 1 but got an Array at <path>"],
         [Pact.like(INT), 2, nil],
         [Pact.like(INT), nil, "Expected #{a_numeric} (like 1) but got nil at <path>"],
         [Pact.like(INT), STRING, "Expected #{a_numeric} (like 1) but got a String (\"foo\") at <path>"],
-        [Pact.like(INT), FLOAT, "Expected #{a_numeric} (like 1) but got a Float (1.0) at <path>"],
+        [Pact.like(INT), FLOAT, "Expected #{a_numeric} (like 1) but got a Float (1.0) at <path>", { treat_all_number_classes_as_equivalent: false }],
+        [Pact.like(INT), FLOAT, nil, { treat_all_number_classes_as_equivalent: true }],
         [Pact.like(INT), HASH, "Expected #{a_numeric} (like 1) but got a Hash at <path>"],
         [Pact.like(INT), ARRAY, "Expected #{a_numeric} (like 1) but got an Array at <path>"],
         [HASH, HASH, nil],
@@ -56,9 +57,9 @@ module Pact::Matchers
         [ARRAY, HASH, "Expected an Array but got a Hash at <path>"]
       ]
 
-      COMBINATIONS.each do | expected, actual, expected_message, options |
-        context "when expected is #{expected.inspect} and actual is #{actual.inspect}", options || {} do
-          let(:difference) { diff({thing: expected}, {thing: actual}) }
+      COMBINATIONS.each do | expected, actual, expected_message, diff_options |
+        context "when expected is #{expected.inspect} and actual is #{actual.inspect}" do
+          let(:difference) { diff({thing: expected}, {thing: actual}, diff_options || {}) }
           let(:message) { difference[:thing] ? difference[:thing].message : nil }
 
           it "returns the message '#{expected_message}'" do

data/spec/lib/pact/matchers/matchers_spec.rb

@@ -30,19 +30,60 @@ module Pact::Matchers
       end
 
       context "when the headers do not match" do
-
         let(:actual) { Pact::Headers.new('Content-Length' => '1')}
         let(:difference) { {"Content-Type" => Difference.new('application/hippo', Pact::KeyNotFound.new)} }
         it "returns a diff" do
           expect(diff(expected, actual)).to eq difference
         end
+      end
+    end
+
+
+    context 'when treat_all_number_classes_as_equivalent is true' do
+      let(:options) { { treat_all_number_classes_as_equivalent: true } }
+
+      describe 'matching numbers with something like' do
+        let(:expected) { Pact::SomethingLike.new( { a: 1.1 } ) }
+        let(:actual)   { { a: 2 } }
 
+        it 'returns an empty diff' do
+          expect(diff(expected, actual, options)).to eq({})
+        end
       end
 
+      describe 'with exact matching' do
+        let(:expected) { { a: 1 } }
+        let(:actual) { { a: 1.0 } }
+
+        it 'returns an empty diff' do
+          expect(diff(expected, actual, options)).to eq({})
+        end
+      end
     end
 
-    describe 'matching with something like' do
+    context 'when treat_all_number_classes_as_equivalent is false' do
+      let(:options) { { treat_all_number_classes_as_equivalent: false } }
 
+      describe 'matching numbers with something like' do
+        let(:expected) { Pact::SomethingLike.new( { a: 1.1 } ) }
+        let(:actual)   { { a: 2 } }
+
+        it 'returns a diff' do
+          expect(diff(expected, actual, options)).to_not eq({})
+        end
+      end
+
+      describe 'with exact matching' do
+        let(:expected) { { a: 1 } }
+        let(:actual) { { a: 1.0 } }
+
+        it 'returns an empty diff because in Ruby 1.0 == 1' do
+          expect(diff(expected, actual)).to eq({})
+        end
+      end
+    end
+
+    describe 'matching with something like' do
       context 'when the actual is something like the expected' do
         let(:expected) { Pact::SomethingLike.new( { a: 1 } ) }
         let(:actual)   { { a: 2 } }
@@ -50,7 +91,6 @@ module Pact::Matchers
         it 'returns an empty diff' do
           expect(diff(expected, actual)).to eq({})
         end
-
       end
 
       context 'when the there is a mismatch of a parent, and a child contains a SomethingLike' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pact-support
 version: !ruby/object:Gem::Version
-  version: 1.10.3
+  version: 1.11.0
 platform: ruby
 authors:
 - James Fraser
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-06-06 00:00:00.000000000 Z
+date: 2019-06-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: randexp