]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Improve the LOG_DRIVER and LOG_COMPILER for the system tests
authorOndřej Surý <ondrej@isc.org>
Fri, 24 Apr 2020 13:29:54 +0000 (15:29 +0200)
committerOndřej Surý <ondrej@isc.org>
Fri, 24 Apr 2020 14:59:43 +0000 (16:59 +0200)
There are several improvements over the default/previous behaviour of
the test log driver and log compiler:

 * The system-test-driver.sh was dropped (it was used incorrectly)
 * The run.sh script is now both log compiler and cli script to run
   individual tests
 * The custom-test-driver was added as extended version of the automake
   test-driver with capability to tee the test output to stdout when
   `--verbose yes` is passed to it (you can use LOG_DRIVER_FLAGS to
   add the option by default)
 * Makefile.am has been extended to honor V=1 for the system tests
   test-driver (e.g. V=1 adds `--verbose yes` to AM_LOG_DRIVER_FLAGS)

.gitlab-ci.yml
bin/tests/system/.gitignore
bin/tests/system/Makefile.am
bin/tests/system/run.sh.in
bin/tests/system/system-test-driver.sh.in [deleted file]
configure.ac
custom-test-driver [new file with mode: 0755]
util/copyrights

index 6c741360895d29821ea218626f25b4b89625d343..103ca7569e4b6bb17413f377bcb93ee5426262bd 100644 (file)
@@ -255,7 +255,8 @@ stages:
     - *setup_interfaces
     - *setup_softhsm
   script:
-    - cd bin/tests/system && make -j${TEST_PARALLEL_JOBS:-1} -k check V=1
+    - cd bin/tests/system
+    - make -j${TEST_PARALLEL_JOBS:-1} -k check V=1
   after_script:
     - cat bin/tests/system/test-suite.log
 
index 998718e75819af4055830bc87b8ce07b2acb2fac..8205de1dbc83823475208dfc8409f2da588dc795 100644 (file)
@@ -15,7 +15,7 @@ parallel.mk
 /get_base_port.state
 /get_base_port.lock
 /run.sh
+/run.log
 /start.sh
 /stop.sh
-/system-test-driver.sh
 /ifconfig.sh
index cca65fa19e0f9d7c1a234a79b00340860a574617..43e00b9486764eec21ae1e1bea236c8ff94b3b01 100644 (file)
@@ -161,7 +161,18 @@ check:
        exit 1
 endif !HAVE_PERL
 
-LOG_COMPILER = $(builddir)/system-test-driver.sh
+LOG_DRIVER_V = $(LOG_DRIVER_V_@AM_V@)
+LOG_DRIVER_V_ = $(LOG_DRIVER_V_@AM_DEFAULT_V@)
+LOG_DRIVER_V_0 = --verbose no
+LOG_DRIVER_V_1 = --verbose yes
+
+LOG_DRIVER = $(top_srcdir)/custom-test-driver
+AM_LOG_DRIVER_FLAGS = $(LOG_DRIVER_V)
+
+LOG_COMPILER = $(builddir)/run.sh
+AM_LOG_FLAGS = -r -p "$$("$(srcdir)/get_base_port.sh")"
+
+$(TESTS): run.sh
 
 clean-local:
        -rm -f get_base_port.state get_base_port.lock
index 1f38dc45a3a03c12bd246718985d709f2ff25f87..f7d9735a73d52f1ae043f25c6db582faa8294527 100644 (file)
@@ -37,24 +37,37 @@ else
        clean=true
 fi
 
-while getopts "knp:r-:" flag; do
-    case "$flag" in
-       -) case "${OPTARG}" in
-               keep) stopservers=false ;;
-               noclean) clean=false ;;
-           esac
-           ;;
-       k) stopservers=false ;;
-       n) clean=false ;;
-       p) baseport=$OPTARG ;;
-       *) echo "invalid option" >&2; exit 1 ;;
-    esac
+do_run=false
+log_flags="-r"
+while getopts "knp:r-:" OPT; do
+       log_flags="$log_flags -$OPT$OPTARG"
+       if [ "$OPT" = "-" ] && [ -n "$OPTARG" ]; then
+               OPT="${OPTARG%%=*}"
+               OPTARG="${OPTARG#$OPT}"
+               OPTARG="${OPTARG#=}"
+       fi
+
+       case "$OPT" in
+               k | keep) stopservers=false ;;
+               n | noclean) clean=false ;;
+               p | port) baseport=$OPTARG ;;
+               r | run) do_run=true ;;
+               s | skip) exit 77 ;;
+               -) break ;;
+               *) echo "invalid option" >&2; exit 1 ;;
+       esac
 done
