diff options
Diffstat (limited to 'spec/features/projects/settings/user_manages_merge_requests_settings_spec.rb')
-rw-r--r-- | spec/features/projects/settings/user_manages_merge_requests_settings_spec.rb | 107 |
1 files changed, 107 insertions, 0 deletions
diff --git a/spec/features/projects/settings/user_manages_merge_requests_settings_spec.rb b/spec/features/projects/settings/user_manages_merge_requests_settings_spec.rb new file mode 100644 index 00000000000..b6e65fcbda1 --- /dev/null +++ b/spec/features/projects/settings/user_manages_merge_requests_settings_spec.rb @@ -0,0 +1,107 @@ +require 'spec_helper' + +describe 'Projects > Settings > User manages merge request settings' do + let(:user) { create(:user) } + let(:project) { create(:project, :public, namespace: user.namespace, path: 'gitlab', name: 'sample') } + + before do + sign_in(user) + visit edit_project_path(project) + end + + it 'shows "Merge commit" strategy' do + page.within '.merge-requests-feature' do + expect(page).to have_content 'Merge commit' + end + end + + it 'shows "Merge commit with semi-linear history " strategy' do + page.within '.merge-requests-feature' do + expect(page).to have_content 'Merge commit with semi-linear history' + end + end + + it 'shows "Fast-forward merge" strategy' do + page.within '.merge-requests-feature' do + expect(page).to have_content 'Fast-forward merge' + end + end + + context 'when Merge Request and Pipelines are initially enabled', :js do + context 'when Pipelines are initially enabled' do + it 'shows the Merge Requests settings' do + expect(page).to have_content('Only allow merge requests to be merged if the pipeline succeeds') + expect(page).to have_content('Only allow merge requests to be merged if all discussions are resolved') + + within('.sharing-permissions-form') do + find('.project-feature-controls[data-for="project[project_feature_attributes][merge_requests_access_level]"] .project-feature-toggle').click + find('input[value="Save changes"]').send_keys(:return) + end + + expect(page).not_to have_content('Only allow merge requests to be merged if the pipeline succeeds') + expect(page).not_to have_content('Only allow merge requests to be merged if all discussions are resolved') + end + end + + context 'when Pipelines are initially disabled', :js do + before do + project.project_feature.update_attribute('builds_access_level', ProjectFeature::DISABLED) + visit edit_project_path(project) + end + + it 'shows the Merge Requests settings that do not depend on Builds feature' do + expect(page).not_to have_content('Only allow merge requests to be merged if the pipeline succeeds') + expect(page).to have_content('Only allow merge requests to be merged if all discussions are resolved') + + within('.sharing-permissions-form') do + find('.project-feature-controls[data-for="project[project_feature_attributes][builds_access_level]"] .project-feature-toggle').click + find('input[value="Save changes"]').send_keys(:return) + end + + expect(page).to have_content('Only allow merge requests to be merged if the pipeline succeeds') + expect(page).to have_content('Only allow merge requests to be merged if all discussions are resolved') + end + end + end + + context 'when Merge Request are initially disabled', :js do + before do + project.project_feature.update_attribute('merge_requests_access_level', ProjectFeature::DISABLED) + visit edit_project_path(project) + end + + it 'does not show the Merge Requests settings' do + expect(page).not_to have_content('Only allow merge requests to be merged if the pipeline succeeds') + expect(page).not_to have_content('Only allow merge requests to be merged if all discussions are resolved') + + within('.sharing-permissions-form') do + find('.project-feature-controls[data-for="project[project_feature_attributes][merge_requests_access_level]"] .project-feature-toggle').click + find('input[value="Save changes"]').send_keys(:return) + end + + expect(page).to have_content('Only allow merge requests to be merged if the pipeline succeeds') + expect(page).to have_content('Only allow merge requests to be merged if all discussions are resolved') + end + end + + describe 'Checkbox to enable merge request link', :js do + it 'is initially checked' do + checkbox = find_field('project_printing_merge_request_link_enabled') + expect(checkbox).to be_checked + end + + it 'when unchecked sets :printing_merge_request_link_enabled to false' do + uncheck('project_printing_merge_request_link_enabled') + within('.merge-request-settings-form') do + click_on('Save changes') + end + + # Wait for save to complete and page to reload + checkbox = find_field('project_printing_merge_request_link_enabled') + expect(checkbox).not_to be_checked + + project.reload + expect(project.printing_merge_request_link_enabled).to be(false) + end + end +end |