Commit 1301572a authored by Ruben Davila's avatar Ruben Davila

Fix broken 'Edit Merge Request' spec.

parent 6cb2559a
......@@ -9,13 +9,9 @@ feature 'Edit Merge Request', feature: true do
before do
project.team << [user, :master]
login_as user
visit edit_namespace_project_merge_request_path(project.namespace, project, merge_request)
visit_edit_mr_page
end
end
context 'editing a MR' do
it 'has class js-quick-submit in form' do
expect(page).to have_selector('.js-quick-submit')
end
......@@ -26,9 +22,7 @@ feature 'Edit Merge Request', feature: true do
project.team << [user, :master]
project.update_attributes(approvals_before_merge: 2)
login_as user
visit edit_namespace_project_merge_request_path(project.namespace, project, merge_request)
visit_edit_mr_page
end
it 'shows the saved MR' do
......@@ -37,4 +31,10 @@ feature 'Edit Merge Request', feature: true do
expect(page).to have_link('Close merge request')
end
end
def visit_edit_mr_page
login_as user
visit edit_namespace_project_merge_request_path(project.namespace, project, merge_request)
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