From 939a4e6ae1ea9e3b760e191679c848d3f2b13c51 Mon Sep 17 00:00:00 2001 From: Peter Solnica Date: Tue, 17 Mar 2026 12:37:42 +0000 Subject: [PATCH] fix(specs): stop sidekiq-scheduler after each test Otherwise this may happen: ``` 1) Sentry::Sidekiq::Cron::Job sidekiq-cron adds job to sidekiq within transaction Failure/Error: expect(::Sidekiq::Queue.new.size).to eq(2) expected: 2 got: 5 (compared using ==) # ./spec/sentry/sidekiq/cron/job_spec.rb:124:in `block in
' Finished in 6.23 seconds (files took 4.01 seconds to load) 75 examples, 1 failure, 2 pending Failed examples: rspec ./spec/sentry/sidekiq/cron/job_spec.rb:118 # Sentry::Sidekiq::Cron::Job sidekiq-cron adds job to sidekiq within transaction ``` --- .../spec/sentry/sidekiq-scheduler/scheduler_spec.rb | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/sentry-sidekiq/spec/sentry/sidekiq-scheduler/scheduler_spec.rb b/sentry-sidekiq/spec/sentry/sidekiq-scheduler/scheduler_spec.rb index 661042fd8..ce7676811 100644 --- a/sentry-sidekiq/spec/sentry/sidekiq-scheduler/scheduler_spec.rb +++ b/sentry-sidekiq/spec/sentry/sidekiq-scheduler/scheduler_spec.rb @@ -19,8 +19,12 @@ scheduler_config = SidekiqScheduler::Config.new(sidekiq_config: sidekiq_config(config_options)) # Making and starting a Manager instance will load the jobs - schedule_manager = SidekiqScheduler::Manager.new(scheduler_config) - schedule_manager.start + @schedule_manager = SidekiqScheduler::Manager.new(scheduler_config) + @schedule_manager.start + end + + after do + @schedule_manager&.stop end it 'patches class' do