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
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
|
# frozen_string_literal: true
require 'spec_helper'
RSpec.describe ResourceStateEvent, feature_category: :team_planning, type: :model do
subject { build(:resource_state_event, issue: issue) }
let(:issue) { create(:issue) }
let(:merge_request) { create(:merge_request) }
it_behaves_like 'a resource event'
it_behaves_like 'a resource event that responds to imported'
it_behaves_like 'a resource event for issues'
it_behaves_like 'a resource event for merge requests'
it_behaves_like 'a note for work item resource event'
describe 'validations' do
describe 'Issuable validation' do
it 'is valid if an issue is set' do
subject.attributes = { issue: build_stubbed(:issue), merge_request: nil }
expect(subject).to be_valid
end
it 'is valid if a merge request is set' do
subject.attributes = { issue: nil, merge_request: build_stubbed(:merge_request) }
expect(subject).to be_valid
end
it 'is invalid if both issue and merge request are set' do
subject.attributes = { issue: build_stubbed(:issue), merge_request: build_stubbed(:merge_request) }
expect(subject).not_to be_valid
end
it 'is invalid if there is no issuable set' do
subject.attributes = { issue: nil, merge_request: nil }
expect(subject).not_to be_valid
end
end
end
context 'callbacks' do
describe '#issue_usage_metrics' do
describe 'when an issue is closed' do
subject(:close_issue) do
create(
described_class.name.underscore.to_sym,
issue: issue,
state: described_class.states[:closed]
)
end
it 'tracks closed issues' do
expect(Gitlab::UsageDataCounters::IssueActivityUniqueCounter).to receive(:track_issue_closed_action)
close_issue
end
it_behaves_like 'internal event tracking' do
subject(:service_action) { close_issue }
let(:event) { Gitlab::UsageDataCounters::IssueActivityUniqueCounter::ISSUE_CLOSED }
let(:project) { issue.project }
let(:user) { issue.author }
let(:issue) do
# The g_project_management_issue_created event is triggered by creating the issue.
# So we'll trigger the irrelevant event outside of the metric time ranges
travel_to(2.months.ago) { create(:issue) }
end
end
end
describe 'when an issue is reopened' do
subject(:reopen_issue) do
create(
described_class.name.underscore.to_sym,
issue: issue,
state: described_class.states[:reopened]
)
end
it 'tracks reopened issues' do
expect(Gitlab::UsageDataCounters::IssueActivityUniqueCounter).to receive(:track_issue_reopened_action)
reopen_issue
end
it_behaves_like 'internal event tracking' do
subject(:service_action) { reopen_issue }
let(:event) { Gitlab::UsageDataCounters::IssueActivityUniqueCounter::ISSUE_REOPENED }
let(:project) { issue.project }
let(:user) { issue.author }
let(:issue) do
# The g_project_management_issue_created event is triggered by creating the issue.
# So we'll trigger the irrelevant event outside of the metric time ranges
travel_to(2.months.ago) { create(:issue) }
end
end
end
it 'does not track merge requests' do
expect(Gitlab::UsageDataCounters::IssueActivityUniqueCounter).not_to receive(:track_issue_closed_action)
create(described_class.name.underscore.to_sym, merge_request: merge_request, state: described_class.states[:closed])
end
end
end
describe '.merged_with_no_event_source', feature_category: :code_review_workflow do
let!(:merged_event) { create(:resource_state_event, merge_request: merge_request, state: :merged) }
before do
create(:resource_state_event, merge_request: merge_request, state: :closed)
create(:resource_state_event, merge_request: merge_request, state: :merged, source_merge_request: merge_request)
create(:resource_state_event, merge_request: merge_request, state: :merged, source_commit: 'abcd1234')
end
subject { described_class.merged_with_no_event_source }
it 'returns expected events' do
expect(subject).to contain_exactly(merged_event)
end
end
end
|
