diff options
Diffstat (limited to 'spec/workers/schedule_merge_request_cleanup_refs_worker_spec.rb')
-rw-r--r-- | spec/workers/schedule_merge_request_cleanup_refs_worker_spec.rb | 25 |
1 files changed, 5 insertions, 20 deletions
diff --git a/spec/workers/schedule_merge_request_cleanup_refs_worker_spec.rb b/spec/workers/schedule_merge_request_cleanup_refs_worker_spec.rb index 869818b257e..ef515e43474 100644 --- a/spec/workers/schedule_merge_request_cleanup_refs_worker_spec.rb +++ b/spec/workers/schedule_merge_request_cleanup_refs_worker_spec.rb @@ -6,16 +6,9 @@ RSpec.describe ScheduleMergeRequestCleanupRefsWorker do subject(:worker) { described_class.new } describe '#perform' do - before do - allow(MergeRequest::CleanupSchedule) - .to receive(:scheduled_merge_request_ids) - .with(described_class::LIMIT) - .and_return([1, 2, 3, 4]) - end - it 'does nothing if the database is read-only' do allow(Gitlab::Database).to receive(:read_only?).and_return(true) - expect(MergeRequestCleanupRefsWorker).not_to receive(:bulk_perform_in) + expect(MergeRequestCleanupRefsWorker).not_to receive(:perform_with_capacity) worker.perform end @@ -26,25 +19,17 @@ RSpec.describe ScheduleMergeRequestCleanupRefsWorker do end it 'does not schedule any merge request clean ups' do - expect(MergeRequestCleanupRefsWorker).not_to receive(:bulk_perform_in) + expect(MergeRequestCleanupRefsWorker).not_to receive(:perform_with_capacity) worker.perform end end include_examples 'an idempotent worker' do - it 'schedules MergeRequestCleanupRefsWorker to be performed by batch' do - expect(MergeRequestCleanupRefsWorker) - .to receive(:bulk_perform_in) - .with( - described_class::DELAY, - [[1], [2], [3], [4]], - batch_size: described_class::BATCH_SIZE - ) + it 'schedules MergeRequestCleanupRefsWorker to be performed with capacity' do + expect(MergeRequestCleanupRefsWorker).to receive(:perform_with_capacity).twice - expect(worker).to receive(:log_extra_metadata_on_done).with(:merge_requests_count, 4) - - worker.perform + subject end end end |