Make CheckWorker CheckJob

Sidekiq workers are no longer supported
alphagov · Sep 26, 2024 · 9920f80 · 9920f80
1 parent d9fc8e3
commit 9920f80
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 7 deletions.
diff --git a/app/controllers/batch_controller.rb b/app/controllers/batch_controller.rb
@@ -48,7 +48,7 @@ def create
       render(json: batch_report(batch), status: :created)
     else
       batch.checks.each do |check|
-        CheckWorker.run(check.id, priority: create_params.priority)
+        CheckJob.run(check.id, priority: create_params.priority)
       end
 
       render(json: batch_report(batch.reload), status: :accepted)

diff --git a/app/controllers/check_controller.rb b/app/controllers/check_controller.rb
@@ -36,7 +36,7 @@ def check
 
     check = Check.create!(link:)
 
-    CheckWorker.run(
+    CheckJob.run(
       check.id,
       priority: check_params.priority,
       synchronous: check_params.synchronous,

diff --git a/app/workers/check_worker.rb → app/workers/check_job.rb b/app/workers/check_worker.rb → app/workers/check_job.rb
@@ -1,5 +1,5 @@
-class CheckWorker
-  include Sidekiq::Worker
+class CheckJob
+  include Sidekiq::Job
   include PerformAsyncInQueue
 
   sidekiq_options retry: 3, lock: :until_and_while_executing, lock_args_method: :unique_args

diff --git a/app/workers/cleanup_worker.rb b/app/workers/cleanup_worker.rb
@@ -5,7 +5,7 @@ class CleanupWorker
 
   def perform
     checks_to_perform.each do |check_id|
-      CheckWorker.perform_async(check_id)
+      CheckJob.perform_async(check_id)
     end
 
     old_batches.delete_all

diff --git a/spec/requests/batch_spec.rb b/spec/requests/batch_spec.rb
@@ -40,7 +40,7 @@
       include_examples "returns batch report"
 
       it "creates a job" do
-        expect(CheckWorker.jobs.size).to eq(2)
+        expect(CheckJob.jobs.size).to eq(2)
       end
     end
 

diff --git a/spec/workers/check_worker_spec.rb → spec/workers/check_job_spec.rb b/spec/workers/check_worker_spec.rb → spec/workers/check_job_spec.rb
@@ -1,6 +1,6 @@
 require "rails_helper"
 
-RSpec.describe CheckWorker do
+RSpec.describe CheckJob do
   specify { expect(described_class).to have_valid_sidekiq_options }
 
   describe "perform" do