summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlex Thiessen <alex.thiessen.de+coreboot@gmail.com>2018-01-05 05:07:23 +0000
committerPatrick Georgi <pgeorgi@google.com>2018-11-16 10:08:48 +0000
commit52fdc050139649ce17df82e98e59e78e79418801 (patch)
tree73337b06032e4bda7aabe2e8195008ff096c1053
parent9b1cbe040b1e502787d8d9576b9c8e567849d181 (diff)
util/gitconfig/test: Add commit-msg hook test
Add a test that, after cloning the repository to a temporary directory, installs git hooks and attempts to do a good and a bad commit, expecting the former to succeed and the latter to fail, thus testing the `commit-msg` hook. Change-Id: Icdaf0109c60cb5b6952b1a2468ab050a742e4201 Signed-off-by: Alex Thiessen <alex.thiessen.de+coreboot@gmail.com> Reviewed-on: https://review.coreboot.org/23281 Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rwxr-xr-xutil/gitconfig/test/commit-message-hook.sh87
1 files changed, 87 insertions, 0 deletions
diff --git a/util/gitconfig/test/commit-message-hook.sh b/util/gitconfig/test/commit-message-hook.sh
new file mode 100755
index 0000000000..a2c53eccde
--- /dev/null
+++ b/util/gitconfig/test/commit-message-hook.sh
@@ -0,0 +1,87 @@
+#!/bin/bash
+
+##
+## This file is part of the coreboot project.
+##
+## Copyright (C) 2003-2018 Alex Thiessen <alex.thiessen.de+coreboot@gmail.com>
+##
+## This program is free software; you can redistribute it and/or modify
+## it under the terms of the GNU General Public License as published by
+## the Free Software Foundation; version 3 or later of the License.
+##
+## This program is distributed in the hope that it will be useful,
+## but WITHOUT ANY WARRANTY; without even the implied warranty of
+## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+## GNU General Public License for more details.
+##
+## SPDX-License-Identifier: GPL-3.0-or-later
+## <https://spdx.org/licenses/GPL-3.0-or-later.html>
+##
+
+set -o errexit
+set -o nounset
+
+# static analysis
+if command -v shellcheck 1>/dev/null; then
+ shellcheck --exclude=1090,1091 \
+ "${BASH_SOURCE[0]}" \
+ "$(dirname "${BASH_SOURCE[0]}")/helpers.sh"
+else
+ echo "shellcheck not found, running unchecked" >&2
+fi
+
+# dependency check
+dependencies=(dirname git make mktemp rm timeout)
+for dependency in "${dependencies[@]}"; do
+ if ! command -v "${dependency}" 1>/dev/null; then
+ echo "missing ${dependency}, test skipped" >&2
+ exit 0
+ fi
+done
+
+source "$(dirname "${BASH_SOURCE[0]}")/helpers.sh"
+
+# setup
+base_dir="$(mktemp --directory --tmpdir \
+ "test-$(basename "${BASH_SOURCE[0]}" .sh)-XXXXXXXX")"
+clone_dir="${base_dir}/coreboot"
+git clone "$(git rev-parse --show-toplevel)" "${clone_dir}" \
+ 1>"${base_dir}/clone.log" 2>&1
+
+(
+ set -o errexit
+ set -o nounset
+
+ clone_submodules "${clone_dir}" "${base_dir}"
+ git config user.name "John Doe"
+ git config user.email "john.doe@example.com"
+ make gitconfig
+
+ # test
+ echo "good case..."
+ log_file="${base_dir}/good_case.log"
+ echo "this is a test" >> README
+ timeout 4m git commit --all --signoff --message="good case" \
+ 1>"${log_file}" 2>&1 \
+ || check_exit_code positive "${log_file}"
+ git reset --hard --quiet HEAD^
+ git clean -d --force --quiet -x
+
+ echo "bad case..."
+ log_file="${base_dir}/bad_case.log"
+ # Goal here is to verify whether a failing `util/lint` test will prevent
+ # a commit. It's a bit tricky because `checkpatch.pl` is run just after
+ # the lint tests and will cover many of those too. So we need a case
+ # that fails with `util/lint` but succeeds with `checkpatch.pl`. I found
+ # that `lint-stable-009-old-licenses` does the job quite well.
+ printf "You should have received a copy of the %s\n" "GNU" > src/test.c
+ git add src/test.c
+ timeout 4m git commit --signoff --message="bad case" \
+ 1>"${log_file}" 2>&1 \
+ && check_exit_code negative "${log_file}"
+ git rm --force --quiet src/test.c
+ git clean -d --force --quiet -x
+)
+
+# teardown
+rm --force --recursive "${base_dir}"