webmock-twirp 0.1.2 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0264d26c4984559b4163ff5740fda0df43882736fa174881a63939c195e85331
-  data.tar.gz: 9d0da633242e3f722d18c0c97042c5312b16ba4fb28fb6509479a04f5c4201eb
+  metadata.gz: 5a732c5a4926da1aa55cfb9348ea84c0cd7506fabc1e598a746dabd4fa57a2c9
+  data.tar.gz: e4ec2abe817732781453442877ffbae84771919cebed81238360c3a002ef7820
 SHA512:
-  metadata.gz: 369e27e257599a09541feb244e9a4c1ff59a7b069c3df154c361c6567ce8f15f0e3b46cbfb0252ebe7c72beed20cb31bb8c947857d84372aec7ae3d697e21602
-  data.tar.gz: 7aa1544321a07d5f8350203f16f331d2e9ee90b5b38928d29d1b4fb2750101109298796ff7b571684108af8aaf3c80d4b2d595a27cdef0b33715c888cb99f1cd
+  metadata.gz: 28e13a626132b193b2d3aaa26c5d192949d296171d63d2b5bc2539fd88eab2a72c633d29bc0a4711d804ba7be144de2dc57a1be8b1d71a5d0b614fa3c927885b
+  data.tar.gz: 7b42012c4c5ea5c49868091dddf44cdafcd4fcb7b76dadae2bd91135c8bb5b65307aca1797eef8153d9839fe72ddf08c0178156227f0c74b0fe561a03642d829

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+###  v0.2.0  (2022-10-17)
+- improved matching
+
 ###  v0.1.2  (2022-10-17)
 - improved matching
 - test coverage

data/lib/webmock/twirp/refinements.rb

@@ -9,10 +9,66 @@ module WebMock
 
       refine Google::Protobuf::MessageExts do
         def normalized_hash(symbolize_keys: true)
-          JSON.parse(
-            to_json(preserve_proto_fieldnames: true),
-            symbolize_names: symbolize_keys,
-          )
+          res = {}
+
+          self.class.descriptor.each do |field|
+            key = symbolize_keys ? field.name.to_sym : field.name
+            value = field.get(self)
+
+            if value.is_a?(Google::Protobuf::MessageExts)
+              # recursively serialize sub-message
+              value = value.normalized_hash(symbolize_keys: symbolize_keys)
+            end
+
+            res[key] = value unless field.default == value
+          end
+
+          res
+        end
+
+        def include?(*args)
+          expected_attrs = Hash === args.last ? args.pop : {}
+
+          # ensure all enumerated keys are present
+          return false unless args.all? { |attr| respond_to?(attr) }
+
+          # ensure all enumerated attributes are present
+          return false unless expected_attrs.keys.all? { |attr| respond_to?(attr) }
+
+          # check expected attribute matches
+          expected_attrs.all? do |expected_attr, expected_value|
+            actual_value = send(expected_attr)
+
+            matches = expected_value === actual_value
+
+            if actual_value.is_a?(Google::Protobuf::MessageExts)
+              if expected_value.is_a?(Hash)
+                matches ||= actual_value.match?(**expected_value)
+              else
+                matches ||= expected_value === actual_value.to_h
+                # matches ||= expected_value === actual_value.normalized_hash
+              end
+            end
+
+            matches
+          end
+        end
+
+        def match?(**attrs)
+          # ensure all enumerated keys are present
+          return false unless attrs.keys.all? { |attr| respond_to?(attr) }
+
+          # ensure each field matches
+          self.class.descriptor.all? do |field|
+            actual_value = field.get(self)
+            expected_value = attrs[field.name.to_sym]
+
+            if actual_value.is_a?(Google::Protobuf::MessageExts) && expected_value.is_a?(Hash)
+              actual_value.match?(**expected_value)
+            else
+              attrs.fetch(field.name.to_sym, field.default) === actual_value
+            end
+          end
         end
       end
     end

data/lib/webmock/twirp/request_stub.rb

@@ -71,27 +71,11 @@ module WebMock
 
         decoder = ->(request) do
           input_class = rpc_from_request(request)[:input_class]
-
-          matched = true
           decoded_request = input_class.decode(request.body)
 
-          if attrs.any?
-            if defined?(RSpec::Matchers::BuiltIn::Include)
-              matcher = RSpec::Matchers::BuiltIn::Include.new(attrs)
-              attr_hash = decoded_request.to_h
-              normalize = ->{ decoded_request.normalized_hash }
-            else
-              matcher = WebMock::Matchers::HashIncludingMatcher.new(attrs)
-              attr_hash = WebMock::Util::HashKeysStringifier.stringify_keys!(decoded_request.to_h, deep: true)
-              normalize = ->{ decoded_request.normalized_hash(symbolize_keys: false) }
-            end
-
-            matched &= (matcher === attr_hash || matcher === normalize.call)
-          end
-
-          if block
-            matched &= block.call(decoded_request)
-          end
+          matched = true
+          matched &= decoded_request.include?(attrs) if attrs.any?
+          matched &= block.call(decoded_request) if block
 
           matched
         end if attrs.any? || block_given?

data/lib/webmock/twirp/version.rb

@@ -1,5 +1,5 @@
 module WebMock
   module Twirp
-    VERSION = "0.1.2"
+    VERSION = "0.2.0"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: webmock-twirp
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.2.0
 platform: ruby
 authors:
 - Daniel Pepper
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-10-17 00:00:00.000000000 Z
+date: 2022-10-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: webmock