merge_requests_settings_spec.rb 3.86 KB
Newer Older
1 2
require 'spec_helper'

3
feature 'Project settings > Merge Requests', :js do
4
  let(:project) { create(:project, :public) }
5 6 7 8
  let(:user) { create(:user) }

  background do
    project.team << [user, :master]
9
    sign_in(user)
10 11
  end

Filipa Lacerda's avatar
Filipa Lacerda committed
12 13
  context 'when Merge Request and Pipelines are initially enabled' do
    context 'when Pipelines are initially enabled' do
14 15 16 17 18
      before do
        visit edit_project_path(project)
      end

      scenario 'shows the Merge Requests settings' do
Filipa Lacerda's avatar
Filipa Lacerda committed
19
        expect(page).to have_content('Only allow merge requests to be merged if the pipeline succeeds')
20 21
        expect(page).to have_content('Only allow merge requests to be merged if all discussions are resolved')

tauriedavis's avatar
tauriedavis committed
22
        within('.sharing-permissions-form') do
23
          find('.project-feature-controls[data-for="project[project_feature_attributes][merge_requests_access_level]"] .project-feature-toggle').click
tauriedavis's avatar
tauriedavis committed
24 25
          click_on('Save changes')
        end
26

Filipa Lacerda's avatar
Filipa Lacerda committed
27
        expect(page).not_to have_content('Only allow merge requests to be merged if the pipeline succeeds')
28 29 30 31
        expect(page).not_to have_content('Only allow merge requests to be merged if all discussions are resolved')
      end
    end

Filipa Lacerda's avatar
Filipa Lacerda committed
32
    context 'when Pipelines are initially disabled' do
33 34 35 36 37 38
      before do
        project.project_feature.update_attribute('builds_access_level', ProjectFeature::DISABLED)
        visit edit_project_path(project)
      end

      scenario 'shows the Merge Requests settings that do not depend on Builds feature' do
Filipa Lacerda's avatar
Filipa Lacerda committed
39
        expect(page).not_to have_content('Only allow merge requests to be merged if the pipeline succeeds')
40 41
        expect(page).to have_content('Only allow merge requests to be merged if all discussions are resolved')

tauriedavis's avatar
tauriedavis committed
42
        within('.sharing-permissions-form') do
43
          find('.project-feature-controls[data-for="project[project_feature_attributes][builds_access_level]"] .project-feature-toggle').click
tauriedavis's avatar
tauriedavis committed
44 45
          click_on('Save changes')
        end
46

Filipa Lacerda's avatar
Filipa Lacerda committed
47
        expect(page).to have_content('Only allow merge requests to be merged if the pipeline succeeds')
48 49 50 51 52 53 54 55 56 57 58 59
        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' do
    before do
      project.project_feature.update_attribute('merge_requests_access_level', ProjectFeature::DISABLED)
      visit edit_project_path(project)
    end

    scenario 'does not show the Merge Requests settings' do
Filipa Lacerda's avatar
Filipa Lacerda committed
60
      expect(page).not_to have_content('Only allow merge requests to be merged if the pipeline succeeds')
61 62
      expect(page).not_to have_content('Only allow merge requests to be merged if all discussions are resolved')

tauriedavis's avatar
tauriedavis committed
63
      within('.sharing-permissions-form') do
64
        find('.project-feature-controls[data-for="project[project_feature_attributes][merge_requests_access_level]"] .project-feature-toggle').click
tauriedavis's avatar
tauriedavis committed
65 66
        click_on('Save changes')
      end
67

Filipa Lacerda's avatar
Filipa Lacerda committed
68
      expect(page).to have_content('Only allow merge requests to be merged if the pipeline succeeds')
69 70 71
      expect(page).to have_content('Only allow merge requests to be merged if all discussions are resolved')
    end
  end
72 73 74 75 76 77 78 79 80 81 82 83 84

  describe 'Checkbox to enable merge request link' do
    before do
      visit edit_project_path(project)
    end

    scenario 'is initially checked' do
      checkbox = find_field('project_printing_merge_request_link_enabled')
      expect(checkbox).to be_checked
    end

    scenario 'when unchecked sets :printing_merge_request_link_enabled to false' do
      uncheck('project_printing_merge_request_link_enabled')
tauriedavis's avatar
tauriedavis committed
85 86 87
      within('.merge-request-settings-form') do
        click_on('Save changes')
      end
88 89 90 91 92 93 94 95 96

      # 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
97
end