2014-04-02 09:33:07 -04:00
|
|
|
require 'spec_helper'
|
|
|
|
|
2015-12-09 05:55:49 -05:00
|
|
|
describe Issues::CloseService, services: true do
|
2014-04-02 09:33:07 -04:00
|
|
|
let(:user) { create(:user) }
|
|
|
|
let(:user2) { create(:user) }
|
2016-08-09 11:51:40 -04:00
|
|
|
let(:guest) { create(:user) }
|
2014-04-02 09:33:07 -04:00
|
|
|
let(:issue) { create(:issue, assignee: user2) }
|
2015-05-15 09:38:05 -04:00
|
|
|
let(:project) { issue.project }
|
2016-02-20 08:59:59 -05:00
|
|
|
let!(:todo) { create(:todo, :assigned, user: user, project: project, target: issue, author: user2) }
|
2014-04-02 09:33:07 -04:00
|
|
|
|
|
|
|
before do
|
|
|
|
project.team << [user, :master]
|
|
|
|
project.team << [user2, :developer]
|
2016-08-09 11:51:40 -04:00
|
|
|
project.team << [guest, :guest]
|
2014-04-02 09:33:07 -04:00
|
|
|
end
|
|
|
|
|
2016-07-11 18:12:31 -04:00
|
|
|
describe '#execute' do
|
2016-10-07 09:20:57 -04:00
|
|
|
let(:service) { described_class.new(project, user) }
|
|
|
|
|
|
|
|
it 'checks if the user is authorized to update the issue' do
|
|
|
|
expect(service).to receive(:can?).with(user, :update_issue, issue).
|
|
|
|
and_call_original
|
|
|
|
|
|
|
|
service.execute(issue)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not close the issue when the user is not authorized to do so' do
|
|
|
|
allow(service).to receive(:can?).with(user, :update_issue, issue).
|
|
|
|
and_return(false)
|
|
|
|
|
|
|
|
expect(service).not_to receive(:close_issue)
|
|
|
|
expect(service.execute(issue)).to eq(issue)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'closes the issue when the user is authorized to do so' do
|
|
|
|
allow(service).to receive(:can?).with(user, :update_issue, issue).
|
|
|
|
and_return(true)
|
|
|
|
|
|
|
|
expect(service).to receive(:close_issue).
|
|
|
|
with(issue, commit: nil, notifications: true, system_note: true)
|
|
|
|
|
|
|
|
service.execute(issue)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#close_issue' do
|
2014-04-02 09:33:07 -04:00
|
|
|
context "valid params" do
|
|
|
|
before do
|
2015-11-30 11:03:07 -05:00
|
|
|
perform_enqueued_jobs do
|
2016-10-07 09:20:57 -04:00
|
|
|
described_class.new(project, user).close_issue(issue)
|
2015-11-30 11:03:07 -05:00
|
|
|
end
|
2014-04-02 09:33:07 -04:00
|
|
|
end
|
|
|
|
|
2016-08-29 15:08:40 -04:00
|
|
|
it { expect(issue).to be_valid }
|
|
|
|
it { expect(issue).to be_closed }
|
2014-04-02 09:33:07 -04:00
|
|
|
|
2016-08-01 11:00:44 -04:00
|
|
|
it 'sends email to user2 about assign of new issue' do
|
2014-04-02 09:33:07 -04:00
|
|
|
email = ActionMailer::Base.deliveries.last
|
2015-02-12 13:17:35 -05:00
|
|
|
expect(email.to.first).to eq(user2.email)
|
|
|
|
expect(email.subject).to include(issue.title)
|
2014-04-02 09:33:07 -04:00
|
|
|
end
|
|
|
|
|
2016-08-01 11:00:44 -04:00
|
|
|
it 'creates system note about issue reassign' do
|
2016-08-29 15:08:40 -04:00
|
|
|
note = issue.notes.last
|
2016-11-23 01:55:23 -05:00
|
|
|
expect(note.note).to include "closed"
|
2014-04-02 09:33:07 -04:00
|
|
|
end
|
2016-02-16 15:31:40 -05:00
|
|
|
|
2016-02-20 08:59:59 -05:00
|
|
|
it 'marks todos as done' do
|
|
|
|
expect(todo.reload).to be_done
|
2016-02-16 15:31:40 -05:00
|
|
|
end
|
2014-04-02 09:33:07 -04:00
|
|
|
end
|
2015-06-12 01:39:50 -04:00
|
|
|
|
2016-08-30 18:11:46 -04:00
|
|
|
context 'when issue is not confidential' do
|
|
|
|
it 'executes issue hooks' do
|
|
|
|
expect(project).to receive(:execute_hooks).with(an_instance_of(Hash), :issue_hooks)
|
|
|
|
expect(project).to receive(:execute_services).with(an_instance_of(Hash), :issue_hooks)
|
|
|
|
|
2016-10-07 09:20:57 -04:00
|
|
|
described_class.new(project, user).close_issue(issue)
|
2016-08-30 18:11:46 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-08-12 15:05:10 -04:00
|
|
|
context 'when issue is confidential' do
|
2016-08-30 18:11:46 -04:00
|
|
|
it 'executes confidential issue hooks' do
|
2016-08-12 15:05:10 -04:00
|
|
|
issue = create(:issue, :confidential, project: project)
|
|
|
|
|
2016-08-30 18:11:46 -04:00
|
|
|
expect(project).to receive(:execute_hooks).with(an_instance_of(Hash), :confidential_issue_hooks)
|
|
|
|
expect(project).to receive(:execute_services).with(an_instance_of(Hash), :confidential_issue_hooks)
|
2016-08-12 15:05:10 -04:00
|
|
|
|
2016-10-07 09:20:57 -04:00
|
|
|
described_class.new(project, user).close_issue(issue)
|
2016-08-12 15:05:10 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'external issue tracker' do
|
2015-06-12 01:39:50 -04:00
|
|
|
before do
|
|
|
|
allow(project).to receive(:default_issues_tracker?).and_return(false)
|
2016-10-07 09:20:57 -04:00
|
|
|
described_class.new(project, user).close_issue(issue)
|
2015-06-12 01:39:50 -04:00
|
|
|
end
|
|
|
|
|
2016-08-29 15:08:40 -04:00
|
|
|
it { expect(issue).to be_valid }
|
|
|
|
it { expect(issue).to be_opened }
|
2016-02-20 08:59:59 -05:00
|
|
|
it { expect(todo.reload).to be_pending }
|
2015-06-12 01:39:50 -04:00
|
|
|
end
|
2014-04-02 09:33:07 -04:00
|
|
|
end
|
|
|
|
end
|