blob: 9f66079c1a95d416c058c10e92029784ddcbc2db (
plain)
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
|
# frozen_string_literal: true
require "spec_helper"
RSpec.describe Discussions::UnresolveService, feature_category: :code_review do
describe "#execute" do
let_it_be(:project) { create(:project, :repository) }
let_it_be(:user) { create(:user, developer_projects: [project]) }
let_it_be(:merge_request) { create(:merge_request, :merge_when_pipeline_succeeds, source_project: project) }
let(:discussion) { create(:diff_note_on_merge_request, noteable: merge_request, project: project).to_discussion }
let(:service) { described_class.new(discussion, user) }
before do
project.add_developer(user)
discussion.resolve!(user)
end
it "unresolves the discussion" do
service.execute
expect(discussion).not_to be_resolved
end
it "counts the unresolve event" do
expect(Gitlab::UsageDataCounters::MergeRequestActivityUniqueCounter)
.to receive(:track_unresolve_thread_action).with(user: user)
service.execute
end
it "sends GraphQL triggers" do
expect(GraphqlTriggers).to receive(:merge_request_merge_status_updated).with(discussion.noteable)
service.execute
end
context "when there are existing unresolved discussions" do
before do
create(:diff_note_on_merge_request, noteable: merge_request, project: project).to_discussion
end
it "does not send a GraphQL triggers" do
expect(GraphqlTriggers).not_to receive(:merge_request_merge_status_updated)
service.execute
end
end
context "when the noteable is not a merge request" do
it "does not send a GraphQL triggers" do
expect(discussion).to receive(:for_merge_request?).and_return(false)
expect(GraphqlTriggers).not_to receive(:merge_request_merge_status_updated)
service.execute
end
end
end
end
|