Merge pull request #25009 from jahfer/suppressor-keep-state

Keep previous state around for nested calls to #suppress

activerecord/lib/active_record/suppressor.rb

@@ -30,10 +30,11 @@ module Suppressor
 
     module ClassMethods
       def suppress(&block)
+        previous_state = SuppressorRegistry.suppressed[name]
         SuppressorRegistry.suppressed[name] = true
         yield
       ensure
-        SuppressorRegistry.suppressed[name] = false
+        SuppressorRegistry.suppressed[name] = previous_state
       end
     end
 

activerecord/test/cases/suppressor_test.rb

@@ -60,4 +60,16 @@ def test_suppresses_validations_on_create
       end
     end
   end
+
+  def test_suppresses_when_nested_multiple_times
+    assert_no_difference -> { Notification.count } do
+      Notification.suppress do
+        Notification.suppress { }
+        Notification.create
+        Notification.create!
+        Notification.new.save
+        Notification.new.save!
+      end
+    end
+  end
 end