Skip to content

Commit 8b111e9

Browse files
committed
fixup! Write a spec for retried job
1 parent f202c6c commit 8b111e9

File tree

2 files changed

+5
-8
lines changed

2 files changed

+5
-8
lines changed

lib/rspec/rails/matchers/active_job.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,6 @@ def matches?(proc)
232232

233233
original_enqueued_jobs = Set.new(queue_adapter.enqueued_jobs)
234234
proc.call
235-
236235
enqueued_jobs = Set.new(queue_adapter.enqueued_jobs)
237236

238237
check(enqueued_jobs - original_enqueued_jobs)

spec/rspec/rails/matchers/active_job_spec.rb

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -101,24 +101,22 @@ def self.name; "LoggingJob"; end
101101
context "when job is retried" do
102102
include ActiveJob::TestHelper
103103

104-
let(:retried_job) do
104+
let(:unreliable_job) do
105105
Class.new(ActiveJob::Base) do
106106
retry_on StandardError, wait: 5, queue: :retry
107107

108-
def self.name; "RetriedJob"; end
108+
def self.name; "UnreliableJob"; end
109109
def perform; raise StandardError; end
110110
end
111111
end
112112

113-
before do
114-
stub_const("RetriedJob", retried_job)
115-
queue_adapter.perform_enqueued_jobs = true
116-
end
113+
before { stub_const("UnreliableJob", unreliable_job) }
117114

118115
it "passes with reenqueued job" do
119116
time = Time.current.change(usec: 0)
120117
travel_to time do
121-
expect { retried_job.perform_later }.to have_enqueued_job(retried_job).on_queue(:retry).at(time + 5)
118+
UnreliableJob.perform_later
119+
expect { perform_enqueued_jobs }.to have_enqueued_job(UnreliableJob).on_queue(:retry).at(time + 5)
122120
end
123121
end
124122
end

0 commit comments

Comments
 (0)