Commit dad55d88 authored by Bryce Johnson's avatar Bryce Johnson

Fix integration test selector for Unapprove button.

parent b4d265cc
...@@ -170,13 +170,13 @@ feature 'Merge request approvals', js: true, feature: true do ...@@ -170,13 +170,13 @@ feature 'Merge request approvals', js: true, feature: true do
it 'I am able to approve' do it 'I am able to approve' do
approve_merge_request approve_merge_request
expect(page).to have_content("Approved by") expect(page).to have_content('Approved by')
end end
it 'I am able to unapprove' do it 'I am able to unapprove', focus: true do
approve_merge_request approve_merge_request
unapprove_merge_request unapprove_merge_request
expect(page).not_to have_content("Approved by") expect(page).not_to have_content('Approved by')
end end
end end
...@@ -194,13 +194,13 @@ feature 'Merge request approvals', js: true, feature: true do ...@@ -194,13 +194,13 @@ feature 'Merge request approvals', js: true, feature: true do
end end
wait_for_ajax wait_for_ajax
expect(page).to have_content("Approved by") expect(page).to have_content('Approved by')
end end
it 'I am able to unapprove' do it 'I am able to unapprove', focus: true do
approve_merge_request approve_merge_request
unapprove_merge_request unapprove_merge_request
expect(page).not_to have_content("Approved by") expect(page).not_to have_content('Approved by')
end end
end end
end end
...@@ -215,7 +215,7 @@ end ...@@ -215,7 +215,7 @@ end
def unapprove_merge_request def unapprove_merge_request
page.within '.mr-state-widget' do page.within '.mr-state-widget' do
click_button 'Remove your approval' find('.unapprove-btn-wrap').click
end end
wait_for_ajax wait_for_ajax
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