rspec-protobuf 0.2.4 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d635dd29059d952dc1e0db41c3cef03aa4d9e18f098db7a61f34736821317831
-  data.tar.gz: db9151a95b2f9d72898eaa5fedaa2b9ecbf0dd1d4bf5cb16506442f7cfcbba88
+  metadata.gz: 067f4cadf677ad482da31b4d3dbcb66cd53e79f434807d67e0e9522144109b32
+  data.tar.gz: fdc42f1600c8cbc1f97388a413ae81bded5cc8451e84128907854a453953d709
 SHA512:
-  metadata.gz: 55abd020cfdbf915a4b6db65e674367333957e59ff86a0dab804d4807dad3e0d4e250c20b5a2043fc76ac88931293206719720e1844563125a46e56224705a33
-  data.tar.gz: a485ad9cfd09055229c816f83baa2105659c3e50ebc5b3a053de6ec758c3bdd854969c606efc681a2f1cfda80821129efb7448cbe64d2bfd828e62ed54cf890b
+  metadata.gz: 548a3253d3e1c1f36e1bc00472822c61eaf6f96a02865ac71533aec137958a34bbd2d24dc5f4a170eb4949b4fcbf31f1966b62d546caf3bd3d3086b2a520a163
+  data.tar.gz: 514ea10c3836b87c2f82bc7b6e9cefa8c94514238791b76c462cad5fdbed82fe21af8ba1ade18dc7a616846e8c863aad4d118621e50dd2f1d792179930216ccf

data/CHANGELOG.md

@@ -1,3 +1,14 @@
+###  v0.3.1  (2023-03-27)
+- bugfix
+
+###  v0.3.0  (2023-03-25)
+- match repeated field
+- match default values
+- test coverage
+- type check
+- simplify
+- symbol matching
+
 ###  v0.2.4  (2023-03-15)
 - repeated field bug
 

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rspec-protobuf (0.2.4)
+    rspec-protobuf (0.3.1)
       google-protobuf (>= 3)
       rspec-expectations (>= 3)
 

data/lib/rspec/protobuf/refinements.rb

@@ -5,63 +5,29 @@ module RSpec
         def include?(*args)
           expected_attrs = Hash === args.last ? args.pop : {}
 
-          # ensure all enumerated keys are present
+          # ensure all enumerated keys are indeed attributes
           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) }
 
-          fields = {}
-          self.class.descriptor.each { |f| fields[f.name.to_sym] = f }
+          fields = Hash[
+            self.class.descriptor.map { |f| [ f.name.to_sym, f ] }
+          ]
 
-          # check expected attribute matches
+          # ensure that expected attribute matches
           expected_attrs.all? do |expected_attr, expected_value|
-            field = fields[expected_attr]
-            actual_value = field.get(self)
-
-            # convert enum to int value to match input type
-            if field.type == :enum && expected_value.is_a?(Integer)
-              actual_value = field.subtype.lookup_name(actual_value)
-            end
-
-            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
-              end
-            end
-
-            matches
+            field_matches?(fields[expected_attr], expected_value)
           end
         end
 
-        def match?(**attrs)
-          # ensure all enumerated keys are present
+        def matches?(**attrs)
+          # ensure all enumerated attributes are present
           return false unless attrs.keys.all? { |attr| respond_to?(attr) }
 
-          # ensure each field matches
+          # ensure that 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
-              unless attrs.key?(field.name.to_sym)
-                # fall back to default value
-                expected_value = field.label == :repeated ? [] : field.default
-              end
-
-              # convert enum to int value to match input type
-              if field.type == :enum && expected_value.is_a?(Integer)
-                actual_value = field.subtype.lookup_name(actual_value)
-              end
-
-              expected_value === actual_value
-            end
+            field_matches?(field, attrs[field.name.to_sym])
           end
         end
 
@@ -82,6 +48,50 @@ module RSpec
 
           res
         end
+
+        private
+
+        def field_matches?(field, expected)
+          actual = field.get(self)
+
+          if expected.nil?
+            # mimic protobuf behavior and convert to default value
+            expected = field.label == :repeated ? [] : field.default
+          elsif expected.is_a?(Symbol) && field.type == :string
+            # convert symbols to strings
+            expected = expected.to_s
+          end
+
+          # convert enum to int value to match input type
+          if field.type == :enum && expected.is_a?(Integer)
+            actual = field.subtype.lookup_name(actual)
+          end
+
+          if field.label == :repeated && expected.is_a?(Array)
+            return false unless actual.length == expected.length
+
+            actual.zip(expected).all? do |a_actual, a_expected|
+              values_match?(a_actual, a_expected)
+            end
+          else
+            values_match?(actual, expected)
+          end
+        end
+
+        def values_match?(actual, expected)
+          expected === actual || if actual.is_a?(Google::Protobuf::MessageExts)
+            case expected
+            when Google::Protobuf::MessageExts
+              # expected === actual
+            when Hash
+              # recurse
+              actual.matches?(**expected)
+            else
+              # eg. RSpec matchers
+              expected === actual.to_h
+            end
+          end
+        end
       end
     end
   end

data/lib/rspec/protobuf/version.rb

@@ -1,5 +1,5 @@
 module RSpec
   module Protobuf
-    VERSION = "0.2.4"
+    VERSION = "0.3.1"
   end
 end

data/lib/rspec/protobuf.rb

@@ -8,7 +8,7 @@ using RSpec::Protobuf::Refinements
 RSpec::Matchers.define :be_a_protobuf do |type = nil, **attrs|
   match do |actual|
     # ensure type is a valid Protobuf message type
-    if type && !(type < Google::Protobuf::MessageExts)
+    if type && !(type.is_a?(Class) && type < Google::Protobuf::MessageExts)
       raise TypeError, "Expected arg to be a Google::Protobuf::MessageExts, found: #{type}"
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspec-protobuf
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.3.1
 platform: ruby
 authors:
 - Daniel Pepper
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-03-15 00:00:00.000000000 Z
+date: 2023-03-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: google-protobuf