We run a bunch of static checkers as part of our testsuite. That's useful,
but it means that if a user doesn't have one of them installed, it fails
the entire testsuite. Alter our scripts to skip the test, rather than
failing outright if the checker tool is not installed.
This requires exeter v0.4.4 or later.
Signed-off-by: David Gibson
---
test/build/static_checkers.sh | 17 +++++++++++++----
test/lib/exeter | 10 +++++++++-
2 files changed, 22 insertions(+), 5 deletions(-)
diff --git a/test/build/static_checkers.sh b/test/build/static_checkers.sh
index 228b99ae..caccd183 100755
--- a/test/build/static_checkers.sh
+++ b/test/build/static_checkers.sh
@@ -15,16 +15,25 @@
. $(dirname $0)/../exeter/sh/exeter.sh
-exeter_register cppcheck make -C .. cppcheck
+do_check() {
+ checker="$1"
+ shift
+ if ! which "${checker}"; then
+ exeter_skip "${checker} not available"
+ fi
+ make "${@}" "${checker}"
+}
+
+exeter_register cppcheck do_check cppcheck -C ..
exeter_set_description cppcheck "passt sources pass cppcheck"
-exeter_register clang_tidy make -C .. clang-tidy
+exeter_register clang_tidy do_check clang-tody -C ..
exeter_set_description clang_tidy "passt sources pass clang-tidy"
-exeter_register flake8 make flake8
+exeter_register flake8 do_check flake8
exeter_set_description flake8 "passt tests in Python pass flake8"
-exeter_register mypy make mypy
+exeter_register mypy do_check mypy
exeter_set_description mypy "passt tests in Python pass mypy --strict"
exeter_main "$@"
diff --git a/test/lib/exeter b/test/lib/exeter
index 530c6909..6ed92a16 100644
--- a/test/lib/exeter
+++ b/test/lib/exeter
@@ -49,7 +49,15 @@ exeter() {
for __testid in $($EXETOOL list -- "$@"); do
__desc="$($EXETOOL desc -- "$@" -- "${__testid}")"
status_test_start "${__desc}"
- context_run host "$* '${__testid}'" && status_test_ok || status_test_fail
+ status=0
+ context_run host "$* '${__testid}'" || status="$?"
+ if [ "$status" = 0 ]; then
+ status_test_ok
+ elif [ "$status" = 77 ]; then
+ status_test_skip
+ else
+ status_test_fail
+ fi
done
cd ..
--
2.51.0