summaryrefslogtreecommitdiff
path: root/spec/services/discussions/unresolve_service_spec.rb
blob: e9f58e4e10eb13d994526e89e1e1f881616d596e (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_workflow 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