Add a flash messages when an issue resolving discussions is created

2 files changed, 24 insertions, 0 deletions

diff --git a/spec/controllers/projects/issues_controller_spec.rb b/spec/controllers/projects/issues_controller_spec.rb
index 46e2f4cef29..70b1ae237ce 100644
--- a/spec/controllers/projects/issues_controller_spec.rb
+++ b/spec/controllers/projects/issues_controller_spec.rb
@@ -495,6 +495,20 @@ describe Projects::IssuesController do
         expect(discussion.resolved?).to eq(true)
       end
 
+      it 'sets a flash message' do
+        post_issue(title: 'Hello')
+
+        expect(flash[:notice]).to eq('Resolved 1 discussion.')
+      end
+
+      it 'pluralizes the flash message when resolving multiple discussions' do
+        create(:diff_note_on_merge_request, noteable: merge_request, project: project, line_number: 15)
+
+        post_issue(title: 'Hello')
+
+        expect(flash[:notice]).to eq('Resolved 2 discussions.')
+      end
+
       it "resolves a single discussion" do
         post_issue(other_params: { discussion_to_resolve: discussion.id })
         discussion.first_note.reload
diff --git a/spec/support/features/resolving_discussions_in_issues_shared_examples.rb b/spec/support/features/resolving_discussions_in_issues_shared_examples.rb
index 669b306d94e..c8e5021d9a6 100644
--- a/spec/support/features/resolving_discussions_in_issues_shared_examples.rb
+++ b/spec/support/features/resolving_discussions_in_issues_shared_examples.rb
@@ -23,6 +23,16 @@ shared_examples 'creating an issue for a discussion' do
     expect(discussion.resolved?).to eq(true)
   end
 
+  it 'shows a flash messaage after resolving a discussion' do
+    click_button 'Submit issue'
+
+    page.within '.flash-notice' do
+      # Only check for the word 'Resolved' since the spec might have resolved
+      # multiple discussions
+      expect(page).to have_content('Resolved')
+    end
+  end
+
   it 'has a hidden field for the merge request' do
     merge_request_field = find('#merge_request_for_resolving_discussions', visible: false)