# frozen_string_literal: true require 'spec_helper' RSpec.describe Gitlab::Database::Reindexing::Coordinator do include Database::DatabaseHelpers include ExclusiveLeaseHelpers describe '.perform' do subject { described_class.new(index, notifier).perform } let(:index) { create(:postgres_index) } let(:notifier) { instance_double(Gitlab::Database::Reindexing::GrafanaNotifier, notify_start: nil, notify_end: nil) } let(:reindexer) { instance_double(Gitlab::Database::Reindexing::ReindexConcurrently, perform: nil) } let(:action) { create(:reindex_action, index: index) } let!(:lease) { stub_exclusive_lease(lease_key, uuid, timeout: lease_timeout) } let(:lease_key) { "gitlab/database/reindexing/coordinator/#{Gitlab::Database::PRIMARY_DATABASE_NAME}" } let(:lease_timeout) { 1.day } let(:uuid) { 'uuid' } around do |example| model = Gitlab::Database.database_base_models[Gitlab::Database::PRIMARY_DATABASE_NAME] Gitlab::Database::SharedModel.using_connection(model.connection) do example.run end end before do swapout_view_for_table(:postgres_indexes) allow(Gitlab::Database::Reindexing::ReindexConcurrently).to receive(:new).with(index).and_return(reindexer) allow(Gitlab::Database::Reindexing::ReindexAction).to receive(:create_for).with(index).and_return(action) end context 'locking' do it 'acquires a lock while reindexing' do expect(lease).to receive(:try_obtain).ordered.and_return(uuid) expect(reindexer).to receive(:perform).ordered expect(Gitlab::ExclusiveLease).to receive(:cancel).ordered.with(lease_key, uuid) subject end it 'does not perform reindexing actions if lease is not granted' do expect(lease).to receive(:try_obtain).ordered.and_return(false) expect(Gitlab::Database::Reindexing::ReindexConcurrently).not_to receive(:new) subject end end context 'notifications' do it 'sends #notify_start before reindexing' do expect(notifier).to receive(:notify_start).with(action).ordered expect(reindexer).to receive(:perform).ordered subject end it 'sends #notify_end after reindexing and updating the action is done' do expect(action).to receive(:finish).ordered expect(notifier).to receive(:notify_end).with(action).ordered subject end end context 'action tracking' do it 'calls #finish on the action' do expect(reindexer).to receive(:perform).ordered expect(action).to receive(:finish).ordered subject end it 'upon error, it still calls finish and raises the error' do expect(reindexer).to receive(:perform).ordered.and_raise('something went wrong') expect(action).to receive(:finish).ordered expect { subject }.to raise_error(/something went wrong/) expect(action).to be_failed end end end end