postburner 1.0.0.pre.11 → 1.0.0.pre.12

data/lib/postburner/strategies/immediate_test_queue.rb

@@ -105,6 +105,10 @@ module Postburner
       # is invoked. If the job has a future run_at, travels to that time before
       # execution. Otherwise executes immediately.
       #
+      # Uses recursion detection to prevent infinite loops when jobs create and
+      # enqueue other jobs (e.g., schedule callbacks). Jobs enqueued while already
+      # inside a perform! call are queued but not executed immediately.
+      #
       # @param job [Postburner::Job] The job to execute
       # @param options [Hash] Unused in test mode
       #
@@ -115,17 +119,30 @@ module Postburner
       # @api private
       #
       def insert(job, options = {})
-        # If job has a future run_at, travel to that time for execution
-        if job.run_at && job.run_at > Time.zone.now
-          travel_to(job.run_at) do
+        # Detect recursion: if we're already performing a job, don't execute
+        # the newly enqueued job immediately. This prevents infinite loops
+        # when job callbacks (like schedule_next_execution) enqueue new jobs.
+        if Thread.current[:postburner_performing]
+          return { status: 'INLINE_DEFERRED', id: nil }
+        end
+
+        begin
+          Thread.current[:postburner_performing] = true
+
+          # If job has a future run_at, travel to that time for execution
+          if job.run_at && job.run_at > Time.current
+            travel_to(job.run_at) do
+              job.perform!(job.args)
+            end
+          else
+            # No future run_at, execute normally
             job.perform!(job.args)
           end
-        else
-          # No future run_at, execute normally
-          job.perform!(job.args)
+        ensure
+          Thread.current[:postburner_performing] = false
         end
 
-        # Return format matching Backburner response (symbol keys)
+        # Return format matching Beanstalkd response (symbol keys)
         { status: 'INLINE', id: nil }
       end
     end

data/lib/postburner/strategies/nice_queue.rb

@@ -77,7 +77,7 @@ module Postburner
       # @note Logs "PREMATURE; RE-INSERTED" message to job's audit trail
       #
       def handle_premature_perform(job)
-        response = job.insert! delay: job.run_at - Time.zone.now
+        response = job.insert! delay: job.run_at - Time.current
         job.log! "PREMATURE; RE-INSERTED: #{response}"
       end
     end

data/lib/postburner/strategies/null_queue.rb

@@ -88,7 +88,7 @@ module Postburner
       # @api private
       #
       def insert(job, options = {})
-        # Return format matching Backburner response (symbol keys)
+        # Return format matching Beanstalkd response (symbol keys)
         { status: 'NULL', id: nil }
       end
 
@@ -118,7 +118,7 @@ module Postburner
       # @api private
       #
       def handle_perform!(job)
-        if job.run_at && job.run_at > Time.zone.now
+        if job.run_at && job.run_at > Time.current
           travel_to(job.run_at) do
             job.perform!(job.args)
           end

data/lib/postburner/strategies/test_queue.rb

@@ -105,7 +105,7 @@ module Postburner
         # Will raise PrematurePerform if run_at is in the future
         job.perform!(job.args)
 
-        # Return format matching Backburner response (symbol keys)
+        # Return format matching Beanstalkd response (symbol keys)
         { status: 'INLINE', id: nil }
       end
 
@@ -121,7 +121,7 @@ module Postburner
       # @raise [Postburner::Job::PrematurePerform] Always raises with helpful message
       #
       def handle_premature_perform(job)
-        raise Postburner::Job::PrematurePerform, "Job scheduled for #{job.run_at} (#{((job.run_at - Time.zone.now) / 60).round(1)} minutes from now). Use `travel_to(job.run_at)` in your test, or set `Postburner.inline_immediate_test_strategy!` to execute scheduled jobs immediately."
+        raise Postburner::Job::PrematurePerform, "Job scheduled for #{job.run_at} (#{((job.run_at - Time.current) / 60).round(1)} minutes from now). Use `travel_to(job.run_at)` in your test, or set `Postburner.inline_immediate_test_strategy!` to execute scheduled jobs immediately."
       end
     end
   end

data/lib/postburner/time_helpers.rb

@@ -18,7 +18,8 @@ module Postburner
   #
   #   travel_to(2.days.from_now) do
   #     # Code here executes as if it's 2 days in the future
-  #     Time.zone.now  # => 2 days from now
+  #     Time.current   # => 2 days from now (preferred in Rails)
+  #     Time.zone.now  # => 2 days from now (equivalent)
   #   end
   #
   # @example In a class method
@@ -54,7 +55,8 @@ module Postburner
     #
     # @example Travel to specific time
     #   travel_to(Time.zone.parse('2025-12-25 00:00:00')) do
-    #     puts Time.zone.now  # => 2025-12-25 00:00:00
+    #     puts Time.current   # => 2025-12-25 00:00:00 (preferred)
+    #     puts Time.zone.now  # => 2025-12-25 00:00:00 (equivalent)
     #   end
     #
     # @example Travel relative to now

data/lib/postburner/tube.rb

@@ -31,7 +31,11 @@ module Postburner
     # Just pass the last known id to after for the next batch.
     #
     def jobs(count=20, limit: 1000, after: nil)
+      # Access raw hash to avoid beaneater FastStruct method definition issues in Ruby 3.4
       stats = @tube.stats
+      stats_hash = stats.instance_variable_get(:@hash) || {}
+      tube_name = stats_hash['name']
+
       jobs = Array.new
 
       min_known = (
@@ -42,7 +46,11 @@ module Postburner
 
       for i in min..max
         job = @tube.client.jobs.find(i)
-        jobs << job if job && stats[:name] == job.stats[:tube]
+        if job
+          job_stats = job.stats
+          job_stats_hash = job_stats.instance_variable_get(:@hash) || {}
+          jobs << job if job_stats_hash['tube'] == tube_name
+        end
         break if jobs.length >= count
       end
 

data/lib/postburner/version.rb

@@ -1,3 +1,3 @@
 module Postburner
-  VERSION = '1.0.0.pre.11'
+  VERSION = '1.0.0.pre.12'
 end