diff options
Diffstat (limited to 'spec/workers/container_expiration_policy_worker_spec.rb')
-rw-r--r-- | spec/workers/container_expiration_policy_worker_spec.rb | 35 |
1 files changed, 33 insertions, 2 deletions
diff --git a/spec/workers/container_expiration_policy_worker_spec.rb b/spec/workers/container_expiration_policy_worker_spec.rb index 6f81d06f653..69ddbe5c0f4 100644 --- a/spec/workers/container_expiration_policy_worker_spec.rb +++ b/spec/workers/container_expiration_policy_worker_spec.rb @@ -113,8 +113,8 @@ RSpec.describe ContainerExpirationPolicyWorker do context 'process stale ongoing cleanups' do let_it_be(:stuck_cleanup) { create(:container_repository, :cleanup_ongoing, expiration_policy_started_at: 1.day.ago) } - let_it_be(:container_repository) { create(:container_repository, :cleanup_scheduled) } - let_it_be(:container_repository) { create(:container_repository, :cleanup_unfinished) } + let_it_be(:container_repository1) { create(:container_repository, :cleanup_scheduled) } + let_it_be(:container_repository2) { create(:container_repository, :cleanup_unfinished) } it 'set them as unfinished' do expect { subject } @@ -137,5 +137,36 @@ RSpec.describe ContainerExpirationPolicyWorker do expect(container_expiration_policy3.reload.enabled).to be false end end + + context 'counts logging' do + let_it_be(:container_repository1) { create(:container_repository, :cleanup_scheduled) } + let_it_be(:container_repository2) { create(:container_repository, :cleanup_unfinished) } + let_it_be(:container_repository3) { create(:container_repository, :cleanup_unfinished) } + + before do + ContainerExpirationPolicy.update_all(enabled: true) + container_repository1.project.container_expiration_policy.update_column(:next_run_at, 5.minutes.ago) + end + + it 'logs all the counts' do + expect(worker).to receive(:log_extra_metadata_on_done).with(:cleanup_required_count, 1) + expect(worker).to receive(:log_extra_metadata_on_done).with(:cleanup_unfinished_count, 2) + expect(worker).to receive(:log_extra_metadata_on_done).with(:cleanup_total_count, 3) + + subject + end + + context 'with load balancing enabled' do + before do + allow(Gitlab::Database::LoadBalancing).to receive(:enable?).and_return(true) + end + + it 'reads the counts from the replica' do + expect(Gitlab::Database::LoadBalancing::Session.current).to receive(:use_replicas_for_read_queries).and_call_original + + subject + end + end + end end end |