Skip to content

Commit a8596ca

Browse files
authored
Fix flaky reoccurring job test (#4257)
1 parent 4a1ba35 commit a8596ca

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

spec/unit/jobs/reoccurring_job_spec.rb

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,12 +76,15 @@ def perform
7676
Jobs::GenericEnqueuer.reset! # Ensure no previous state interferes
7777

7878
pollable_job = Jobs::Enqueuer.new({ queue: Jobs::Queues.generic, priority: 22 }).enqueue_pollable(FakeJob.new)
79-
expect(Delayed::Job.where(guid: PollableJobModel.first.delayed_job_guid).first[:priority]).to eq(42)
79+
first_delayed_job_guid = pollable_job.delayed_job_guid
80+
expect(Delayed::Job.where(guid: first_delayed_job_guid).first[:priority]).to eq(42)
8081

8182
execute_all_jobs(expected_successes: 1, expected_failures: 0, jobs_to_execute: 1)
8283

83-
expect(Delayed::Job.where(guid: PollableJobModel.first.delayed_job_guid).first[:priority]).to eq(42)
84-
expect(PollableJobModel.first.delayed_job_guid).not_to eq(pollable_job.delayed_job_guid)
84+
pollable_job.reload
85+
second_delayed_job_guid = pollable_job.delayed_job_guid
86+
expect(Delayed::Job.where(guid: second_delayed_job_guid).first[:priority]).to eq(42)
87+
expect(first_delayed_job_guid).not_to eq(second_delayed_job_guid)
8588
end
8689

8790
it 'waits for the polling interval' do

0 commit comments

Comments
 (0)