prevent resolve buttons from being clicked twice triggering a race condition

2ea76a41 · Mike Greiling · 9985406a · 2ea76a41
Commit 2ea76a41 authored Nov 01, 2017 by Mike Greiling
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 4 deletions

spec/features/merge_requests/diff_notes_resolve_spec.rb spec/features/merge_requests/diff_notes_resolve_spec.rb +2 -4

No files found.
--- a/spec/features/merge_requests/diff_notes_resolve_spec.rb
+++ b/spec/features/merge_requests/diff_notes_resolve_spec.rb
@@ -241,10 +241,8 @@ feature 'Diff notes resolve', :js do
      end
      it 'resolves discussion' do
-        page.all('.note').each do |note|
+        page.all('.note .line-resolve-btn').each do |button|
-          note.all('.line-resolve-btn').each do |button|
+          button.click
-            button.click
-          end
        end
        expect(page).to have_content('Resolved by')