Commit 0e10711b authored by Winnie Hellmann's avatar Winnie Hellmann

Add failing test for clearing local batch comment draft

parent d590467f
...@@ -167,6 +167,8 @@ describe 'Merge request > Batch comments', :js do ...@@ -167,6 +167,8 @@ describe 'Merge request > Batch comments', :js do
expect(page).to have_content('1/1 discussion resolved') expect(page).to have_content('1/1 discussion resolved')
expect(page).to have_selector('.line-resolve-btn.is-active') expect(page).to have_selector('.line-resolve-btn.is-active')
end end
expect_empty_local_draft
end end
end end
...@@ -210,6 +212,8 @@ describe 'Merge request > Batch comments', :js do ...@@ -210,6 +212,8 @@ describe 'Merge request > Batch comments', :js do
expect(page).to have_content('0/1 discussion resolved') expect(page).to have_content('0/1 discussion resolved')
expect(page).to have_selector('.line-resolve-btn.is-disabled') expect(page).to have_selector('.line-resolve-btn.is-disabled')
end end
expect_empty_local_draft
end end
end end
end end
...@@ -263,3 +267,10 @@ def write_reply_to_discussion(button_text: 'Start a review', text: 'Line is wron ...@@ -263,3 +267,10 @@ def write_reply_to_discussion(button_text: 'Start a review', text: 'Line is wron
wait_for_requests wait_for_requests
end end
def expect_empty_local_draft
page.within('.discussion-reply-holder') do
click_button('Reply...')
expect(find('#note_note').value).to eq('')
end
end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment