2022-01-26 12:08:38 +05:30
|
|
|
# frozen_string_literal: true
|
|
|
|
|
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
RSpec.describe Ci::ProcessSyncEventsService do
|
|
|
|
let!(:group) { create(:group) }
|
|
|
|
let!(:project1) { create(:project, group: group) }
|
|
|
|
let!(:project2) { create(:project, group: group) }
|
|
|
|
let!(:parent_group_1) { create(:group) }
|
|
|
|
let!(:parent_group_2) { create(:group) }
|
|
|
|
|
|
|
|
subject(:service) { described_class.new(sync_event_class, hierarchy_class) }
|
|
|
|
|
|
|
|
describe '#perform' do
|
|
|
|
subject(:execute) { service.execute }
|
|
|
|
|
|
|
|
context 'for Projects::SyncEvent' do
|
|
|
|
let(:sync_event_class) { Projects::SyncEvent }
|
|
|
|
let(:hierarchy_class) { ::Ci::ProjectMirror }
|
|
|
|
|
|
|
|
before do
|
|
|
|
Projects::SyncEvent.delete_all
|
|
|
|
|
|
|
|
project1.update!(group: parent_group_1)
|
|
|
|
project2.update!(group: parent_group_2)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'consumes events' do
|
|
|
|
expect { execute }.to change(Projects::SyncEvent, :count).from(2).to(0)
|
|
|
|
|
2022-03-02 08:16:31 +05:30
|
|
|
expect(project1.reload.ci_project_mirror).to have_attributes(
|
2022-01-26 12:08:38 +05:30
|
|
|
namespace_id: parent_group_1.id
|
|
|
|
)
|
2022-03-02 08:16:31 +05:30
|
|
|
expect(project2.reload.ci_project_mirror).to have_attributes(
|
2022-01-26 12:08:38 +05:30
|
|
|
namespace_id: parent_group_2.id
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'enqueues Projects::ProcessSyncEventsWorker if any left' do
|
|
|
|
stub_const("#{described_class}::BATCH_SIZE", 1)
|
|
|
|
|
|
|
|
expect(Projects::ProcessSyncEventsWorker).to receive(:perform_async)
|
|
|
|
|
|
|
|
execute
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not enqueue Projects::ProcessSyncEventsWorker if no left' do
|
|
|
|
stub_const("#{described_class}::BATCH_SIZE", 2)
|
|
|
|
|
|
|
|
expect(Projects::ProcessSyncEventsWorker).not_to receive(:perform_async)
|
|
|
|
|
|
|
|
execute
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when there is no event' do
|
|
|
|
before do
|
|
|
|
Projects::SyncEvent.delete_all
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does nothing' do
|
|
|
|
expect { execute }.not_to change(Projects::SyncEvent, :count)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the FF ci_namespace_project_mirrors is disabled' do
|
|
|
|
before do
|
|
|
|
stub_feature_flags(ci_namespace_project_mirrors: false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does nothing' do
|
|
|
|
expect { execute }.not_to change(Projects::SyncEvent, :count)
|
|
|
|
end
|
|
|
|
end
|
2022-03-02 08:16:31 +05:30
|
|
|
|
|
|
|
it 'does not delete non-executed events' do
|
|
|
|
new_project = create(:project)
|
|
|
|
sync_event_class.delete_all
|
|
|
|
|
|
|
|
project1.update!(group: parent_group_2)
|
|
|
|
new_project.update!(group: parent_group_1)
|
|
|
|
project2.update!(group: parent_group_1)
|
|
|
|
|
|
|
|
new_project_sync_event = new_project.sync_events.last
|
|
|
|
|
|
|
|
allow(sync_event_class).to receive(:preload_synced_relation).and_return(
|
|
|
|
sync_event_class.where.not(id: new_project_sync_event)
|
|
|
|
)
|
|
|
|
|
|
|
|
expect { execute }.to change(Projects::SyncEvent, :count).from(3).to(1)
|
|
|
|
expect(new_project_sync_event.reload).to be_persisted
|
|
|
|
end
|
2022-01-26 12:08:38 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
context 'for Namespaces::SyncEvent' do
|
|
|
|
let(:sync_event_class) { Namespaces::SyncEvent }
|
|
|
|
let(:hierarchy_class) { ::Ci::NamespaceMirror }
|
|
|
|
|
|
|
|
before do
|
|
|
|
Namespaces::SyncEvent.delete_all
|
|
|
|
|
|
|
|
group.update!(parent: parent_group_2)
|
|
|
|
parent_group_2.update!(parent: parent_group_1)
|
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'event consuming' do
|
|
|
|
it 'consumes events' do
|
|
|
|
expect { execute }.to change(Namespaces::SyncEvent, :count).from(2).to(0)
|
|
|
|
|
2022-03-02 08:16:31 +05:30
|
|
|
expect(group.reload.ci_namespace_mirror).to have_attributes(
|
2022-01-26 12:08:38 +05:30
|
|
|
traversal_ids: [parent_group_1.id, parent_group_2.id, group.id]
|
|
|
|
)
|
2022-03-02 08:16:31 +05:30
|
|
|
expect(parent_group_2.reload.ci_namespace_mirror).to have_attributes(
|
2022-01-26 12:08:38 +05:30
|
|
|
traversal_ids: [parent_group_1.id, parent_group_2.id]
|
|
|
|
)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the FFs sync_traversal_ids, use_traversal_ids and use_traversal_ids_for_ancestors are disabled' do
|
|
|
|
before do
|
|
|
|
stub_feature_flags(sync_traversal_ids: false,
|
|
|
|
use_traversal_ids: false,
|
|
|
|
use_traversal_ids_for_ancestors: false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'event consuming'
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'event consuming'
|
|
|
|
|
|
|
|
it 'enqueues Namespaces::ProcessSyncEventsWorker if any left' do
|
|
|
|
stub_const("#{described_class}::BATCH_SIZE", 1)
|
|
|
|
|
|
|
|
expect(Namespaces::ProcessSyncEventsWorker).to receive(:perform_async)
|
|
|
|
|
|
|
|
execute
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not enqueue Namespaces::ProcessSyncEventsWorker if no left' do
|
|
|
|
stub_const("#{described_class}::BATCH_SIZE", 2)
|
|
|
|
|
|
|
|
expect(Namespaces::ProcessSyncEventsWorker).not_to receive(:perform_async)
|
|
|
|
|
|
|
|
execute
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|