# frozen_string_literal: true require 'spec_helper' describe MergeRequests::CreateService, :clean_gitlab_redis_shared_state do include ProjectForksHelper let(:project) { create(:project, :repository) } let(:user) { create(:user) } let(:assignee) { create(:user) } describe '#execute' do context 'valid params' do let(:opts) do { title: 'Awesome merge_request', description: 'please fix', source_branch: 'feature', target_branch: 'master', force_remove_source_branch: '1' } end let(:service) { described_class.new(project, user, opts) } let(:merge_request) { service.execute } before do project.add_maintainer(user) project.add_developer(assignee) allow(service).to receive(:execute_hooks) end it 'creates an MR' do expect(merge_request).to be_valid expect(merge_request.work_in_progress?).to be(false) expect(merge_request.title).to eq('Awesome merge_request') expect(merge_request.assignees).to be_empty expect(merge_request.merge_params['force_remove_source_branch']).to eq('1') end it 'executes hooks with default action' do expect(service).to have_received(:execute_hooks).with(merge_request) end it 'refreshes the number of open merge requests', :use_clean_rails_memory_store_caching do expect { service.execute } .to change { project.open_merge_requests_count }.from(0).to(1) end it 'does not creates todos' do attributes = { project: project, target_id: merge_request.id, target_type: merge_request.class.name } expect(Todo.where(attributes).count).to be_zero end it 'creates exactly 1 create MR event', :sidekiq_might_not_need_inline do attributes = { action: Event::CREATED, target_id: merge_request.id, target_type: merge_request.class.name } expect(Event.where(attributes).count).to eq(1) end describe 'when marked with /wip' do context 'in title and in description' do let(:opts) do { title: 'WIP: Awesome merge_request', description: "well this is not done yet\n/wip", source_branch: 'feature', target_branch: 'master', assignees: [assignee] } end it 'sets MR to WIP' do expect(merge_request.work_in_progress?).to be(true) end end context 'in description only' do let(:opts) do { title: 'Awesome merge_request', description: "well this is not done yet\n/wip", source_branch: 'feature', target_branch: 'master', assignees: [assignee] } end it 'sets MR to WIP' do expect(merge_request.work_in_progress?).to be(true) end end end context 'when merge request is assigned to someone' do let(:opts) do { title: 'Awesome merge_request', description: 'please fix', source_branch: 'feature', target_branch: 'master', assignees: [assignee] } end it { expect(merge_request.assignees).to eq([assignee]) } it 'creates a todo for new assignee' do attributes = { project: project, author: user, user: assignee, target_id: merge_request.id, target_type: merge_request.class.name, action: Todo::ASSIGNED, state: :pending } expect(Todo.where(attributes).count).to eq 1 end end context 'when head pipelines already exist for merge request source branch' do let(:shas) { project.repository.commits(opts[:source_branch], limit: 2).map(&:id) } let!(:pipeline_1) { create(:ci_pipeline, project: project, ref: opts[:source_branch], project_id: project.id, sha: shas[1]) } let!(:pipeline_2) { create(:ci_pipeline, project: project, ref: opts[:source_branch], project_id: project.id, sha: shas[0]) } let!(:pipeline_3) { create(:ci_pipeline, project: project, ref: "other_branch", project_id: project.id) } before do # rubocop: disable DestroyAll project.merge_requests .where(source_branch: opts[:source_branch], target_branch: opts[:target_branch]) .destroy_all # rubocop: enable DestroyAll end it 'sets head pipeline' do merge_request = service.execute expect(merge_request.reload.head_pipeline).to eq(pipeline_2) expect(merge_request).to be_persisted end context 'when the new pipeline is associated with an old sha' do let!(:pipeline_1) { create(:ci_pipeline, project: project, ref: opts[:source_branch], project_id: project.id, sha: shas[0]) } let!(:pipeline_2) { create(:ci_pipeline, project: project, ref: opts[:source_branch], project_id: project.id, sha: shas[1]) } it 'sets an old pipeline with associated with the latest sha as the head pipeline' do merge_request = service.execute expect(merge_request.reload.head_pipeline).to eq(pipeline_1) expect(merge_request).to be_persisted end end context 'when there are no pipelines with the diff head sha' do let!(:pipeline_1) { create(:ci_pipeline, project: project, ref: opts[:source_branch], project_id: project.id, sha: shas[1]) } let!(:pipeline_2) { create(:ci_pipeline, project: project, ref: opts[:source_branch], project_id: project.id, sha: shas[1]) } it 'does not set the head pipeline' do merge_request = service.execute expect(merge_request.reload.head_pipeline).to be_nil expect(merge_request).to be_persisted end end end describe 'Pipelines for merge requests' do before do stub_ci_pipeline_yaml_file(YAML.dump(config)) end context "when .gitlab-ci.yml has merge_requests keywords" do let(:config) do { test: { stage: 'test', script: 'echo', only: ['merge_requests'] } } end it 'creates a detached merge request pipeline and sets it as a head pipeline' do expect(merge_request).to be_persisted merge_request.reload expect(merge_request.pipelines_for_merge_request.count).to eq(1) expect(merge_request.actual_head_pipeline).to be_detached_merge_request_pipeline end context 'when merge request is submitted from forked project' do let(:target_project) { fork_project(project, nil, repository: true) } let(:opts) do { title: 'Awesome merge_request', source_branch: 'feature', target_branch: 'master', target_project_id: target_project.id } end before do target_project.add_developer(assignee) target_project.add_maintainer(user) end it 'create legacy detached merge request pipeline for fork merge request', :sidekiq_might_not_need_inline do expect(merge_request.actual_head_pipeline) .to be_legacy_detached_merge_request_pipeline end end context 'when ci_use_merge_request_ref feature flag is false' do before do stub_feature_flags(ci_use_merge_request_ref: false) end it 'create legacy detached merge request pipeline for non-fork merge request' do expect(merge_request.actual_head_pipeline) .to be_legacy_detached_merge_request_pipeline end end context 'when there are no commits between source branch and target branch' do let(:opts) do { title: 'Awesome merge_request', description: 'please fix', source_branch: 'not-merged-branch', target_branch: 'master' } end it 'does not create a detached merge request pipeline' do expect(merge_request).to be_persisted merge_request.reload expect(merge_request.pipelines_for_merge_request.count).to eq(0) end end context "when branch pipeline was created before a merge request pipline has been created" do before do create(:ci_pipeline, project: merge_request.source_project, sha: merge_request.diff_head_sha, ref: merge_request.source_branch, tag: false) merge_request end it 'sets the latest detached merge request pipeline as the head pipeline' do expect(merge_request.actual_head_pipeline).to be_merge_request_event end end end context "when .gitlab-ci.yml does not have merge_requests keywords" do let(:config) do { test: { stage: 'test', script: 'echo' } } end it 'does not create a detached merge request pipeline' do expect(merge_request).to be_persisted merge_request.reload expect(merge_request.pipelines_for_merge_request.count).to eq(0) end end end it 'increments the usage data counter of create event' do counter = Gitlab::UsageDataCounters::MergeRequestCounter expect { service.execute }.to change { counter.read(:create) }.by(1) end end it_behaves_like 'new issuable record that supports quick actions' do let(:default_params) do { source_branch: 'feature', target_branch: 'master' } end end context 'Quick actions' do context 'with assignee and milestone in params and command' do let(:merge_request) { described_class.new(project, user, opts).execute } let(:milestone) { create(:milestone, project: project) } let(:opts) do { assignee_ids: create(:user).id, milestone_id: 1, title: 'Title', description: %(/assign @#{assignee.username}\n/milestone %"#{milestone.name}"), source_branch: 'feature', target_branch: 'master' } end before do project.add_maintainer(user) project.add_maintainer(assignee) end it 'assigns and sets milestone to issuable from command' do expect(merge_request).to be_persisted expect(merge_request.assignees).to eq([assignee]) expect(merge_request.milestone).to eq(milestone) end end end context 'merge request create service' do context 'asssignee_id' do let(:assignee) { create(:user) } before do project.add_maintainer(user) end it 'removes assignee_id when user id is invalid' do opts = { title: 'Title', description: 'Description', assignee_ids: [-1] } merge_request = described_class.new(project, user, opts).execute expect(merge_request.assignee_ids).to be_empty end it 'removes assignee_id when user id is 0' do opts = { title: 'Title', description: 'Description', assignee_ids: [0] } merge_request = described_class.new(project, user, opts).execute expect(merge_request.assignee_ids).to be_empty end it 'saves assignee when user id is valid' do project.add_maintainer(assignee) opts = { title: 'Title', description: 'Description', assignee_ids: [assignee.id] } merge_request = described_class.new(project, user, opts).execute expect(merge_request.assignees).to eq([assignee]) end context 'when assignee is set' do let(:opts) do { title: 'Title', description: 'Description', assignee_ids: [assignee.id], source_branch: 'feature', target_branch: 'master' } end it 'invalidates open merge request counter for assignees when merge request is assigned' do project.add_maintainer(assignee) described_class.new(project, user, opts).execute expect(assignee.assigned_open_merge_requests_count).to eq 1 end end context "when issuable feature is private" do before do project.project_feature.update(issues_access_level: ProjectFeature::PRIVATE, merge_requests_access_level: ProjectFeature::PRIVATE) end levels = [Gitlab::VisibilityLevel::INTERNAL, Gitlab::VisibilityLevel::PUBLIC] levels.each do |level| it "removes not authorized assignee when project is #{Gitlab::VisibilityLevel.level_name(level)}" do project.update(visibility_level: level) opts = { title: 'Title', description: 'Description', assignee_ids: [assignee.id] } merge_request = described_class.new(project, user, opts).execute expect(merge_request.assignee_id).to be_nil end end end end end context 'while saving references to issues that the created merge request closes' do let(:first_issue) { create(:issue, project: project) } let(:second_issue) { create(:issue, project: project) } let(:opts) do { title: 'Awesome merge_request', source_branch: 'feature', target_branch: 'master', force_remove_source_branch: '1' } end before do project.add_maintainer(user) project.add_developer(assignee) end it 'creates a `MergeRequestsClosingIssues` record for each issue' do issue_closing_opts = opts.merge(description: "Closes #{first_issue.to_reference} and #{second_issue.to_reference}") service = described_class.new(project, user, issue_closing_opts) allow(service).to receive(:execute_hooks) merge_request = service.execute issue_ids = MergeRequestsClosingIssues.where(merge_request: merge_request).pluck(:issue_id) expect(issue_ids).to match_array([first_issue.id, second_issue.id]) end end context 'when source and target projects are different' do let(:target_project) { fork_project(project, nil, repository: true) } let(:opts) do { title: 'Awesome merge_request', source_branch: 'feature', target_branch: 'master', target_project_id: target_project.id } end context 'when user can not access source project' do before do target_project.add_developer(assignee) target_project.add_maintainer(user) end it 'raises an error' do expect { described_class.new(project, user, opts).execute } .to raise_error Gitlab::Access::AccessDeniedError end end context 'when user can not access target project' do before do target_project.add_developer(assignee) target_project.add_maintainer(user) end it 'raises an error' do expect { described_class.new(project, user, opts).execute } .to raise_error Gitlab::Access::AccessDeniedError end end context 'when the user has access to both projects' do before do target_project.add_developer(user) project.add_developer(user) end it 'creates the merge request', :sidekiq_might_not_need_inline do merge_request = described_class.new(project, user, opts).execute expect(merge_request).to be_persisted end it 'does not create the merge request when the target project is archived' do target_project.update!(archived: true) expect { described_class.new(project, user, opts).execute } .to raise_error Gitlab::Access::AccessDeniedError end end end context 'when user sets source project id' do let(:another_project) { create(:project) } let(:opts) do { title: 'Awesome merge_request', source_branch: 'feature', target_branch: 'master', source_project_id: another_project.id } end before do project.add_developer(assignee) project.add_maintainer(user) end it 'ignores source_project_id' do merge_request = described_class.new(project, user, opts).execute expect(merge_request.source_project_id).to eq(project.id) end end end end