Don't memoize interruption adapter

Some tests rely on changing the interruption adapter mid-execution to test specific behaviour. We shouldn't break that with this feature.
Shopify · Feb 8, 2024 · e28ecc3 · e28ecc3
1 parent 6ba201d
commit e28ecc3
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/lib/job-iteration/iteration.rb b/lib/job-iteration/iteration.rb
@@ -75,10 +75,6 @@ def around_iterate(&blk)
         set_callback(:iterate, :around, &blk)
       end
 
-      def interruption_adapter
-        JobIteration.interruption_adapter || JobIteration::InterruptionAdapters.lookup(queue_adapter_name)
-      end
-
       private
 
       def ban_perform_definition
@@ -125,8 +121,9 @@ def retry_job(*, **)
 
     private
 
-    # @api private
-    attr_reader :interruption_adapter
+    def interruption_adapter # @private
+      JobIteration.interruption_adapter || JobIteration::InterruptionAdapters.lookup(queue_adapter_name)
+    end
 
     def enumerator_builder
       JobIteration.enumerator_builder.new(self)