diff options
Diffstat (limited to 'spec/services/notification_service_spec.rb')
| -rw-r--r-- | spec/services/notification_service_spec.rb | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/spec/services/notification_service_spec.rb b/spec/services/notification_service_spec.rb index d3cc367649f..028c3ea6610 100644 --- a/spec/services/notification_service_spec.rb +++ b/spec/services/notification_service_spec.rb @@ -2895,6 +2895,7 @@ RSpec.describe NotificationService, :mailer, feature_category: :team_planning do describe '#review_requested_of_merge_request' do let(:merge_request) { create(:merge_request, author: author, source_project: project, reviewers: [reviewer]) } + let(:mailer) { double } let_it_be(:current_user) { create(:user) } let_it_be(:reviewer) { create(:user) } @@ -2917,8 +2918,16 @@ RSpec.describe NotificationService, :mailer, feature_category: :team_planning do should_not_email(@u_lazy_participant) end + it 'deliver email immediately' do + allow(Notify).to receive(:request_review_merge_request_email) + .with(Integer, Integer, Integer, String).and_return(mailer) + expect(mailer).to receive(:deliver_later).with({}) + + notification.review_requested_of_merge_request(merge_request, current_user, reviewer) + end + it 'adds "review requested" reason for new reviewer' do - notification.review_requested_of_merge_request(merge_request, current_user, [reviewer]) + notification.review_requested_of_merge_request(merge_request, current_user, reviewer) merge_request.reviewers.each do |reviewer| email = find_email_for(reviewer) |