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
131
132
133
134
135
136
|
# frozen_string_literal: true
require 'spec_helper'
RSpec.describe Discussion, feature_category: :team_planning do
subject { described_class.new([first_note, second_note, third_note]) }
let(:first_note) { create(:diff_note_on_merge_request) }
let(:merge_request) { first_note.noteable }
let(:second_note) { create(:diff_note_on_merge_request, in_reply_to: first_note) }
let(:third_note) { create(:diff_note_on_merge_request) }
describe '.lazy_find' do
let!(:note1) { create(:discussion_note_on_merge_request).to_discussion }
let!(:note2) { create(:discussion_note_on_merge_request, in_reply_to: note1).to_discussion }
subject { [note1, note2].map { |note| described_class.lazy_find(note.discussion_id) } }
it 'batches requests' do
expect do
[described_class.lazy_find(note1.id),
described_class.lazy_find(note2.id)].map(&:__sync)
end.not_to exceed_query_limit(1)
end
end
describe '.build' do
it 'returns a discussion of the right type' do
discussion = described_class.build([first_note, second_note], merge_request)
expect(discussion).to be_a(DiffDiscussion)
expect(discussion.notes.count).to be(2)
expect(discussion.first_note).to be(first_note)
expect(discussion.noteable).to be(merge_request)
end
end
describe '.build_collection' do
it 'returns an array of discussions of the right type' do
discussions = described_class.build_collection([first_note, second_note, third_note], merge_request)
expect(discussions).to eq(
[
DiffDiscussion.new([first_note, second_note], merge_request),
DiffDiscussion.new([third_note], merge_request)
])
end
end
describe 'authorization' do
it 'delegates to the first note' do
policy = DeclarativePolicy.policy_for(instance_double(User, id: 1), subject)
expect(policy).to be_a(NotePolicy)
end
end
describe '#cache_key' do
let(:notes_sha) { Digest::SHA1.hexdigest("#{first_note.post_processed_cache_key}:#{second_note.post_processed_cache_key}:#{third_note.post_processed_cache_key}") }
it 'returns the cache key' do
expect(subject.cache_key).to eq("#{described_class::CACHE_VERSION}:#{subject.id}:#{notes_sha}:")
end
context 'when discussion is resolved' do
before do
subject.resolve!(first_note.author)
end
it 'returns the cache key with resolved at' do
expect(subject.cache_key).to eq("#{described_class::CACHE_VERSION}:#{subject.id}:#{notes_sha}:#{subject.resolved_at}")
end
end
end
describe '#to_global_id' do
context 'with a single DiffNote discussion' do
it 'returns GID on Discussion class' do
discussion = described_class.build([first_note], merge_request)
discussion_id = discussion.id
expect(discussion.class.name.to_s).to eq("DiffDiscussion")
expect(discussion.to_global_id.to_s).to eq("gid://gitlab/Discussion/#{discussion_id}")
end
end
context 'with multiple DiffNotes discussion' do
it 'returns GID on Discussion class' do
discussion = described_class.build([first_note, second_note], merge_request)
discussion_id = discussion.id
expect(discussion.class.name.to_s).to eq("DiffDiscussion")
expect(discussion.to_global_id.to_s).to eq("gid://gitlab/Discussion/#{discussion_id}")
end
end
context 'with discussions on issue' do
let_it_be(:note_1, refind: true) { create(:note) }
let_it_be(:noteable) { note_1.noteable }
context 'with a single Note' do
it 'returns GID on Discussion class' do
discussion = described_class.build([note_1], noteable)
discussion_id = discussion.id
expect(discussion.class.name.to_s).to eq("IndividualNoteDiscussion")
expect(discussion.to_global_id.to_s).to eq("gid://gitlab/Discussion/#{discussion_id}")
end
end
context 'with multiple Notes' do
let_it_be(:note_1, refind: true) { create(:note, type: 'DiscussionNote') }
let_it_be(:note_2, refind: true) { create(:note, in_reply_to: note_1) }
it 'returns GID on Discussion class' do
discussion = described_class.build([note_1, note_2], noteable)
discussion_id = discussion.id
expect(discussion.class.name.to_s).to eq("Discussion")
expect(discussion.to_global_id.to_s).to eq("gid://gitlab/Discussion/#{discussion_id}")
end
end
end
context 'with system notes' do
let_it_be(:system_note, refind: true) { create(:note, system: true) }
let_it_be(:noteable) { system_note.noteable }
it 'returns GID on Discussion class' do
discussion = described_class.build([system_note], noteable)
discussion_id = discussion.id
expect(discussion.class.name.to_s).to eq("IndividualNoteDiscussion")
expect(discussion.to_global_id.to_s).to eq("gid://gitlab/Discussion/#{discussion_id}")
end
end
end
end
|