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
|
require 'spec_helper'
describe Gitlab::Git::Hook do
include TestRepo
describe '.directory' do
it 'does not raise an KeyError' do
expect { described_class.directory }.not_to raise_error
end
end
describe '#trigger' do
let(:tmp_dir) { Dir.mktmpdir }
let(:hook_names) { %w[pre-receive post-receive update] }
let(:repo) { gitlab_git_from_gitaly(test_repo_read_only) }
before do
hook_names.each do |f|
path = File.join(tmp_dir, f)
File.write(path, script)
FileUtils.chmod("u+x", path)
end
allow(Gitlab.config.git).to receive(:hooks_directory).and_return(tmp_dir)
allow(Gitlab.config.gitlab_shell).to receive(:path).and_return('/foobar/gitlab-shell')
end
after do
FileUtils.remove_entry(tmp_dir)
end
context 'when the hooks require environment variables' do
let(:vars) do
{
'GL_ID' => 'user-123',
'GL_USERNAME' => 'janedoe',
'GL_REPOSITORY' => repo.gl_repository,
'GL_PROTOCOL' => 'web',
'PWD' => repo.path,
'GIT_DIR' => repo.path,
'GITALY_GITLAB_SHELL_DIR' => '/foobar/gitlab-shell'
}
end
let(:script) do
[
"#!/bin/sh",
vars.map do |key, value|
<<-SCRIPT
if [ x$#{key} != x#{value} ]; then
echo "unexpected value: #{key}=$#{key}"
exit 1
fi
SCRIPT
end.join,
"exit 0"
].join("\n")
end
it 'returns true' do
hook_names.each do |hook|
trigger_result = described_class.new(hook, repo)
.trigger(vars['GL_ID'], vars['GL_USERNAME'], '0' * 40, 'a' * 40, 'master')
expect(trigger_result.first).to be(true), "#{hook} failed: #{trigger_result.last}"
end
end
end
context 'when the hooks are successful' do
let(:script) { "#!/bin/sh\nexit 0\n" }
it 'returns true' do
hook_names.each do |hook|
trigger_result = described_class.new(hook, repo)
.trigger('user-456', 'admin', '0' * 40, 'a' * 40, 'master')
expect(trigger_result.first).to be(true)
end
end
end
context 'when the hooks fail' do
let(:script) { "#!/bin/sh\nexit 1\n" }
it 'returns false' do
hook_names.each do |hook|
trigger_result = described_class.new(hook, repo)
.trigger('user-1', 'admin', '0' * 40, 'a' * 40, 'master')
expect(trigger_result.first).to be(false)
end
end
end
end
end
|
