Remove post push hook

This commit is contained in:
Donal McBreen
2023-05-22 15:06:05 +01:00
parent 0bc1fbfb74
commit 38023fe538
6 changed files with 2 additions and 36 deletions

View File

@@ -912,10 +912,9 @@ You can change their location by setting `hooks_path` in the configuration file.
If the script returns a non-zero exit code the command will be aborted. If the script returns a non-zero exit code the command will be aborted.
There are currently two hooks: There is currently one hook:
- pre-build - pre-build
- post-push
## Stage of development ## Stage of development

View File

@@ -32,8 +32,6 @@ class Mrsk::Cli::Build < Mrsk::Cli::Base
end end
end end
end end
run_hook "post-push"
end end
end end

View File

@@ -1,20 +0,0 @@
#!/bin/sh
# A sample post-push hook
#
# Checks:
# 1. We have a clean checkout
#
# These environment variables are available:
# MRSK_RECORDED_AT
# MRSK_PERFORMER
# MRSK_VERSION
# MRSK_DESTINATION (if set)
if [ -n "$(git status --porcelain)" ]; then
echo "Git checkout is not clean, aborting..."
git status --porcelain >&2
exit 1
fi
exit 0

View File

@@ -49,14 +49,6 @@ class CliBuildTest < CliTestCase
assert @executions.none? { |args| args[0..2] == [:docker, :buildx, :build] } assert @executions.none? { |args| args[0..2] == [:docker, :buildx, :build] }
end end
test "push post-push hook failure" do
fail_hook("post-push")
assert_raises(Mrsk::Cli::HookError) { run_command("push") }
assert @executions.any? { |args| args[0..2] == [:docker, :buildx, :build] }
end
test "pull" do test "pull" do
run_command("pull").tap do |output| run_command("pull").tap do |output|
assert_match /docker image rm --force dhh\/app:999/, output assert_match /docker image rm --force dhh\/app:999/, output

View File

@@ -1,3 +0,0 @@
#!/bin/sh
echo "Built and pushed!"
mkdir -p /tmp/${TEST_ID} && touch /tmp/${TEST_ID}/post-push

View File

@@ -84,7 +84,7 @@ class IntegrationTest < ActiveSupport::TestCase
end end
def assert_hooks_ran def assert_hooks_ran
[ "pre-build", "post-push" ].each do |hook| [ "pre-build" ].each do |hook|
file = "/tmp/#{ENV["TEST_ID"]}/#{hook}" file = "/tmp/#{ENV["TEST_ID"]}/#{hook}"
assert_match /File: #{file}/, deployer_exec("stat #{file}", capture: true) assert_match /File: #{file}/, deployer_exec("stat #{file}", capture: true)
end end