+
 shift $((OPTIND-1))
 
+if ! $do_run; then
+       env - TESTS="$1" TEST_SUITE_LOG=run.log LOG_DRIVER_FLAGS="--verbose yes --color-tests yes" LOG_FLAGS="$log_flags" make -e check
+       exit $?
+fi
+
 if [ $# -eq 0 ]; then
-    echofail "Usage: $0 [-k] [-n] [-p <PORT>] test-directory [test-options]" >&2;
-    exit 1
+       echofail "Usage: $0 [-k] [-n] [-p <PORT>] test-directory [test-options]" >&2;
+       exit 1
 fi
 
 systest=$(basename "${1%%/}")
diff --git a/bin/tests/system/system-test-driver.sh.in b/bin/tests/system/system-test-driver.sh.in
deleted file mode 100644 (file)
index edffd7b..0000000
+++ /dev/null
@@ -1,60 +0,0 @@
-#!/bin/sh
-# shellcheck disable=SC2181
-# shellcheck disable=SC2034
-
-builddir=@abs_builddir@
-srcdir=@abs_srcdir@
-
-usage() {
-    echo "$0 --test-name=NAME --log-file=PATH.log --trs-file=PATH.trs --color-tests={yes|no} --expect-failure={yes|no} --enable-hard-errors={yes|no}"
-}
-
-TEST_NAME=
-LOG_FILE=
-TRS_FILE=
-COLOR_TESTS=yes
-EXPECT_FAILURE=no
-HARD_ERRORS=yes
-
-while getopts -: OPT; do
-       if [ "$OPT" = "-" ] && [ -n "$OPTARG" ]; then
-               OPT="${OPTARG%%=*}"
-               OPTARG="${OPTARG#$OPT}"
-               OPTARG="${OPTARG#=}"
-       fi
-       case "$OPT" in
-               test-name) TEST_NAME="$OPTARG" ;;
-               log-file) LOG_FILE="$OPTARG" ;;
-               trs-file) TRS_FILE="$OPTARG" ;;
-               color-tests) COLOR_TESTS="$OPTARG" ;;
-               expect-failure) EXPECT_FAILURE="$OPTARG" ;;
-               hard-errors) HARD_ERRORS="$OPTARG" ;;
-               *) break ;;
-       esac
-done
-shift $((OPTIND-1))
-
-if [ -z "$1" ]; then
-    echo "fatal: test name required"
-    usage
-    exit 1
-fi
-
-TEST_PROGRAM="$1"
-shift
-
-if [ -z "$TEST_NAME" ]; then
-    TEST_NAME="$(basename "$TEST_PROGRAM")"
-fi
-if [ -z "$LOG_FILE" ]; then
-    LOG_FILE="$TEST_PROGRAM.log"
-fi
-if [ -z "$TRS_FILE" ]; then
-    TRS_FILE="$TEST_PROGRAM.trs"
-fi
-
-echo "Running $TEST_PROGRAM"
-
-"${builddir}/run.sh" -p "$("${srcdir}/get_base_port.sh")" "$@" "$TEST_PROGRAM"
-
-exit $?
index 36fdcfb3b0f63ab4f013178e89c3e81f519b347e..36d038b810901a61fc17586c636dac693dee7f36 100644 (file)
@@ -1595,8 +1595,6 @@ AC_CONFIG_FILES([bin/tests/system/start.sh],
                [chmod +x bin/tests/system/start.sh])
 AC_CONFIG_FILES([bin/tests/system/stop.sh],
                [chmod +x bin/tests/system/stop.sh])
-AC_CONFIG_FILES([bin/tests/system/system-test-driver.sh],
-               [chmod +x bin/tests/system/system-test-driver.sh])
 
 # Misc
 
