Commit 352ec5eb authored by Paul Slaughter's avatar Paul Slaughter

CE port of "Move merge request approval settings"

This is for the upcoming feature for MR approval rules (https://gitlab.com/gitlab-org/gitlab-ee/issues/1979)
parent 50e6cc11
...@@ -95,6 +95,8 @@ ...@@ -95,6 +95,8 @@
= render 'projects/merge_request_settings', form: f = render 'projects/merge_request_settings', form: f
= f.submit 'Save changes', class: "btn btn-success qa-save-merge-request-changes" = f.submit 'Save changes', class: "btn btn-success qa-save-merge-request-changes"
= render_if_exists 'projects/merge_request_approvals_settings', expanded: expanded
= render_if_exists 'projects/service_desk_settings' = render_if_exists 'projects/service_desk_settings'
%section.settings.no-animate{ class: ('expanded' if expanded) } %section.settings.no-animate{ class: ('expanded' if expanded) }
......
...@@ -10,19 +10,19 @@ describe 'Projects > Settings > User manages merge request settings' do ...@@ -10,19 +10,19 @@ describe 'Projects > Settings > User manages merge request settings' do
end end
it 'shows "Merge commit" strategy' do it 'shows "Merge commit" strategy' do
page.within '.merge-requests-feature' do page.within '#js-merge-request-settings' do
expect(page).to have_content 'Merge commit' expect(page).to have_content 'Merge commit'
end end
end end
it 'shows "Merge commit with semi-linear history " strategy' do it 'shows "Merge commit with semi-linear history " strategy' do
page.within '.merge-requests-feature' do page.within '#js-merge-request-settings' do
expect(page).to have_content 'Merge commit with semi-linear history' expect(page).to have_content 'Merge commit with semi-linear history'
end end
end end
it 'shows "Fast-forward merge" strategy' do it 'shows "Fast-forward merge" strategy' do
page.within '.merge-requests-feature' do page.within '#js-merge-request-settings' do
expect(page).to have_content 'Fast-forward merge' expect(page).to have_content 'Fast-forward merge'
end end
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