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 PoolRepository, feature_category: :source_code_management do
describe 'associations' do
it { is_expected.to belong_to(:shard) }
it { is_expected.to belong_to(:source_project) }
it { is_expected.to have_many(:member_projects) }
end
describe 'validations' do
let!(:pool_repository) { create(:pool_repository) }
it { is_expected.to validate_presence_of(:shard) }
end
describe 'scopes' do
let_it_be(:project1) { create(:project) }
let_it_be(:project2) { create(:project) }
let_it_be(:new_shard) { create(:shard, name: 'new') }
let_it_be(:pool_repository1) { create(:pool_repository, source_project: project1, disk_path: 'disk_path') }
let_it_be(:pool_repository2) do
create(:pool_repository, source_project: project1, disk_path: 'disk_path', shard: new_shard)
end
let_it_be(:another_pool_repository) { create(:pool_repository, source_project: project2) }
describe '.by_source_project' do
subject { described_class.by_source_project(project1) }
it 'returns pool repositories per source project from all shards' do
is_expected.to match_array([pool_repository1, pool_repository2])
end
end
describe '.by_disk_path_and_shard_name' do
subject { described_class.by_disk_path_and_shard_name('disk_path', new_shard.name) }
it 'returns only a requested pool repository' do
is_expected.to match_array([pool_repository2])
end
end
end
describe '#disk_path' do
it 'sets the hashed disk_path' do
pool = create(:pool_repository)
expect(pool.disk_path).to match(%r{\A@pools/\h{2}/\h{2}/\h{64}})
end
it 'keeps disk_path if already provided' do
pool = create(:pool_repository, disk_path: '@pools/aa/bbbb')
expect(pool.disk_path).to eq('@pools/aa/bbbb')
end
end
describe '#unlink_repository' do
let(:pool) { create(:pool_repository, :ready) }
before do
pool.link_repository(pool.source_project.repository)
end
context 'when the last member leaves' do
it 'schedules pool removal' do
expect(::ObjectPool::DestroyWorker).to receive(:perform_async).with(pool.id).and_call_original
expect(pool.source_project.repository).to receive(:disconnect_alternates).and_call_original
pool.unlink_repository(pool.source_project.repository)
end
end
context 'when skipping disconnect' do
it 'does not change the alternates file' do
expect(pool.source_project.repository).not_to receive(:disconnect_alternates)
pool.unlink_repository(pool.source_project.repository, disconnect: false)
end
end
context 'when the second member leaves' do
it 'does not schedule pool removal' do
other_project = create(:project, :repository, pool_repository: pool)
pool.link_repository(other_project.repository)
expect(::ObjectPool::DestroyWorker).not_to receive(:perform_async).with(pool.id)
expect(pool.source_project.repository).to receive(:disconnect_alternates).and_call_original
pool.unlink_repository(pool.source_project.repository)
end
end
end
describe '#object_pool' do
subject { pool.object_pool }
let(:pool) { build(:pool_repository, :ready, source_project: project, disk_path: disk_path) }
let(:project) { build(:project) }
let(:disk_path) { 'disk_path' }
it 'returns an object pool instance' do
is_expected.to be_a_kind_of(Gitlab::Git::ObjectPool)
is_expected.to have_attributes(
storage: pool.shard.name,
relative_path: "#{pool.disk_path}.git",
source_repository: pool.source_project.repository.raw,
gl_project_path: pool.source_project.full_path
)
end
context 'when source project is missing' do
let(:project) { nil }
it 'returns an object pool instance' do
is_expected.to be_a_kind_of(Gitlab::Git::ObjectPool)
is_expected.to have_attributes(
storage: pool.shard.name,
relative_path: "#{pool.disk_path}.git",
source_repository: nil,
gl_project_path: nil
)
end
end
end
end
|