Commit 501d2df4 authored by Clement Ho's avatar Clement Ho

Add explicit test to test resolved discussion toggle content

parent 9ac5338b
...@@ -88,16 +88,26 @@ feature 'Diff notes resolve', :js do ...@@ -88,16 +88,26 @@ feature 'Diff notes resolve', :js do
end end
end end
it 'hides resolved discussion' do describe 'resolved discussion' do
before do
page.within '.diff-content' do page.within '.diff-content' do
click_button 'Resolve discussion' click_button 'Resolve discussion'
end end
visit_merge_request visit_merge_request
end
it 'hides when resolve discussion is clicked' do
expect(page).to have_selector('.discussion-body', visible: false) expect(page).to have_selector('.discussion-body', visible: false)
end end
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 it 'allows user to resolve from reply form without a comment' do
page.within '.diff-content' do page.within '.diff-content' do
click_button 'Reply...' click_button 'Reply...'
......
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