diff options
Diffstat (limited to 'spec')
| -rw-r--r-- | spec/gitlab_access_spec.rb | 77 | ||||
| -rw-r--r-- | spec/gitlab_custom_hook_spec.rb | 295 | ||||
| -rw-r--r-- | spec/gitlab_post_receive_spec.rb | 284 |
3 files changed, 0 insertions, 656 deletions
diff --git a/spec/gitlab_access_spec.rb b/spec/gitlab_access_spec.rb deleted file mode 100644 index 6c1d348..0000000 --- a/spec/gitlab_access_spec.rb +++ /dev/null @@ -1,77 +0,0 @@ -require 'spec_helper' -require 'gitlab_access' - -describe GitlabAccess do - let(:repository_path) { "/home/git/repositories" } - let(:repo_name) { 'dzaporozhets/gitlab-ci' } - let(:repo_path) { File.join(repository_path, repo_name) + ".git" } - let(:api) do - double(GitlabNet).tap do |api| - allow(api).to receive(:check_access).and_return(GitAccessStatus.new(true, - '200', - 'ok', - gl_repository: 'project-1', - gl_project_path: 'group/subgroup/project', - gl_id: 'user-123', - gl_username: 'testuser', - git_config_options: ['receive.MaxInputSize=10000'], - gitaly: nil, - git_protocol: 'version=2')) - end - end - subject do - GitlabAccess.new(nil, repo_path, 'key-123', 'wow', 'ssh').tap do |access| - allow(access).to receive(:exec_cmd).and_return(:exec_called) - allow(access).to receive(:api).and_return(api) - end - end - - before do - allow_any_instance_of(GitlabConfig).to receive(:repos_path).and_return(repository_path) - end - - describe :initialize do - it { expect(subject.repo_path).to eq(repo_path) } - it { expect(subject.changes).to eq(['wow']) } - it { expect(subject.protocol).to eq('ssh') } - end - - describe "#exec" do - context "access is granted" do - it "returns true" do - expect(subject.exec).to be_truthy - end - end - - context "access is denied" do - before do - allow(api).to receive(:check_access).and_return(GitAccessStatus.new( - false, - '401', - 'denied', - gl_repository: nil, - gl_project_path: nil, - gl_id: nil, - gl_username: nil, - git_config_options: nil, - gitaly: nil, - git_protocol: nil - )) - end - - it "returns false" do - expect(subject.exec).to be_falsey - end - end - - context "API connection fails" do - before do - allow(api).to receive(:check_access).and_raise(GitlabNet::ApiUnreachableError) - end - - it "returns false" do - expect(subject.exec).to be_falsey - end - end - end -end diff --git a/spec/gitlab_custom_hook_spec.rb b/spec/gitlab_custom_hook_spec.rb deleted file mode 100644 index 540cd2b..0000000 --- a/spec/gitlab_custom_hook_spec.rb +++ /dev/null @@ -1,295 +0,0 @@ -# coding: utf-8 -require 'spec_helper' -require 'gitlab_custom_hook' - -describe GitlabCustomHook do - let(:original_root_path) { ROOT_PATH } - let(:tmp_repo_path) { File.join(original_root_path, 'tmp', 'repo.git') } - let(:tmp_root_path) { File.join(original_root_path, 'tmp') } - let(:global_custom_hooks_path) { global_hook_path('custom_global_hooks') } - let(:hook_ok) { File.join(original_root_path, 'spec', 'support', 'hook_ok') } - let(:hook_fail) { File.join(original_root_path, 'spec', 'support', 'hook_fail') } - let(:hook_gl_id) { File.join(original_root_path, 'spec', 'support', 'gl_id_test_hook') } - - let(:vars) { { "GL_ID" => "key_1" } } - let(:old_value) { "old-value" } - let(:new_value) { "new-value" } - let(:ref_name) { "name/of/ref" } - let(:changes) { "#{old_value} #{new_value} #{ref_name}\n" } - - let(:gitlab_custom_hook) { GitlabCustomHook.new(tmp_repo_path, 'key_1') } - - def hook_path(path) - File.join(tmp_repo_path, path.split('/')) - end - - def global_hook_path(path) - File.join(tmp_root_path, path.split('/')) - end - - def create_hook(path, which) - FileUtils.ln_sf(which, hook_path(path)) - end - - # global hooks multiplexed - def create_global_hooks_d(which, hook_name = 'hook') - create_hook('hooks/pre-receive.d/' + hook_name, which) - create_hook('hooks/update.d/' + hook_name, which) - create_hook('hooks/post-receive.d/' + hook_name, which) - end - - # repo hooks - def create_repo_hooks(which) - create_hook('custom_hooks/pre-receive', which) - create_hook('custom_hooks/update', which) - create_hook('custom_hooks/post-receive', which) - end - - # repo hooks multiplexed - def create_repo_hooks_d(which, hook_name = 'hook') - create_hook('custom_hooks/pre-receive.d/' + hook_name, which) - create_hook('custom_hooks/update.d/' + hook_name, which) - create_hook('custom_hooks/post-receive.d/' + hook_name, which) - end - - def cleanup_hook_setup - FileUtils.rm_rf(File.join(tmp_repo_path)) - FileUtils.rm_rf(File.join(global_custom_hooks_path)) - FileUtils.rm_rf(File.join(tmp_root_path, 'hooks')) - FileUtils.rm_f(File.join(tmp_root_path, 'config.yml')) - end - - def expect_call_receive_hook(path) - expect(gitlab_custom_hook) - .to receive(:call_receive_hook) - .with(hook_path(path), changes) - .and_call_original - end - - def expect_call_update_hook(path) - expect(gitlab_custom_hook) - .to receive(:system) - .with(vars, hook_path(path), ref_name, old_value, new_value) - .and_call_original - end - - # setup paths - # <repository>.git/hooks/ - symlink to gitlab-shell/hooks global dir - # <repository>.git/hooks/<hook_name> - executed by git itself, this is gitlab-shell/hooks/<hook_name> - # <repository>.git/hooks/<hook_name>.d/* - global hooks: all executable files (minus editor backup files) - # <repository>.git/custom_hooks/<hook_name> - per project hook (this is already existing behavior) - # <repository>.git/custom_hooks/<hook_name>.d/* - per project hooks - # - # custom hooks are invoked in such way that first failure prevents other scripts being ran - # as global scripts are ran first, failing global skips repo hooks - - before do - cleanup_hook_setup - - FileUtils.mkdir_p(File.join(tmp_repo_path, 'custom_hooks')) - FileUtils.mkdir_p(File.join(tmp_root_path, 'hooks')) - - ['pre-receive', 'update', 'post-receive'].each do |hook| - FileUtils.mkdir_p(File.join(tmp_repo_path, 'custom_hooks', "#{hook}.d")) - FileUtils.mkdir_p(File.join(tmp_root_path, 'hooks', "#{hook}.d")) - end - - FileUtils.symlink(File.join(tmp_root_path, 'hooks'), File.join(tmp_repo_path, 'hooks')) - FileUtils.symlink(File.join(ROOT_PATH, 'config.yml.example'), File.join(tmp_root_path, 'config.yml')) - - stub_const('ROOT_PATH', tmp_root_path) - end - - after do - cleanup_hook_setup - end - - context 'with gl_id_test_hook as repo hook' do - before do - create_repo_hooks(hook_gl_id) - end - - context 'pre_receive hook' do - it 'passes GL_ID variable to hook' do - expect(gitlab_custom_hook.pre_receive(changes)).to eq(true) - end - end - - context 'post_receive hook' do - it 'passes GL_ID variable to hook' do - expect(gitlab_custom_hook.post_receive(changes)).to eq(true) - end - end - - context 'update hook' do - it 'passes GL_ID variable to hook' do - expect(gitlab_custom_hook.update(ref_name, old_value, new_value)).to eq(true) - end - end - end - - context 'with gl_id_test_hook as global hook' do - before do - create_global_hooks_d(hook_gl_id) - end - - context 'pre_receive hook' do - it 'passes GL_ID variable to hook' do - expect(gitlab_custom_hook.pre_receive(changes)).to eq(true) - end - end - - context 'post_receive hook' do - it 'passes GL_ID variable to hook' do - expect(gitlab_custom_hook.post_receive(changes)).to eq(true) - end - end - - context 'update hook' do - it 'passes GL_ID variable to hook' do - expect(gitlab_custom_hook.update(ref_name, old_value, new_value)).to eq(true) - end - end - end - - context "having no hooks" do - it "returns true" do - expect(gitlab_custom_hook.pre_receive(changes)).to eq(true) - expect(gitlab_custom_hook.update(ref_name, old_value, new_value)).to eq(true) - expect(gitlab_custom_hook.post_receive(changes)).to eq(true) - end - end - - context "having only successful repo hooks" do - before do - create_repo_hooks(hook_ok) - end - - it "returns true" do - expect(gitlab_custom_hook.pre_receive(changes)).to eq(true) - expect(gitlab_custom_hook.update(ref_name, old_value, new_value)).to eq(true) - expect(gitlab_custom_hook.post_receive(changes)).to eq(true) - end - end - - context "having both successful repo and global hooks" do - before do - create_repo_hooks(hook_ok) - create_global_hooks_d(hook_ok) - end - - it "returns true" do - expect(gitlab_custom_hook.pre_receive(changes)).to eq(true) - expect(gitlab_custom_hook.update(ref_name, old_value, new_value)).to eq(true) - expect(gitlab_custom_hook.post_receive(changes)).to eq(true) - end - end - - context "having failing repo and successful global hooks" do - before do - create_repo_hooks_d(hook_fail) - create_global_hooks_d(hook_ok) - end - - it "returns false" do - expect(gitlab_custom_hook.pre_receive(changes)).to eq(false) - expect(gitlab_custom_hook.update(ref_name, old_value, new_value)).to eq(false) - expect(gitlab_custom_hook.post_receive(changes)).to eq(false) - end - - it "only executes the global hook" do - expect_call_receive_hook("custom_hooks/pre-receive.d/hook") - expect_call_update_hook("custom_hooks/update.d/hook") - expect_call_receive_hook("custom_hooks/post-receive.d/hook") - - gitlab_custom_hook.pre_receive(changes) - gitlab_custom_hook.update(ref_name, old_value, new_value) - gitlab_custom_hook.post_receive(changes) - end - end - - context "having successful repo but failing global hooks" do - before do - create_repo_hooks_d(hook_ok) - create_global_hooks_d(hook_fail) - end - - it "returns false" do - expect(gitlab_custom_hook.pre_receive(changes)).to eq(false) - expect(gitlab_custom_hook.update(ref_name, old_value, new_value)).to eq(false) - expect(gitlab_custom_hook.post_receive(changes)).to eq(false) - end - - it "executes the relevant hooks" do - expect_call_receive_hook("hooks/pre-receive.d/hook") - expect_call_receive_hook("custom_hooks/pre-receive.d/hook") - expect_call_update_hook("hooks/update.d/hook") - expect_call_update_hook("custom_hooks/update.d/hook") - expect_call_receive_hook("hooks/post-receive.d/hook") - expect_call_receive_hook("custom_hooks/post-receive.d/hook") - - gitlab_custom_hook.pre_receive(changes) - gitlab_custom_hook.update(ref_name, old_value, new_value) - gitlab_custom_hook.post_receive(changes) - end - end - - context "executing hooks in expected order" do - before do - create_repo_hooks_d(hook_ok, '01-test') - create_repo_hooks_d(hook_ok, '02-test') - create_global_hooks_d(hook_ok, '03-test') - create_global_hooks_d(hook_ok, '04-test') - end - - it "executes hooks in order" do - expect_call_receive_hook("custom_hooks/pre-receive.d/01-test").ordered - expect_call_receive_hook("custom_hooks/pre-receive.d/02-test").ordered - expect_call_receive_hook("hooks/pre-receive.d/03-test").ordered - expect_call_receive_hook("hooks/pre-receive.d/04-test").ordered - - expect_call_update_hook("custom_hooks/update.d/01-test").ordered - expect_call_update_hook("custom_hooks/update.d/02-test").ordered - expect_call_update_hook("hooks/update.d/03-test").ordered - expect_call_update_hook("hooks/update.d/04-test").ordered - - expect_call_receive_hook("custom_hooks/post-receive.d/01-test").ordered - expect_call_receive_hook("custom_hooks/post-receive.d/02-test").ordered - expect_call_receive_hook("hooks/post-receive.d/03-test").ordered - expect_call_receive_hook("hooks/post-receive.d/04-test").ordered - - gitlab_custom_hook.pre_receive(changes) - gitlab_custom_hook.update(ref_name, old_value, new_value) - gitlab_custom_hook.post_receive(changes) - end - end - - context "when the custom_hooks_dir config option is set" do - before do - allow(gitlab_custom_hook.config).to receive(:custom_hooks_dir).and_return(global_custom_hooks_path) - - FileUtils.mkdir_p(File.join(global_custom_hooks_path, "pre-receive.d")) - FileUtils.ln_sf(hook_ok, File.join(global_custom_hooks_path, "pre-receive.d", "hook")) - - create_global_hooks_d(hook_fail) - end - - it "finds hooks in that directory" do - expect(gitlab_custom_hook) - .to receive(:call_receive_hook) - .with(global_hook_path("custom_global_hooks/pre-receive.d/hook"), changes) - .and_call_original - - expect(gitlab_custom_hook.pre_receive(changes)).to eq(true) - end - - it "does not execute hooks in the default location" do - expect(gitlab_custom_hook) - .not_to receive(:call_receive_hook) - .with("hooks/pre-receive.d/hook", changes) - .and_call_original - - gitlab_custom_hook.pre_receive(changes) - end - end -end diff --git a/spec/gitlab_post_receive_spec.rb b/spec/gitlab_post_receive_spec.rb deleted file mode 100644 index d061830..0000000 --- a/spec/gitlab_post_receive_spec.rb +++ /dev/null @@ -1,284 +0,0 @@ -# coding: utf-8 -require 'spec_helper' -require 'gitlab_post_receive' - -describe GitlabPostReceive do - let(:repository_path) { "/home/git/repositories" } - let(:repo_name) { 'dzaporozhets/gitlab-ci' } - let(:actor) { 'key-123' } - let(:changes) { "123456 789012 refs/heads/tést\n654321 210987 refs/tags/tag" } - let(:wrongly_encoded_changes) { changes.encode("ISO-8859-1").force_encoding("UTF-8") } - let(:base64_changes) { Base64.encode64(wrongly_encoded_changes) } - let(:repo_path) { File.join(repository_path, repo_name) + ".git" } - let(:gl_repository) { "project-1" } - let(:push_options) { [] } - let(:gitlab_post_receive) { GitlabPostReceive.new(gl_repository, repo_path, actor, wrongly_encoded_changes, push_options) } - let(:broadcast_message) { "test " * 10 + "message " * 10 } - let(:enqueued_at) { Time.new(2016, 6, 23, 6, 59) } - let(:new_merge_request_urls) do - [{ - 'branch_name' => 'new_branch', - 'url' => 'http://localhost/dzaporozhets/gitlab-ci/merge_requests/new?merge_request%5Bsource_branch%5D=new_branch', - 'new_merge_request' => true - }] - end - let(:existing_merge_request_urls) do - [{ - 'branch_name' => 'feature_branch', - 'url' => 'http://localhost/dzaporozhets/gitlab-ci/merge_requests/1', - 'new_merge_request' => false - }] - end - - before do - $logger = double('logger').as_null_object # Global vars are bad - allow_any_instance_of(GitlabConfig).to receive(:repos_path).and_return(repository_path) - end - - describe "#exec" do - let(:response) { { 'reference_counter_decreased' => true } } - - it 'calls the api to notify the execution of the hook' do - expect_any_instance_of(GitlabNet).to receive(:post_receive).and_return(response) - - expect(gitlab_post_receive.exec).to eq(true) - end - - context 'merge request urls and broadcast messages' do - let(:response) do - { - 'reference_counter_decreased' => true, - 'merge_request_urls' => new_merge_request_urls, - 'broadcast_message' => broadcast_message - } - end - - it 'prints the merge request urls and broadcast message' do - expect_any_instance_of(GitlabNet).to receive(:post_receive).and_return(response) - assert_broadcast_message_printed(gitlab_post_receive) - assert_new_mr_printed(gitlab_post_receive) - - expect(gitlab_post_receive.exec).to eq(true) - end - - context 'when contains long url string at end' do - let(:broadcast_message) { "test " * 10 + "message " * 10 + "https://localhost:5000/test/a/really/long/url/that/is/in/the/broadcast/message/do-not-truncate-when-url" } - - it 'doesnt truncate url' do - expect_any_instance_of(GitlabNet).to receive(:post_receive).and_return(response) - assert_broadcast_message_printed_keep_long_url_end(gitlab_post_receive) - assert_new_mr_printed(gitlab_post_receive) - - expect(gitlab_post_receive.exec).to eq(true) - end - end - - context 'when contains long url string at start' do - let(:broadcast_message) { "https://localhost:5000/test/a/really/long/url/that/is/in/the/broadcast/message/do-not-truncate-when-url " + "test " * 10 + "message " * 11} - - it 'doesnt truncate url' do - expect_any_instance_of(GitlabNet).to receive(:post_receive).and_return(response) - assert_broadcast_message_printed_keep_long_url_start(gitlab_post_receive) - assert_new_mr_printed(gitlab_post_receive) - - expect(gitlab_post_receive.exec).to eq(true) - end - end - - context 'when contains long url string in middle' do - let(:broadcast_message) { "test " * 11 + "https://localhost:5000/test/a/really/long/url/that/is/in/the/broadcast/message/do-not-truncate-when-url " + "message " * 11} - - it 'doesnt truncate url' do - expect_any_instance_of(GitlabNet).to receive(:post_receive).and_return(response) - assert_broadcast_message_printed_keep_long_url_middle(gitlab_post_receive) - assert_new_mr_printed(gitlab_post_receive) - - expect(gitlab_post_receive.exec).to eq(true) - end - end - end - - context 'when warnings are present' do - let(:response) do - { - 'reference_counter_decreased' => true, - 'warnings' => 'My warning message' - } - end - - it 'treats the warning as a broadcast message' do - expect_any_instance_of(GitlabNet).to receive(:post_receive).and_return(response) - expect(gitlab_post_receive).to receive(:print_formatted_alert_message).with("WARNINGS:\nMy warning message") - expect(gitlab_post_receive.exec).to eq(true) - end - end - - context 'when redirected message available' do - let(:message) { "This is a redirected message" } - let(:response) do - { - 'reference_counter_decreased' => true, - 'redirected_message' => message - } - end - - it 'prints redirected message' do - expect_any_instance_of(GitlabNet).to receive(:post_receive).and_return(response) - assert_redirected_message_printed(gitlab_post_receive) - expect(gitlab_post_receive.exec).to eq(true) - end - - context 'when project created message is available' do - let(:message) { "This is a created project message" } - let(:response) do - { - 'reference_counter_decreased' => true, - 'project_created_message' => message - } - end - - it 'prints project created message' do - expect_any_instance_of(GitlabNet).to receive(:post_receive).and_return(response) - - assert_project_created_message_printed(gitlab_post_receive) - - expect(gitlab_post_receive.exec).to be true - end - end - end - end - - private - - def assert_new_mr_printed(gitlab_post_receive) - expect(gitlab_post_receive).to receive(:puts).ordered - expect(gitlab_post_receive).to receive(:puts).with( - "To create a merge request for new_branch, visit:" - ).ordered - expect(gitlab_post_receive).to receive(:puts).with( - " http://localhost/dzaporozhets/gitlab-ci/merge_requests/new?merge_request%5Bsource_branch%5D=new_branch" - ).ordered - expect(gitlab_post_receive).to receive(:puts).ordered - end - - def assert_existing_mr_printed(gitlab_post_receive) - expect(gitlab_post_receive).to receive(:puts).ordered - expect(gitlab_post_receive).to receive(:puts).with( - "View merge request for feature_branch:" - ).ordered - expect(gitlab_post_receive).to receive(:puts).with( - " http://localhost/dzaporozhets/gitlab-ci/merge_requests/1" - ).ordered - expect(gitlab_post_receive).to receive(:puts).ordered - end - - def assert_broadcast_message_printed(gitlab_post_receive) - expect(gitlab_post_receive).to receive(:puts).ordered - expect(gitlab_post_receive).to receive(:puts).with( - "========================================================================" - ).ordered - expect(gitlab_post_receive).to receive(:puts).ordered - - expect(gitlab_post_receive).to receive(:puts).with( - " test test test test test test test test test test message message" - ).ordered - expect(gitlab_post_receive).to receive(:puts).with( - " message message message message message message message message" - ).ordered - - expect(gitlab_post_receive).to receive(:puts).ordered - expect(gitlab_post_receive).to receive(:puts).with( - "========================================================================" - ).ordered - end - - def assert_redirected_message_printed(gitlab_post_receive) - expect(gitlab_post_receive).to receive(:puts).with("This is a redirected message") - end - - def assert_project_created_message_printed(gitlab_post_receive) - expect(gitlab_post_receive).to receive(:puts).with("This is a created project message") - end - - def assert_broadcast_message_printed_keep_long_url_end(gitlab_post_receive) - expect(gitlab_post_receive).to receive(:puts).ordered - expect(gitlab_post_receive).to receive(:puts).with( - "========================================================================" - ).ordered - expect(gitlab_post_receive).to receive(:puts).ordered - - expect(gitlab_post_receive).to receive(:puts).with( - " test test test test test test test test test test message message" - ).ordered - expect(gitlab_post_receive).to receive(:puts).with( - " message message message message message message message message" - ).ordered - - expect(gitlab_post_receive).to receive(:puts).with( - "https://localhost:5000/test/a/really/long/url/that/is/in/the/broadcast/message/do-not-truncate-when-url" - ).ordered - - expect(gitlab_post_receive).to receive(:puts).ordered - expect(gitlab_post_receive).to receive(:puts).with( - "========================================================================" - ).ordered - end - - def assert_broadcast_message_printed_keep_long_url_start(gitlab_post_receive) - expect(gitlab_post_receive).to receive(:puts).ordered - expect(gitlab_post_receive).to receive(:puts).with( - "========================================================================" - ).ordered - expect(gitlab_post_receive).to receive(:puts).ordered - - expect(gitlab_post_receive).to receive(:puts).with( - "https://localhost:5000/test/a/really/long/url/that/is/in/the/broadcast/message/do-not-truncate-when-url" - ).ordered - - expect(gitlab_post_receive).to receive(:puts).with( - " test test test test test test test test test test message message" - ).ordered - - expect(gitlab_post_receive).to receive(:puts).with( - " message message message message message message message message" - ).ordered - - expect(gitlab_post_receive).to receive(:puts).with( - " message" - ).ordered - - expect(gitlab_post_receive).to receive(:puts).ordered - expect(gitlab_post_receive).to receive(:puts).with( - "========================================================================" - ).ordered - end - - def assert_broadcast_message_printed_keep_long_url_middle(gitlab_post_receive) - expect(gitlab_post_receive).to receive(:puts).ordered - expect(gitlab_post_receive).to receive(:puts).with( - "========================================================================" - ).ordered - expect(gitlab_post_receive).to receive(:puts).ordered - - expect(gitlab_post_receive).to receive(:puts).with( - " test test test test test test test test test test test" - ).ordered - - expect(gitlab_post_receive).to receive(:puts).with( - "https://localhost:5000/test/a/really/long/url/that/is/in/the/broadcast/message/do-not-truncate-when-url" - ).ordered - - expect(gitlab_post_receive).to receive(:puts).with( - " message message message message message message message message" - ).ordered - - expect(gitlab_post_receive).to receive(:puts).with( - " message message message" - ).ordered - - expect(gitlab_post_receive).to receive(:puts).ordered - expect(gitlab_post_receive).to receive(:puts).with( - "========================================================================" - ).ordered - end -end |