2012-10-09 04:14:17 -04:00
|
|
|
# == Schema Information
|
|
|
|
#
|
|
|
|
# Table name: merge_requests
|
|
|
|
#
|
2013-08-21 05:34:02 -04:00
|
|
|
# id :integer not null, primary key
|
|
|
|
# target_branch :string(255) not null
|
|
|
|
# source_branch :string(255) not null
|
|
|
|
# source_project_id :integer not null
|
|
|
|
# author_id :integer
|
|
|
|
# assignee_id :integer
|
|
|
|
# title :string(255)
|
|
|
|
# created_at :datetime not null
|
|
|
|
# updated_at :datetime not null
|
|
|
|
# st_commits :text(2147483647)
|
|
|
|
# st_diffs :text(2147483647)
|
|
|
|
# milestone_id :integer
|
|
|
|
# state :string(255)
|
|
|
|
# merge_status :string(255)
|
|
|
|
# target_project_id :integer not null
|
|
|
|
# iid :integer
|
2013-10-01 08:15:28 -04:00
|
|
|
# description :text
|
2012-10-09 04:14:17 -04:00
|
|
|
#
|
|
|
|
|
2011-11-28 02:39:43 -05:00
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe MergeRequest do
|
2011-11-28 16:24:08 -05:00
|
|
|
describe "Validation" do
|
|
|
|
it { should validate_presence_of(:target_branch) }
|
|
|
|
it { should validate_presence_of(:source_branch) }
|
|
|
|
end
|
|
|
|
|
2012-09-26 14:17:17 -04:00
|
|
|
describe "Mass assignment" do
|
|
|
|
it { should_not allow_mass_assignment_of(:author_id) }
|
|
|
|
it { should_not allow_mass_assignment_of(:project_id) }
|
|
|
|
end
|
|
|
|
|
2013-02-20 08:15:01 -05:00
|
|
|
describe "Respond to" do
|
|
|
|
it { should respond_to(:unchecked?) }
|
|
|
|
it { should respond_to(:can_be_merged?) }
|
|
|
|
it { should respond_to(:cannot_be_merged?) }
|
|
|
|
end
|
2013-03-20 17:46:30 -04:00
|
|
|
|
2012-08-29 01:52:19 -04:00
|
|
|
describe 'modules' do
|
2013-01-03 02:37:13 -05:00
|
|
|
it { should include_module(Issuable) }
|
2012-08-09 13:45:12 -04:00
|
|
|
end
|
2012-10-04 18:25:40 -04:00
|
|
|
|
|
|
|
describe "#mr_and_commit_notes" do
|
2012-11-05 22:31:55 -05:00
|
|
|
let!(:merge_request) { create(:merge_request) }
|
2012-10-04 18:25:40 -04:00
|
|
|
|
|
|
|
before do
|
2013-04-25 10:15:33 -04:00
|
|
|
merge_request.stub(:commits) { [merge_request.source_project.repository.commit] }
|
2013-10-07 12:42:59 -04:00
|
|
|
create(:note, commit_id: merge_request.commits.first.id, noteable_type: 'Commit', project: merge_request.project)
|
|
|
|
create(:note, noteable: merge_request, project: merge_request.project)
|
2012-10-04 18:25:40 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
it "should include notes for commits" do
|
|
|
|
merge_request.commits.should_not be_empty
|
|
|
|
merge_request.mr_and_commit_notes.count.should == 2
|
|
|
|
end
|
|
|
|
end
|
2012-10-09 18:25:29 -04:00
|
|
|
|
2012-11-05 22:31:55 -05:00
|
|
|
subject { create(:merge_request) }
|
2012-10-09 18:25:29 -04:00
|
|
|
|
|
|
|
describe '#is_being_reassigned?' do
|
|
|
|
it 'returns true if the merge_request assignee has changed' do
|
2012-11-05 22:31:55 -05:00
|
|
|
subject.assignee = create(:user)
|
2012-10-09 18:25:29 -04:00
|
|
|
subject.is_being_reassigned?.should be_true
|
|
|
|
end
|
|
|
|
it 'returns false if the merge request assignee has not changed' do
|
|
|
|
subject.is_being_reassigned?.should be_false
|
|
|
|
end
|
|
|
|
end
|
2013-04-25 10:15:33 -04:00
|
|
|
|
|
|
|
describe '#for_fork?' do
|
|
|
|
it 'returns true if the merge request is for a fork' do
|
2014-01-23 06:24:25 -05:00
|
|
|
subject.source_project = create(:project, namespace: create(:group))
|
|
|
|
subject.target_project = create(:project, namespace: create(:group))
|
2013-04-25 10:15:33 -04:00
|
|
|
|
|
|
|
subject.for_fork?.should be_true
|
|
|
|
end
|
2014-01-23 06:24:25 -05:00
|
|
|
|
2013-04-25 10:15:33 -04:00
|
|
|
it 'returns false if is not for a fork' do
|
|
|
|
subject.for_fork?.should be_false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#allow_source_branch_removal?' do
|
|
|
|
it 'should not allow removal when mr is a fork' do
|
|
|
|
|
|
|
|
subject.disallow_source_branch_removal?.should be_true
|
|
|
|
end
|
|
|
|
it 'should not allow removal when the mr is not a fork, but the source branch is the root reference' do
|
|
|
|
subject.target_project = subject.source_project
|
|
|
|
subject.source_branch = subject.source_project.repository.root_ref
|
|
|
|
subject.disallow_source_branch_removal?.should be_true
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'should not disallow removal when the mr is not a fork, and but source branch is not the root reference' do
|
|
|
|
subject.target_project = subject.source_project
|
|
|
|
subject.source_branch = "Something Different #{subject.source_project.repository.root_ref}"
|
|
|
|
subject.for_fork?.should be_false
|
|
|
|
subject.disallow_source_branch_removal?.should be_false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2013-05-30 19:16:49 -04:00
|
|
|
describe 'detection of issues to be closed' do
|
|
|
|
let(:issue0) { create :issue, project: subject.project }
|
|
|
|
let(:issue1) { create :issue, project: subject.project }
|
2013-12-14 08:43:48 -05:00
|
|
|
let(:commit0) { double('commit0', closes_issues: [issue0]) }
|
|
|
|
let(:commit1) { double('commit1', closes_issues: [issue0]) }
|
|
|
|
let(:commit2) { double('commit2', closes_issues: [issue1]) }
|
2013-05-30 19:16:49 -04:00
|
|
|
|
|
|
|
before do
|
2013-12-12 07:20:08 -05:00
|
|
|
subject.stub(commits: [commit0, commit1, commit2])
|
2013-05-30 19:16:49 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'accesses the set of issues that will be closed on acceptance' do
|
2013-11-06 12:13:45 -05:00
|
|
|
subject.project.stub(default_branch: subject.target_branch)
|
2013-05-30 19:16:49 -04:00
|
|
|
|
|
|
|
subject.closes_issues.should == [issue0, issue1].sort_by(&:id)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'only lists issues as to be closed if it targets the default branch' do
|
2013-11-06 12:13:45 -05:00
|
|
|
subject.project.stub(default_branch: 'master')
|
2013-05-30 19:16:49 -04:00
|
|
|
subject.target_branch = 'something-else'
|
|
|
|
|
|
|
|
subject.closes_issues.should be_empty
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'an editable mentionable' do
|
|
|
|
let(:subject) { create :merge_request, source_project: mproject, target_project: mproject }
|
|
|
|
let(:backref_text) { "merge request !#{subject.iid}" }
|
|
|
|
let(:set_mentionable_text) { ->(txt){ subject.title = txt } }
|
|
|
|
end
|
2011-11-28 02:39:43 -05:00
|
|
|
end
|