diff options
Diffstat (limited to 'spec/workers/merge_requests/assignees_change_worker_spec.rb')
-rw-r--r-- | spec/workers/merge_requests/assignees_change_worker_spec.rb | 59 |
1 files changed, 59 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 |