diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2021-06-16 21:25:58 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2021-06-16 21:25:58 +0300 |
commit | a5f4bba440d7f9ea47046a0a561d49adf0a1e6d4 (patch) | |
tree | fb69158581673816a8cd895f9d352dcb3c678b1e /spec/services/merge_requests/handle_assignees_change_service_spec.rb | |
parent | d16b2e8639e99961de6ddc93909f3bb5c1445ba1 (diff) |
Add latest changes from gitlab-org/gitlab@14-0-stable-eev14.0.0-rc42
Diffstat (limited to 'spec/services/merge_requests/handle_assignees_change_service_spec.rb')
-rw-r--r-- | spec/services/merge_requests/handle_assignees_change_service_spec.rb | 22 |
1 files changed, 18 insertions, 4 deletions
diff --git a/spec/services/merge_requests/handle_assignees_change_service_spec.rb b/spec/services/merge_requests/handle_assignees_change_service_spec.rb index 0bf18f16abb..f9eed6eea2d 100644 --- a/spec/services/merge_requests/handle_assignees_change_service_spec.rb +++ b/spec/services/merge_requests/handle_assignees_change_service_spec.rb @@ -6,7 +6,7 @@ RSpec.describe MergeRequests::HandleAssigneesChangeService do let_it_be(:project) { create(:project, :repository) } let_it_be(:user) { create(:user) } let_it_be(:assignee) { create(:user) } - let_it_be(:merge_request) { create(:merge_request, author: user, source_project: project, assignees: [assignee]) } + let_it_be_with_reload(:merge_request) { create(:merge_request, author: user, source_project: project, assignees: [assignee]) } let_it_be(:old_assignees) { create_list(:user, 3) } let(:options) { {} } @@ -45,13 +45,27 @@ RSpec.describe MergeRequests::HandleAssigneesChangeService do service.execute(merge_request, old_assignees, options) end + let(:note) { merge_request.notes.system.last } + let(:removed_note) { "unassigned #{old_assignees.map(&:to_reference).to_sentence}" } + + context 'when unassigning all users' do + before do + merge_request.update!(assignee_ids: []) + end + + it 'creates assignee note' do + execute + + expect(note).not_to be_nil + expect(note.note).to eq removed_note + end + end + it 'creates assignee note' do execute - note = merge_request.notes.last - expect(note).not_to be_nil - expect(note.note).to include "assigned to #{assignee.to_reference} and unassigned #{old_assignees.map(&:to_reference).to_sentence}" + expect(note.note).to include "assigned to #{assignee.to_reference} and #{removed_note}" end it 'sends email notifications to old and new assignees', :mailer, :sidekiq_inline do |