2011-10-08 21:36:38 +00:00
|
|
|
require 'spec_helper'
|
|
|
|
|
2015-12-09 09:50:51 +00:00
|
|
|
describe Issue, models: true do
|
2011-10-08 21:36:38 +00:00
|
|
|
describe "Associations" do
|
2015-02-12 18:17:35 +00:00
|
|
|
it { is_expected.to belong_to(:milestone) }
|
2011-10-08 21:36:38 +00:00
|
|
|
end
|
|
|
|
|
2012-08-29 05:52:19 +00:00
|
|
|
describe 'modules' do
|
2015-05-03 03:11:21 +00:00
|
|
|
subject { described_class }
|
|
|
|
|
|
|
|
it { is_expected.to include_module(InternalId) }
|
2015-02-12 18:17:35 +00:00
|
|
|
it { is_expected.to include_module(Issuable) }
|
2015-05-03 03:11:21 +00:00
|
|
|
it { is_expected.to include_module(Referable) }
|
|
|
|
it { is_expected.to include_module(Sortable) }
|
|
|
|
it { is_expected.to include_module(Taskable) }
|
2012-08-29 05:52:19 +00:00
|
|
|
end
|
|
|
|
|
2012-11-06 03:31:55 +00:00
|
|
|
subject { create(:issue) }
|
2012-05-20 18:15:13 +00:00
|
|
|
|
2016-02-26 08:55:43 +00:00
|
|
|
describe "act_as_paranoid" do
|
|
|
|
it { is_expected.to have_db_column(:deleted_at) }
|
|
|
|
it { is_expected.to have_db_index(:deleted_at) }
|
|
|
|
end
|
|
|
|
|
2015-05-03 03:11:21 +00:00
|
|
|
describe '#to_reference' do
|
|
|
|
it 'returns a String reference to the object' do
|
|
|
|
expect(subject.to_reference).to eq "##{subject.iid}"
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'supports a cross-project reference' do
|
|
|
|
cross = double('project')
|
|
|
|
expect(subject.to_reference(cross)).
|
|
|
|
to eq "#{subject.project.to_reference}##{subject.iid}"
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2012-05-20 18:15:13 +00:00
|
|
|
describe '#is_being_reassigned?' do
|
|
|
|
it 'returns true if the issue assignee has changed' do
|
2012-11-06 03:31:55 +00:00
|
|
|
subject.assignee = create(:user)
|
2015-02-12 18:17:35 +00:00
|
|
|
expect(subject.is_being_reassigned?).to be_truthy
|
2012-05-20 18:15:13 +00:00
|
|
|
end
|
|
|
|
it 'returns false if the issue assignee has not changed' do
|
2015-02-12 18:17:35 +00:00
|
|
|
expect(subject.is_being_reassigned?).to be_falsey
|
2012-05-20 18:15:13 +00:00
|
|
|
end
|
|
|
|
end
|
2013-02-18 09:41:32 +00:00
|
|
|
|
|
|
|
describe '#is_being_reassigned?' do
|
2013-07-29 10:46:00 +00:00
|
|
|
it 'returns issues assigned to user' do
|
2015-05-03 03:14:31 +00:00
|
|
|
user = create(:user)
|
|
|
|
create_list(:issue, 2, assignee: user)
|
2013-02-18 09:41:32 +00:00
|
|
|
|
2015-02-12 18:17:35 +00:00
|
|
|
expect(Issue.open_for(user).count).to eq 2
|
2013-02-18 09:41:32 +00:00
|
|
|
end
|
|
|
|
end
|
2013-05-30 23:16:49 +00:00
|
|
|
|
2015-10-13 07:41:46 +00:00
|
|
|
describe '#closed_by_merge_requests' do
|
|
|
|
let(:project) { create(:project) }
|
|
|
|
let(:issue) { create(:issue, project: project, state: "opened")}
|
|
|
|
let(:closed_issue) { build(:issue, project: project, state: "closed")}
|
|
|
|
|
|
|
|
let(:mr) do
|
|
|
|
opts = {
|
|
|
|
title: 'Awesome merge_request',
|
|
|
|
description: "Fixes #{issue.to_reference}",
|
|
|
|
source_branch: 'feature',
|
|
|
|
target_branch: 'master'
|
|
|
|
}
|
|
|
|
MergeRequests::CreateService.new(project, project.owner, opts).execute
|
|
|
|
end
|
|
|
|
|
|
|
|
let(:closed_mr) do
|
|
|
|
opts = {
|
|
|
|
title: 'Awesome merge_request 2',
|
|
|
|
description: "Fixes #{issue.to_reference}",
|
|
|
|
source_branch: 'feature',
|
|
|
|
target_branch: 'master',
|
|
|
|
state: 'closed'
|
|
|
|
}
|
|
|
|
MergeRequests::CreateService.new(project, project.owner, opts).execute
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns the merge request to close this issue' do
|
|
|
|
allow(mr).to receive(:closes_issue?).with(issue).and_return(true)
|
|
|
|
|
|
|
|
expect(issue.closed_by_merge_requests).to eq([mr])
|
|
|
|
end
|
|
|
|
|
|
|
|
it "returns an empty array when the current issue is closed already" do
|
|
|
|
expect(closed_issue.closed_by_merge_requests).to eq([])
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-03-08 14:56:19 +00:00
|
|
|
describe '#referenced_merge_requests' do
|
|
|
|
it 'returns the referenced merge requests' do
|
|
|
|
project = create(:project, :public)
|
|
|
|
|
|
|
|
mr1 = create(:merge_request,
|
|
|
|
source_project: project,
|
|
|
|
source_branch: 'master',
|
|
|
|
target_branch: 'feature')
|
|
|
|
|
|
|
|
mr2 = create(:merge_request,
|
|
|
|
source_project: project,
|
|
|
|
source_branch: 'feature',
|
|
|
|
target_branch: 'master')
|
|
|
|
|
|
|
|
issue = create(:issue, description: mr1.to_reference, project: project)
|
|
|
|
|
|
|
|
create(:note_on_issue,
|
|
|
|
noteable: issue,
|
|
|
|
note: mr2.to_reference,
|
|
|
|
project_id: project.id)
|
|
|
|
|
|
|
|
expect(issue.referenced_merge_requests).to eq([mr1, mr2])
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-03-17 10:11:22 +00:00
|
|
|
describe '#can_move?' do
|
|
|
|
let(:user) { create(:user) }
|
|
|
|
let(:issue) { create(:issue) }
|
|
|
|
subject { issue.can_move?(user) }
|
|
|
|
|
|
|
|
context 'user is not a member of project issue belongs to' do
|
|
|
|
it { is_expected.to eq false}
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'user is reporter in project issue belongs to' do
|
|
|
|
let(:project) { create(:project) }
|
|
|
|
let(:issue) { create(:issue, project: project) }
|
|
|
|
|
|
|
|
before { project.team << [user, :reporter] }
|
|
|
|
|
|
|
|
it { is_expected.to eq true }
|
|
|
|
|
2016-03-23 08:39:37 +00:00
|
|
|
context 'issue not persisted' do
|
|
|
|
let(:issue) { build(:issue, project: project) }
|
|
|
|
it { is_expected.to eq false }
|
|
|
|
end
|
|
|
|
|
2016-03-17 10:11:22 +00:00
|
|
|
context 'checking destination project also' do
|
|
|
|
subject { issue.can_move?(user, to_project) }
|
|
|
|
let(:to_project) { create(:project) }
|
|
|
|
|
|
|
|
context 'destination project allowed' do
|
|
|
|
before { to_project.team << [user, :reporter] }
|
|
|
|
it { is_expected.to eq true }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'destination project not allowed' do
|
|
|
|
before { to_project.team << [user, :guest] }
|
|
|
|
it { is_expected.to eq false }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#moved?' do
|
|
|
|
let(:issue) { create(:issue) }
|
|
|
|
subject { issue.moved? }
|
|
|
|
|
|
|
|
context 'issue not moved' do
|
|
|
|
it { is_expected.to eq false }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'issue already moved' do
|
|
|
|
let(:moved_to_issue) { create(:issue) }
|
|
|
|
let(:issue) { create(:issue, moved_to: moved_to_issue) }
|
|
|
|
|
|
|
|
it { is_expected.to eq true }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-02-22 08:20:04 +00:00
|
|
|
describe '#related_branches' do
|
2016-04-15 04:01:26 +00:00
|
|
|
let(:user) { build(:admin) }
|
2016-04-12 10:20:19 +00:00
|
|
|
|
2016-04-15 04:01:26 +00:00
|
|
|
before do
|
2016-02-22 08:20:04 +00:00
|
|
|
allow(subject.project.repository).to receive(:branch_names).
|
2016-04-15 04:20:53 +00:00
|
|
|
and_return(["mpempe", "#{subject.iid}mepmep", subject.to_branch_name, "#{subject.iid}-branch"])
|
2016-04-12 10:20:19 +00:00
|
|
|
|
|
|
|
# Without this stub, the `create(:merge_request)` above fails because it can't find
|
|
|
|
# the source branch. This seems like a reasonable compromise, in comparison with
|
|
|
|
# setting up a full repo here.
|
|
|
|
allow_any_instance_of(MergeRequest).to receive(:create_merge_request_diff)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "selects the right branches when there are no referenced merge requests" do
|
2016-04-15 04:20:53 +00:00
|
|
|
expect(subject.related_branches(user)).to eq([subject.to_branch_name, "#{subject.iid}-branch"])
|
2016-04-12 10:20:19 +00:00
|
|
|
end
|
2016-02-22 08:20:04 +00:00
|
|
|
|
2016-04-12 10:20:19 +00:00
|
|
|
it "selects the right branches when there is a referenced merge request" do
|
2016-04-13 03:34:08 +00:00
|
|
|
merge_request = create(:merge_request, { description: "Closes ##{subject.iid}",
|
|
|
|
source_project: subject.project,
|
2016-04-15 04:20:53 +00:00
|
|
|
source_branch: "#{subject.iid}-branch" })
|
2016-04-12 10:20:19 +00:00
|
|
|
merge_request.create_cross_references!(user)
|
2016-05-23 23:37:59 +00:00
|
|
|
expect(subject.referenced_merge_requests).not_to be_empty
|
2016-04-12 06:13:15 +00:00
|
|
|
expect(subject.related_branches(user)).to eq([subject.to_branch_name])
|
2016-02-22 08:20:04 +00:00
|
|
|
end
|
2016-04-13 19:20:03 +00:00
|
|
|
|
|
|
|
it 'excludes stable branches from the related branches' do
|
|
|
|
allow(subject.project.repository).to receive(:branch_names).
|
|
|
|
and_return(["#{subject.iid}-0-stable"])
|
|
|
|
|
2016-04-15 04:20:53 +00:00
|
|
|
expect(subject.related_branches(user)).to eq []
|
2016-04-13 19:20:03 +00:00
|
|
|
end
|
2016-02-22 08:20:04 +00:00
|
|
|
end
|
|
|
|
|
2013-05-30 23:16:49 +00:00
|
|
|
it_behaves_like 'an editable mentionable' do
|
2015-10-12 14:23:15 +00:00
|
|
|
subject { create(:issue) }
|
2015-04-16 20:25:25 +00:00
|
|
|
|
2015-06-18 00:40:26 +00:00
|
|
|
let(:backref_text) { "issue #{subject.to_reference}" }
|
2013-05-30 23:16:49 +00:00
|
|
|
let(:set_mentionable_text) { ->(txt){ subject.description = txt } }
|
|
|
|
end
|
2014-10-06 02:17:28 +00:00
|
|
|
|
|
|
|
it_behaves_like 'a Taskable' do
|
|
|
|
let(:subject) { create :issue }
|
|
|
|
end
|
2016-02-12 18:42:25 +00:00
|
|
|
|
|
|
|
describe "#to_branch_name" do
|
2016-04-12 10:31:44 +00:00
|
|
|
let(:issue) { create(:issue, title: 'testing-issue') }
|
2016-02-12 18:42:25 +00:00
|
|
|
|
2016-04-13 19:20:03 +00:00
|
|
|
it 'starts with the issue iid' do
|
2016-04-15 04:20:53 +00:00
|
|
|
expect(issue.to_branch_name).to match /\A#{issue.iid}-[A-Za-z\-]+\z/
|
2016-02-12 18:42:25 +00:00
|
|
|
end
|
2016-04-12 10:31:44 +00:00
|
|
|
|
|
|
|
it "contains the issue title if not confidential" do
|
2016-04-15 04:20:53 +00:00
|
|
|
expect(issue.to_branch_name).to match /testing-issue\z/
|
2016-04-12 10:31:44 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
it "does not contain the issue title if confidential" do
|
|
|
|
issue = create(:issue, title: 'testing-issue', confidential: true)
|
2016-04-19 03:52:55 +00:00
|
|
|
expect(issue.to_branch_name).to match /confidential-issue\z/
|
2016-04-12 10:31:44 +00:00
|
|
|
end
|
2016-02-12 18:42:25 +00:00
|
|
|
end
|
2016-05-26 11:38:28 +00:00
|
|
|
|
|
|
|
describe '#participants' do
|
|
|
|
context 'using a public project' do
|
|
|
|
let(:project) { create(:project, :public) }
|
|
|
|
let(:issue) { create(:issue, project: project) }
|
|
|
|
|
|
|
|
let!(:note1) do
|
|
|
|
create(:note_on_issue, noteable: issue, project: project, note: 'a')
|
|
|
|
end
|
|
|
|
|
|
|
|
let!(:note2) do
|
|
|
|
create(:note_on_issue, noteable: issue, project: project, note: 'b')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'includes the issue author' do
|
|
|
|
expect(issue.participants).to include(issue.author)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'includes the authors of the notes' do
|
|
|
|
expect(issue.participants).to include(note1.author, note2.author)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'using a private project' do
|
|
|
|
it 'does not include mentioned users that do not have access to the project' do
|
|
|
|
project = create(:project)
|
|
|
|
user = create(:user)
|
|
|
|
issue = create(:issue, project: project)
|
|
|
|
|
|
|
|
create(:note_on_issue,
|
|
|
|
noteable: issue,
|
|
|
|
project: project,
|
|
|
|
note: user.to_reference)
|
|
|
|
|
|
|
|
expect(issue.participants).not_to include(user)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2016-06-02 13:25:40 +00:00
|
|
|
|
|
|
|
describe 'cached counts' do
|
|
|
|
it 'updates when assignees change' do
|
|
|
|
user1 = create(:user)
|
|
|
|
user2 = create(:user)
|
|
|
|
issue = create(:issue, assignee: user1)
|
|
|
|
|
|
|
|
expect(user1.assigned_open_issues_count).to eq(1)
|
|
|
|
expect(user2.assigned_open_issues_count).to eq(0)
|
|
|
|
|
|
|
|
issue.assignee = user2
|
|
|
|
issue.save
|
|
|
|
|
|
|
|
expect(user1.assigned_open_issues_count).to eq(0)
|
|
|
|
expect(user2.assigned_open_issues_count).to eq(1)
|
|
|
|
end
|
|
|
|
end
|
2011-10-08 21:36:38 +00:00
|
|
|
end
|