2017-09-10 17:25:29 +05:30
|
|
|
require 'spec_helper'
|
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
describe BackgroundMigrationWorker, :sidekiq, :clean_gitlab_redis_shared_state do
|
|
|
|
let(:worker) { described_class.new }
|
|
|
|
|
2018-11-18 11:00:15 +05:30
|
|
|
describe '.minimum_interval' do
|
|
|
|
it 'returns 2 minutes' do
|
|
|
|
expect(described_class.minimum_interval).to eq(2.minutes.to_i)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
describe '.perform' do
|
|
|
|
it 'performs a background migration' do
|
|
|
|
expect(Gitlab::BackgroundMigration)
|
|
|
|
.to receive(:perform)
|
|
|
|
.with('Foo', [10, 20])
|
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
worker.perform('Foo', [10, 20])
|
2017-09-10 17:25:29 +05:30
|
|
|
end
|
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
it 'reschedules a migration if it was performed recently' do
|
|
|
|
expect(worker)
|
|
|
|
.to receive(:always_perform?)
|
|
|
|
.and_return(false)
|
2017-09-10 17:25:29 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
worker.lease_for('Foo').try_obtain
|
2017-09-10 17:25:29 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
expect(Gitlab::BackgroundMigration)
|
|
|
|
.not_to receive(:perform)
|
2017-09-10 17:25:29 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
expect(described_class)
|
|
|
|
.to receive(:perform_in)
|
|
|
|
.with(a_kind_of(Numeric), 'Foo', [10, 20])
|
2017-09-10 17:25:29 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
worker.perform('Foo', [10, 20])
|
2017-09-10 17:25:29 +05:30
|
|
|
end
|
2018-11-18 11:00:15 +05:30
|
|
|
|
|
|
|
it 'reschedules a migration if the database is not healthy' do
|
|
|
|
allow(worker)
|
|
|
|
.to receive(:always_perform?)
|
|
|
|
.and_return(false)
|
|
|
|
|
|
|
|
allow(worker)
|
|
|
|
.to receive(:healthy_database?)
|
|
|
|
.and_return(false)
|
|
|
|
|
|
|
|
expect(described_class)
|
|
|
|
.to receive(:perform_in)
|
|
|
|
.with(a_kind_of(Numeric), 'Foo', [10, 20])
|
|
|
|
|
|
|
|
worker.perform('Foo', [10, 20])
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#healthy_database?' do
|
|
|
|
context 'using MySQL', :mysql do
|
|
|
|
it 'returns true' do
|
|
|
|
expect(worker.healthy_database?).to eq(true)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'using PostgreSQL', :postgresql do
|
|
|
|
context 'when replication lag is too great' do
|
|
|
|
it 'returns false' do
|
|
|
|
allow(Postgresql::ReplicationSlot)
|
|
|
|
.to receive(:lag_too_great?)
|
|
|
|
.and_return(true)
|
|
|
|
|
|
|
|
expect(worker.healthy_database?).to eq(false)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when replication lag is small enough' do
|
|
|
|
it 'returns true' do
|
|
|
|
allow(Postgresql::ReplicationSlot)
|
|
|
|
.to receive(:lag_too_great?)
|
|
|
|
.and_return(false)
|
|
|
|
|
|
|
|
expect(worker.healthy_database?).to eq(true)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2017-09-10 17:25:29 +05:30
|
|
|
end
|
|
|
|
end
|