diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-10-21 07:08:36 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-10-21 07:08:36 +0000 |
commit | 48aff82709769b098321c738f3444b9bdaa694c6 (patch) | |
tree | e00c7c43e2d9b603a5a6af576b1685e400410dee /spec/services/admin/propagate_integration_service_spec.rb | |
parent | 879f5329ee916a948223f8f43d77fba4da6cd028 (diff) | |
download | gitlab-ce-48aff82709769b098321c738f3444b9bdaa694c6.tar.gz |
Add latest changes from gitlab-org/gitlab@13-5-stable-eev13.5.0-rc42
Diffstat (limited to 'spec/services/admin/propagate_integration_service_spec.rb')
-rw-r--r-- | spec/services/admin/propagate_integration_service_spec.rb | 142 |
1 files changed, 43 insertions, 99 deletions
diff --git a/spec/services/admin/propagate_integration_service_spec.rb b/spec/services/admin/propagate_integration_service_spec.rb index 49d974b7154..5df4d9db8b1 100644 --- a/spec/services/admin/propagate_integration_service_spec.rb +++ b/spec/services/admin/propagate_integration_service_spec.rb @@ -10,129 +10,73 @@ RSpec.describe Admin::PropagateIntegrationService do stub_jira_service_test end - let(:excluded_attributes) { %w[id project_id group_id inherit_from_id instance created_at updated_at default] } - let!(:project) { create(:project) } - let!(:group) { create(:group) } - let!(:instance_integration) do - JiraService.create!( - instance: true, - active: true, - push_events: true, - url: 'http://update-jira.instance.com', - username: 'user', - password: 'secret' - ) - end + let(:group) { create(:group) } - let!(:inherited_integration) do - JiraService.create!( - project: create(:project), - inherit_from_id: instance_integration.id, - instance: false, - active: true, - push_events: false, - url: 'http://jira.instance.com', - username: 'user', - password: 'secret' - ) + let_it_be(:project) { create(:project) } + let_it_be(:instance_integration) { create(:jira_service, :instance) } + let_it_be(:not_inherited_integration) { create(:jira_service, project: project) } + let_it_be(:inherited_integration) do + create(:jira_service, project: create(:project), inherit_from_id: instance_integration.id) end - - let!(:not_inherited_integration) do - JiraService.create!( - project: create(:project), - inherit_from_id: nil, - instance: false, - active: true, - push_events: false, - url: 'http://jira.instance.com', - username: 'user', - password: 'secret' - ) + let_it_be(:different_type_inherited_integration) do + create(:redmine_service, project: project, inherit_from_id: instance_integration.id) end - let!(:different_type_inherited_integration) do - BambooService.create!( - project: create(:project), - inherit_from_id: instance_integration.id, - instance: false, - active: true, - push_events: false, - bamboo_url: 'http://gitlab.com', - username: 'mic', - password: 'password', - build_key: 'build' - ) - end + context 'with inherited integration' do + let(:integration) { inherited_integration } - shared_examples 'inherits settings from integration' do - it 'updates the inherited integrations' do - described_class.propagate(instance_integration) + it 'calls to PropagateIntegrationProjectWorker' do + expect(PropagateIntegrationInheritWorker).to receive(:perform_async) + .with(instance_integration.id, inherited_integration.id, inherited_integration.id) - expect(integration.reload.inherit_from_id).to eq(instance_integration.id) - expect(integration.attributes.except(*excluded_attributes)) - .to eq(instance_integration.attributes.except(*excluded_attributes)) + described_class.propagate(instance_integration) end + end - context 'integration with data fields' do - let(:excluded_attributes) { %w[id service_id created_at updated_at] } + context 'with a project without integration' do + let(:another_project) { create(:project) } - it 'updates the data fields from inherited integrations' do - described_class.propagate(instance_integration) + it 'calls to PropagateIntegrationProjectWorker' do + expect(PropagateIntegrationProjectWorker).to receive(:perform_async) + .with(instance_integration.id, another_project.id, another_project.id) - expect(integration.reload.data_fields.attributes.except(*excluded_attributes)) - .to eq(instance_integration.data_fields.attributes.except(*excluded_attributes)) - end + described_class.propagate(instance_integration) end end - shared_examples 'does not inherit settings from integration' do - it 'does not update the not inherited integrations' do - described_class.propagate(instance_integration) + context 'with a group without integration' do + it 'calls to PropagateIntegrationProjectWorker' do + expect(PropagateIntegrationGroupWorker).to receive(:perform_async) + .with(instance_integration.id, group.id, group.id) - expect(integration.reload.attributes.except(*excluded_attributes)) - .not_to eq(instance_integration.attributes.except(*excluded_attributes)) + described_class.propagate(instance_integration) end end - context 'update only inherited integrations' do - it_behaves_like 'inherits settings from integration' do - let(:integration) { inherited_integration } - end - - it_behaves_like 'does not inherit settings from integration' do - let(:integration) { not_inherited_integration } - end + context 'for a group-level integration' do + let(:group_integration) { create(:jira_service, group: group, project: nil) } - it_behaves_like 'does not inherit settings from integration' do - let(:integration) { different_type_inherited_integration } - end + context 'with a project without integration' do + let(:another_project) { create(:project, group: group) } - it_behaves_like 'inherits settings from integration' do - let(:integration) { project.jira_service } - end + it 'calls to PropagateIntegrationProjectWorker' do + expect(PropagateIntegrationProjectWorker).to receive(:perform_async) + .with(group_integration.id, another_project.id, another_project.id) - it_behaves_like 'inherits settings from integration' do - let(:integration) { Service.find_by(group_id: group.id) } + described_class.propagate(group_integration) + end end - end - it 'updates project#has_external_issue_tracker for issue tracker services' do - described_class.propagate(instance_integration) + context 'with a group without integration' do + let(:subgroup) { create(:group, parent: group) } - expect(project.reload.has_external_issue_tracker).to eq(true) - end - - it 'updates project#has_external_wiki for external wiki services' do - instance_integration = ExternalWikiService.create!( - instance: true, - active: true, - push_events: false, - external_wiki_url: 'http://external-wiki-url.com' - ) - - described_class.propagate(instance_integration) + it 'calls to PropagateIntegrationGroupWorker' do + expect(PropagateIntegrationGroupWorker).to receive(:perform_async) + .with(group_integration.id, subgroup.id, subgroup.id) - expect(project.reload.has_external_wiki).to eq(true) + described_class.propagate(group_integration) + end + end end end end |