sidekiq-amigo 1.13.0 → 1.13.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 70bb0febcaf8c1a36e19fc4dd56fb766be4fa046c941b1f2aa66b7dcf5372872
-  data.tar.gz: 620b2c3a6bf664931eb8a6e84cc7d8762597f4aae91aa17765bc1d1fb0b432ad
+  metadata.gz: c75430ea4c493cfee46c6de0daf1e64b8a6bea0e17017524dc2bc540bf7f30b2
+  data.tar.gz: 3bd32447b4f3d52f3512570bb3af6d5cab2643619bc83c9e95c17c0fa66418a3
 SHA512:
-  metadata.gz: 20bcdab3b140069cd0eeb6a559032e8e4d986ffdc6bdd7ed03030a45b5b83a50600bb7d465736a0a7dcdd2e369a2a054bb2f968a77111ec5cef807876d19d8f0
-  data.tar.gz: ddbb6ac910db1687c6e5f471f965d2b5f2dc78b650564f28c24153e8ce1d1d7b350f03da4b0571673084865ac8c9c5f9a27453e090ad94f6b12f9507649dde6d
+  metadata.gz: 6fa8f511ead61d4a89473fdf3eef6ed58299219e98d8172098beddbda6b73ba19822c777536f924986940bac0787382b3752f1fd921405793f14dc8307ce9897
+  data.tar.gz: ca0f43716e1067f606956d14b3db8457115e57d1732c181d0006491c979b440058c8d256e3d0da5f242c99409d14392169d3d78717c6544dc8423cf3bfe94cf2

data/lib/amigo/spec_helpers.rb

@@ -72,7 +72,7 @@ module Amigo
         true
       end
 
-      def matches?(given_proc)
+      def matches?(given_proc, negative_expectation=false)
         unless given_proc.respond_to?(:call)
           warn "publish matcher used with non-proc object #{given_proc.inspect}"
           return false
@@ -88,24 +88,41 @@ module Amigo
           given_proc.call
         end
 
-        self.match_expected_events
+        self.match_expected_events(negative_expectation)
 
         return @error.nil? && @missing.empty?
       end
 
+      # rubocop:disable Naming/PredicatePrefix
+      def does_not_match?(given_proc)
+        !matches?(given_proc, true)
+      end
+      # rubocop:enable Naming/PredicatePrefix
+
       def on_publish_error(err)
         @error = err
       end
 
-      def match_expected_events
+      def match_expected_events(negative_expectation)
         @expected_events.each do |expected_event, expected_payload|
+          if expected_event.nil? && !negative_expectation
+            RSpec::Expectations.configuration.false_positives_handler.call(
+              "Using the `publish` matcher without providing a specific " \
+              "event name risks false positives, since `publish` " \
+              "will match any event. Instead, provide the name " \
+              "of the event you are matching against."\
+              "This message can be suppressed by setting: " \
+              "`RSpec::Expectations.configuration.on_potential_false" \
+              "_positives = :nothing`",
+            )
+          end
           match = @recorded_events.find do |recorded|
             self.event_names_match?(expected_event, recorded.name) &&
               self.payloads_match?(expected_payload, recorded.payload)
           end
 
           if match
-            self.add_matched(expected_event, expected_payload)
+            self.add_matched(expected_event, expected_payload, match)
           else
             self.add_missing(expected_event, expected_payload)
           end
@@ -113,6 +130,7 @@ module Amigo
       end
 
       def event_names_match?(expected, actual)
+        return true if expected.nil?
         return expected.matches?(actual) if expected.respond_to?(:matches?)
         return expected.match?(actual) if expected.respond_to?(:match?)
         return expected == actual
@@ -123,8 +141,8 @@ module Amigo
         return expected.nil? || expected.empty? || expected == actual
       end
 
-      def add_matched(event, payload)
-        @matched << [event, payload]
+      def add_matched(topic, payload, event)
+        @matched << [topic, payload, event]
       end
 
       def add_missing(event, payload)
@@ -156,10 +174,12 @@ module Amigo
 
       def failure_message_when_negated
         messages = []
-        @matched.each do |event, _payload|
-          message = "expected a '%s' event not to be fired" % [event]
-          message << (" with a payload of %p" % [@expected_payload]) if @expected_payload
-          message << " but one was."
+        @matched.each do |topic, payload, event|
+          message = "expected a '#{topic || event.name}' event not to be fired"
+          message << " with a payload of #{payload.inspect}" if payload
+          message << " but one was"
+          event.payload.any? && message << ": #{event.payload.inspect}"
+          message << "."
           messages << message
         end
 

data/lib/amigo/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Amigo
-  VERSION = "1.13.0"
+  VERSION = "1.13.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sidekiq-amigo
 version: !ruby/object:Gem::Version
-  version: 1.13.0
+  version: 1.13.1
 platform: ruby
 authors:
 - Lithic Technology