From f9075eb04083030b70b6b58b6c4042d7d74c701c Mon Sep 17 00:00:00 2001
From: Lin Jen-Shin <godfat@godfat.org>
Date: Mon, 1 Aug 2016 19:16:59 +0800
Subject: [PATCH] Add test for running hooks for pipeline after touched
---
spec/models/ci/pipeline_spec.rb | 33 +++++++++++++++++++++++++++++++--
1 file changed, 31 insertions(+), 2 deletions(-)
diff --git a/spec/models/ci/pipeline_spec.rb b/spec/models/ci/pipeline_spec.rb
index 0d4c86955ce..aa05fc78f94 100644
--- a/spec/models/ci/pipeline_spec.rb
+++ b/spec/models/ci/pipeline_spec.rb
@@ -513,7 +513,7 @@ describe Ci::Pipeline, models: true do
create :ci_build, :success, pipeline: pipeline, name: 'rspec'
create :ci_build, :allowed_to_fail, :failed, pipeline: pipeline, name: 'rubocop'
end
-
+
it 'returns true' do
is_expected.to be_truthy
end
@@ -524,7 +524,7 @@ describe Ci::Pipeline, models: true do
create :ci_build, :success, pipeline: pipeline, name: 'rspec'
create :ci_build, :allowed_to_fail, :success, pipeline: pipeline, name: 'rubocop'
end
-
+
it 'returns false' do
is_expected.to be_falsey
end
@@ -542,4 +542,33 @@ describe Ci::Pipeline, models: true do
end
end
end
+
+ describe '#execute_hooks' do
+ let!(:hook) do
+ create(:project_hook, project: project, pipeline_events: enabled)
+ end
+ let(:enabled) { raise NotImplementedError }
+
+ before do
+ WebMock.stub_request(:post, hook.url)
+ pipeline.touch
+ ProjectWebHookWorker.drain
+ end
+
+ context 'with pipeline hooks enabled' do
+ let(:enabled) { true }
+
+ it 'executes pipeline_hook after touched' do
+ expect(WebMock).to have_requested(:post, hook.url).once
+ end
+ end
+
+ context 'with pipeline hooks disabled' do
+ let(:enabled) { false }
+
+ it 'did not execute pipeline_hook after touched' do
+ expect(WebMock).not_to have_requested(:post, hook.url)
+ end
+ end
+ end
end
--
GitLab