diff --git a/custom-test-driver b/custom-test-driver
new file mode 100755 (executable)
index 0000000..e11acd5
--- /dev/null
@@ -0,0 +1,156 @@
+#! /bin/sh
+# test-driver - basic testsuite driver script.
+
+scriptversion=2020-04-24.14; # UTC
+
+# Copyright (C) 2011-2020 Free Software Foundation, Inc.
+#
+# 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; either version 2, or (at your option)
+# any later version.
+#
+# 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.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <https://www.gnu.org/licenses/>.
+
+# As a special exception to the GNU General Public License, if you
+# distribute this file as part of a program that contains a
+# configuration script generated by Autoconf, you may include it under
+# the same distribution terms that you use for the rest of that program.
+
+# This file is maintained in Automake, please report
+# bugs to <bug-automake@gnu.org> or send patches to
+# <automake-patches@gnu.org>.
+
+# Make unconditional expansion of undefined variables an error.  This
+# helps a lot in preventing typo-related bugs.
+set -u
+
+usage_error ()
+{
+  echo "$0: $*" >&2
+  print_usage >&2
+  exit 2
+}
+
+print_usage ()
+{
+  cat <<END
+Usage:
+  test-driver --test-name=NAME --log-file=PATH --trs-file=PATH
+              [--expect-failure={yes|no}] [--color-tests={yes|no}]
+              [--enable-hard-errors={yes|no}] [--]
+              TEST-SCRIPT [TEST-SCRIPT-ARGUMENTS]
+The '--test-name', '--log-file' and '--trs-file' options are mandatory.
+END
+}
+
+test_name= # Used for reporting.
+log_file=  # Where to save the output of the test script.
+trs_file=  # Where to save the metadata of the test run.
+status_file= # Where to save the status of the test run.
+expect_failure=no
+color_tests=no
+enable_hard_errors=yes
+verbose=no
+while test $# -gt 0; do
+  case $1 in
+  --help) print_usage; exit $?;;
+  --version) echo "test-driver $scriptversion"; exit $?;;
+  --test-name) test_name=$2; shift;;
+  --log-file) log_file=$2; shift;;
+  --trs-file) trs_file=$2; shift;;
+  --color-tests) color_tests=$2; shift;;
+  --expect-failure) expect_failure=$2; shift;;
+  --enable-hard-errors) enable_hard_errors=$2; shift;;
+  --verbose) verbose=$2; shift;;
+  --) shift; break;;
+  -*) usage_error "invalid option: '$1'";;
+   *) break;;
+  esac
+  shift
+done
+
+missing_opts=
+test x"$test_name" = x && missing_opts="$missing_opts --test-name"
+test x"$log_file"  = x && missing_opts="$missing_opts --log-file"
+test x"$trs_file"  = x && missing_opts="$missing_opts --trs-file"
+if test x"$missing_opts" != x; then
+  usage_error "the following mandatory options are missing:$missing_opts"
+fi
+
+if test $# -eq 0; then
+  usage_error "missing argument"
+fi
+
+if test $color_tests = yes; then
+  # Keep this in sync with 'lib/am/check.am:$(am__tty_colors)'.
+  red='\e[0;31m' # Red.
+  grn='\e[0;32m' # Green.
+  lgn='\e[1;32m' # Light green.
+  blu='\e[1;34m' # Blue.
+  mgn='\e[0;35m' # Magenta.
+  std='\e[m'     # No color.
+else
+  red= grn= lgn= blu= mgn= std=
+fi
+
+do_exit='rm -f $log_file $trs_file $status_file; (exit $st); exit $st'
+trap "st=129; $do_exit" 1
+trap "st=130; $do_exit" 2
+trap "st=141; $do_exit" 13
+trap "st=143; $do_exit" 15
+
+# Test script is run here.
+if test $verbose = yes; then
+  status_file=$(mktemp)
+  ("$@" 2>&1; echo $? > "$status_file") | tee $log_file
+else
+  "$@" >$log_file 2>&1; echo $? > "$status_file"
+fi
+read -r estatus < "$status_file"
+
+if test $enable_hard_errors = no && test $estatus -eq 99; then
+  tweaked_estatus=1
+else
+  tweaked_estatus=$estatus
+fi
+
+case $tweaked_estatus:$expect_failure in
+  0:yes) col=$red res=XPASS recheck=yes gcopy=yes;;
+  0:*)   col=$grn res=PASS  recheck=no  gcopy=no;;
+  77:*)  col=$blu res=SKIP  recheck=no  gcopy=yes;;
+  99:*)  col=$mgn res=ERROR recheck=yes gcopy=yes;;
+  *:yes) col=$lgn res=XFAIL recheck=no  gcopy=yes;;
+  *:*)   col=$red res=FAIL  recheck=yes gcopy=yes;;
+esac
+
+# Report the test outcome and exit status in the logs, so that one can
+# know whether the test passed or failed simply by looking at the '.log'
+# file, without the need of also peaking into the corresponding '.trs'
+# file (automake bug#11814).
+echo "$res $test_name (exit status: $estatus)" >>$log_file
+
+# Report outcome to console.
+echo "${col}${res}${std}: $test_name"
+
+# Register the test result, and other relevant metadata.
+echo ":test-result: $res" > $trs_file
+echo ":global-test-result: $res" >> $trs_file
+echo ":recheck: $recheck" >> $trs_file
+echo ":copy-in-global-log: $gcopy" >> $trs_file
+
+# Local Variables:
+# mode: shell-script
+# sh-indentation: 2
+# eval: (add-hook 'before-save-hook 'time-stamp)
+# time-stamp-start: "scriptversion="
+# time-stamp-format: "%:y-%02m-%02d.%02H"
+# time-stamp-time-zone: "UTC0"
+# time-stamp-end: "; # UTC"
+# End:
index 2aa21cca2b2a6f42be7fbb8ce981f8d2b4b849d9..a16a02e5ecee391f5e47df220c5ea145f79bf608 100644 (file)
 ./bin/tests/system/synthfromdnssec/ns1/sign.sh SH      2017,2018,2019,2020
 ./bin/tests/system/synthfromdnssec/setup.sh    SH      2017,2018,2019,2020
 ./bin/tests/system/synthfromdnssec/tests.sh    SH      2017,2018,2019,2020
