Tests for not logged in user

1 files changed, 30 insertions, 7 deletions

diff --git a/spec/features/merge_requests/diff_notes_resolve_spec.rb b/spec/features/merge_requests/diff_notes_resolve_spec.rb
index cdfaffb19f9..0216a8f91af 100644
--- a/spec/features/merge_requests/diff_notes_resolve_spec.rb
+++ b/spec/features/merge_requests/diff_notes_resolve_spec.rb
@@ -433,17 +433,40 @@ feature 'Diff notes resolve', feature: true, js: true do
   end
 
   context 'unauthorized user' do
-    before do
-      visit_merge_request
+    context 'no resolved comments' do
+      before do
+        visit_merge_request
+      end
+
+      it 'does not allow user to mark note as resolved' do
+        page.within '.diff-content .note' do
+          expect(page).not_to have_selector('.line-resolve-btn')
+        end
+
+        page.within '.line-resolve-all-container' do
+          expect(page).to have_content('0/1 discussion resolved')
+        end
+      end
     end
 
-    it 'does not allow user to mark note as resolved' do
-      page.within '.diff-content .note' do
-        expect(page).not_to have_selector('.line-resolve-btn')
+    context 'resolved comment' do
+      before do
+        note.resolve!(user)
+        visit_merge_request
+      end
+
+      it 'shows resolved icon' do
+        expect(page).to have_content '1/1 discussion resolved'
+
+        click_link 'Toggle discussion'
+        expect(page).to have_selector('.line-resolve-btn.is-active')
       end
 
-      page.within '.line-resolve-all-container' do
-        expect(page).to have_content('0/1 discussion resolved')
+      it 'does not allow user to click resolve button' do
+        expect(page).to have_selector('.line-resolve-btn.is-disabled')
+        click_link 'Toggle discussion'
+
+        expect(page).to have_selector('.line-resolve-btn.is-disabled')
       end
     end
   end