From 52fdc050139649ce17df82e98e59e78e79418801 Mon Sep 17 00:00:00 2001 From: Alex Thiessen Date: Fri, 5 Jan 2018 05:07:23 +0000 Subject: 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 Reviewed-on: https://review.coreboot.org/23281 Reviewed-by: Stefan Reinauer Reviewed-by: Paul Menzel Tested-by: build bot (Jenkins) --- util/gitconfig/test/commit-message-hook.sh | 87 ++++++++++++++++++++++++++++++ 1 file changed, 87 insertions(+) create mode 100755 util/gitconfig/test/commit-message-hook.sh (limited to 'util/gitconfig/test') 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 +## +## 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 +## +## + +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}" -- cgit v1.2.3