Merge branch 'add-resolve-discussion-test' into 'master'

Add explicit test to test resolved discussion toggle content

See merge request gitlab-org/gitlab-ce!14796

1 files changed, 15 insertions, 5 deletions

diff --git a/spec/features/merge_requests/diff_notes_resolve_spec.rb b/spec/features/merge_requests/diff_notes_resolve_spec.rb
index 637e6036384..475c8586f45 100644
--- a/spec/features/merge_requests/diff_notes_resolve_spec.rb
+++ b/spec/features/merge_requests/diff_notes_resolve_spec.rb
@@ -88,14 +88,24 @@ feature 'Diff notes resolve', :js do
         end
       end
 
-      it 'hides resolved discussion' do
-        page.within '.diff-content' do
-          click_button 'Resolve discussion'
+      describe 'resolved discussion' do
+        before do
+          page.within '.diff-content' do
+            click_button 'Resolve discussion'
+          end
+
+          visit_merge_request
         end
 
-        visit_merge_request
+        it 'hides when resolve discussion is clicked' do
+          expect(page).to have_selector('.discussion-body', visible: false)
+        end
 
-        expect(page).to have_selector('.discussion-body', visible: false)
+        it 'shows resolved discussion when toggled' do
+          find(".timeline-content .discussion[data-discussion-id='#{note.discussion_id}'] .discussion-toggle-button").click
+
+          expect(page.find(".timeline-content #note_#{note.noteable_id}")).to be_visible
+        end
       end
 
       it 'allows user to resolve from reply form without a comment' do