Commit 44a18847 authored by Thong Kuah's avatar Thong Kuah

Fix spec failing on certain times

This is because the worker runs for each DB depending on the minute, so
we get flakiness for test environments with multiple databases setup.
parent 9b3e2570
......@@ -102,8 +102,22 @@ RSpec.describe LooseForeignKeys::CleanupWorker do
loose_fk_parent_table_2.delete_all
end
it 'cleans up all rows' do
def perform_for(db:)
time = Time.current.midnight
if db == :main
time += 2.minutes
elsif db == :ci
time += 3.minutes
end
travel_to(time) do
described_class.new.perform
end
end
it 'cleans up all rows' do
perform_for(db: :main)
expect(loose_fk_child_table_1_1.count).to eq(0)
expect(loose_fk_child_table_1_2.where(parent_id_with_different_column: nil).count).to eq(4)
......@@ -118,7 +132,7 @@ RSpec.describe LooseForeignKeys::CleanupWorker do
it 'cleans up all rows' do
expect(LooseForeignKeys::BatchCleanerService).to receive(:new).exactly(:twice).and_call_original
described_class.new.perform
perform_for(db: :main)
expect(loose_fk_child_table_1_1.count).to eq(0)
expect(loose_fk_child_table_1_2.where(parent_id_with_different_column: nil).count).to eq(4)
......@@ -137,7 +151,7 @@ RSpec.describe LooseForeignKeys::CleanupWorker do
end
it 'cleans up 2 rows' do
expect { described_class.new.perform }.to change { count_deletable_rows }.by(-2)
expect { perform_for(db: :main) }.to change { count_deletable_rows }.by(-2)
end
end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment