diff options
Diffstat (limited to 'spec/workers/merge_requests')
4 files changed, 223 insertions, 0 deletions
diff --git a/spec/workers/merge_requests/assignees_change_worker_spec.rb b/spec/workers/merge_requests/assignees_change_worker_spec.rb new file mode 100644 index 00000000000..33478daf8d3 --- /dev/null +++ b/spec/workers/merge_requests/assignees_change_worker_spec.rb @@ -0,0 +1,59 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe MergeRequests::AssigneesChangeWorker do + include AfterNextHelpers + + let_it_be(:merge_request) { create(:merge_request) } + let_it_be(:user) { create(:user) } + let_it_be(:old_assignees) { create_list(:user, 3) } + + let(:user_ids) { old_assignees.map(&:id).to_a } + let(:worker) { described_class.new } + + it_behaves_like 'an idempotent worker' do + let(:job_args) { [merge_request.id, user.id, user_ids] } + end + + describe '#perform' do + context 'with a non-existing merge request' do + it 'does nothing' do + expect(::MergeRequests::HandleAssigneesChangeService).not_to receive(:new) + + worker.perform(non_existing_record_id, user.id, user_ids) + end + end + + context 'with a non-existing user' do + it 'does nothing' do + expect(::MergeRequests::HandleAssigneesChangeService).not_to receive(:new) + + worker.perform(merge_request.id, non_existing_record_id, user_ids) + end + end + + context 'when there are no changes' do + it 'does nothing' do + expect(::MergeRequests::HandleAssigneesChangeService).not_to receive(:new) + + worker.perform(merge_request.id, user.id, merge_request.assignee_ids) + end + end + + context 'when the old users cannot be found' do + it 'does nothing' do + expect(::MergeRequests::HandleAssigneesChangeService).not_to receive(:new) + + worker.perform(merge_request.id, user.id, [non_existing_record_id]) + end + end + + it 'gets MergeRequests::UpdateAssigneesService to handle the changes' do + expect_next(::MergeRequests::HandleAssigneesChangeService) + .to receive(:execute).with(merge_request, match_array(old_assignees), execute_hooks: true) + + worker.perform(merge_request.id, user.id, user_ids) + end + end +end diff --git a/spec/workers/merge_requests/create_pipeline_worker_spec.rb b/spec/workers/merge_requests/create_pipeline_worker_spec.rb new file mode 100644 index 00000000000..8efce5220be --- /dev/null +++ b/spec/workers/merge_requests/create_pipeline_worker_spec.rb @@ -0,0 +1,61 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe MergeRequests::CreatePipelineWorker do + subject(:worker) { described_class.new } + + describe '#perform' do + let(:user) { create(:user) } + let(:project) { create(:project) } + let(:merge_request) { create(:merge_request) } + + context 'when the objects exist' do + it 'calls the merge request create pipeline service and calls update head pipeline' do + aggregate_failures do + expect_next_instance_of(MergeRequests::CreatePipelineService, project, user) do |service| + expect(service).to receive(:execute).with(merge_request) + end + + expect(MergeRequest).to receive(:find_by_id).with(merge_request.id).and_return(merge_request) + expect(merge_request).to receive(:update_head_pipeline) + + subject.perform(project.id, user.id, merge_request.id) + end + end + end + + shared_examples 'when object does not exist' do + it 'does not call the create pipeline service' do + expect(MergeRequests::CreatePipelineService).not_to receive(:new) + + expect { subject.perform(project.id, user.id, merge_request.id) } + .not_to raise_exception + end + end + + context 'when the project does not exist' do + before do + project.destroy! + end + + it_behaves_like 'when object does not exist' + end + + context 'when the user does not exist' do + before do + user.destroy! + end + + it_behaves_like 'when object does not exist' + end + + context 'when the merge request does not exist' do + before do + merge_request.destroy! + end + + it_behaves_like 'when object does not exist' + end + end +end diff --git a/spec/workers/merge_requests/handle_assignees_change_worker_spec.rb b/spec/workers/merge_requests/handle_assignees_change_worker_spec.rb new file mode 100644 index 00000000000..4b45f3562d6 --- /dev/null +++ b/spec/workers/merge_requests/handle_assignees_change_worker_spec.rb @@ -0,0 +1,62 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe MergeRequests::HandleAssigneesChangeWorker do + include AfterNextHelpers + + let_it_be(:merge_request) { create(:merge_request) } + let_it_be(:user) { create(:user) } + let_it_be(:old_assignees) { create_list(:user, 3) } + + let(:user_ids) { old_assignees.map(&:id).to_a } + let(:options) { {} } + let(:worker) { described_class.new } + + it_behaves_like 'an idempotent worker' do + let(:job_args) { [merge_request.id, user.id, user_ids, options] } + end + + describe '#perform' do + it 'calls MergeRequests::HandleAssigneesChangeService#execute to handle the changes' do + expect_next(::MergeRequests::HandleAssigneesChangeService) + .to receive(:execute).with(merge_request, match_array(old_assignees), options) + + worker.perform(merge_request.id, user.id, user_ids, options) + end + + context 'when there are no changes' do + it 'still calls MergeRequests::HandleAssigneesChangeService#execute' do + expect_next(::MergeRequests::HandleAssigneesChangeService) + .to receive(:execute).with(merge_request, [], options) + + worker.perform(merge_request.id, user.id, merge_request.assignee_ids, options) + end + end + + context 'when the old assignees cannot be found' do + it 'still calls MergeRequests::HandleAssigneesChangeService#execute' do + expect_next(::MergeRequests::HandleAssigneesChangeService) + .to receive(:execute).with(merge_request, [], options) + + worker.perform(merge_request.id, user.id, [non_existing_record_id], options) + end + end + + context 'with a non-existing merge request' do + it 'does nothing' do + expect(::MergeRequests::HandleAssigneesChangeService).not_to receive(:new) + + worker.perform(non_existing_record_id, user.id, user_ids, options) + end + end + + context 'with a non-existing user' do + it 'does nothing' do + expect(::MergeRequests::HandleAssigneesChangeService).not_to receive(:new) + + worker.perform(merge_request.id, non_existing_record_id, user_ids, options) + end + end + end +end diff --git a/spec/workers/merge_requests/resolve_todos_worker_spec.rb b/spec/workers/merge_requests/resolve_todos_worker_spec.rb new file mode 100644 index 00000000000..223b8b6803c --- /dev/null +++ b/spec/workers/merge_requests/resolve_todos_worker_spec.rb @@ -0,0 +1,41 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe MergeRequests::ResolveTodosWorker do + include AfterNextHelpers + + let_it_be(:merge_request) { create(:merge_request) } + let_it_be(:user) { create(:user) } + + let(:worker) { described_class.new } + + it_behaves_like 'an idempotent worker' do + let(:job_args) { [merge_request.id, user.id] } + end + + describe '#perform' do + it 'calls MergeRequests::ResolveTodosService#execute' do + expect_next(::MergeRequests::ResolveTodosService, merge_request, user) + .to receive(:execute) + + worker.perform(merge_request.id, user.id) + end + + context 'with a non-existing merge request' do + it 'does nothing' do + expect(::MergeRequests::ResolveTodosService).not_to receive(:new) + + worker.perform(non_existing_record_id, user.id) + end + end + + context 'with a non-existing user' do + it 'does nothing' do + expect(::MergeRequests::ResolveTodosService).not_to receive(:new) + + worker.perform(merge_request.id, non_existing_record_id) + end + end + end +end |