fail the spec whenever we write a snapshot

levinmr · Dec 8, 2022 · 14004e4 · 14004e4
1 parent dc6c2c3
commit 14004e4
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/lib/rspec/snapshot/matchers/match_snapshot.rb b/lib/rspec/snapshot/matchers/match_snapshot.rb
@@ -50,7 +50,11 @@ def matches?(actual)
 
           @expected = read_snapshot
 
-          @actual == @expected
+          if should_write?
+            false
+          else
+            @actual == @expected
+          end
         end
 
         # === is the method called when matching an argument
@@ -98,7 +102,11 @@ def diffable?
         end
 
         def failure_message
-          "\nexpected: #{@expected}\n     got: #{@actual}\n"
+          if should_write?
+            "failing because we wrote a snapshot"
+          else
+            "\nexpected: #{@expected}\n     got: #{@actual}\n"
+          end
         end
 
         def failure_message_when_negated