1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
|
# frozen_string_literal: true
require 'spec_helper'
RSpec.describe WorkItems::DeleteService, feature_category: :team_planning do
let_it_be(:group) { create(:group) }
let_it_be(:project) { create(:project, :repository, group: group) }
let_it_be(:guest) { create(:user, guest_of: group) }
let_it_be(:owner) { create(:user, owner_of: group) }
let_it_be(:work_item, refind: true) { create(:work_item, project: project, author: guest) }
let(:user) { guest }
let(:service) { described_class.new(container: project, current_user: user) }
before_all do
# note necessary to test note removal as part of work item deletion
create(:note, project: project, noteable: work_item)
end
describe '#execute' do
subject(:result) { service.execute(work_item) }
context 'when user can delete the work item' do
it { is_expected.to be_success }
it 'publish WorkItems::WorkItemDeletedEvent' do
expect { service.execute(work_item) }
.to publish_event(::WorkItems::WorkItemDeletedEvent)
.with({
id: work_item.id,
namespace_id: work_item.namespace_id,
work_item_parent_id: work_item.work_item_parent&.id
}.tap(&:compact_blank!))
end
# currently we don't expect destroy to fail. Mocking here for coverage and keeping
# the service's return type consistent
context 'when there are errors preventing to delete the work item' do
before do
allow(work_item).to receive(:destroy).and_return(false)
work_item.errors.add(:title)
end
it { is_expected.to be_error }
it 'returns error messages' do
expect(result.errors).to contain_exactly('Title is invalid')
end
end
end
context 'when user cannot delete the work item' do
let(:user) { create(:user) }
it { is_expected.to be_error }
it 'returns error messages' do
expect(result.errors).to contain_exactly('User not authorized to delete work item')
end
it 'does not publish WorkItems::WorkItemDeletedEvent' do
expect { service.execute(work_item) }
.not_to publish_event(::WorkItems::WorkItemDeletedEvent)
end
end
end
end
|