-./bin/tests/system/system-test-driver.sh.in    X       2020
 ./bin/tests/system/tcp/ans6/ans.py             PYTHON  2019,2020
 ./bin/tests/system/tcp/clean.sh                        SH      2014,2016,2018,2019,2020
 ./bin/tests/system/tcp/setup.sh                        SH      2018,2019,2020
 ./bin/tests/system/win32/pipequeries.vcxproj.filters.in        X       2016,2018,2019,2020
 ./bin/tests/system/win32/pipequeries.vcxproj.in        X       2016,2017,2018,2019,2020
 ./bin/tests/system/win32/pipequeries.vcxproj.user      X       2016,2018,2019,2020
+./bin/tests/system/wire_test.c                 C       2020
 ./bin/tests/system/xfer/ans5/badkeydata                X       2011,2018,2019,2020
 ./bin/tests/system/xfer/ans5/badmessageid      X       2020
 ./bin/tests/system/xfer/ans5/goodaxfr          X       2011,2018,2019,2020
 ./contrib/scripts/named-bootconf.sh            SH.PORTION      1999,2000,2001,2004,2006,2007,2012,2014,2016,2018,2019,2020
 ./contrib/scripts/nanny.pl                     PERL    2000,2001,2004,2007,2012,2014,2016,2018,2019,2020
 ./contrib/scripts/zone-edit.sh.in              SH      2010,2012,2014,2016,2018,2019,2020
+./custom-test-driver                           X       2020
 ./doc/arm/Bv9ARM-book.xml                      SGML    2000,2001,2002,2003,2004,2005,2006,2007,2008,2009,2010,2011,2012,2013,2014,2015,2016,2017,2018,2019,2020
 ./doc/arm/Bv9ARM.ch01.html                     X       2000,2001,2003,2004,2005,2006,2007,2008,2009,2010,2011,2012,2013,2014,2015,2016,2017,2018,2019,2020
 ./doc/arm/Bv9ARM.ch02.html                     X       2000,2001,2003,2004,2005,2006,2007,2008,2009,2010,2011,2012,2013,2014,2015,2016,2017,2018,2019,2020