aboutgitcodebugslistschat
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2025-10-10 14:45:47 +1100
committerStefano Brivio <sbrivio@redhat.com>2025-10-15 23:16:38 +0200
commit898cf50e94e596409869d57e58e29f1ff069664a (patch)
tree8a051743fe50a2735e12169837087772546e504d
parent5da0316f27c9b36b7ee4ba181d38a8dc358b2328 (diff)
downloadpasst-898cf50e94e596409869d57e58e29f1ff069664a.tar
passt-898cf50e94e596409869d57e58e29f1ff069664a.tar.gz
passt-898cf50e94e596409869d57e58e29f1ff069664a.tar.bz2
passt-898cf50e94e596409869d57e58e29f1ff069664a.tar.lz
passt-898cf50e94e596409869d57e58e29f1ff069664a.tar.xz
passt-898cf50e94e596409869d57e58e29f1ff069664a.tar.zst
passt-898cf50e94e596409869d57e58e29f1ff069664a.zip
test: Use ${} consistently in lib/exeter
Most of the test shell scripts use {} around variable names even when it's not required, for consistency. lib/exeter didn't do so consistently, however. Correct that. Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
-rw-r--r--test/lib/exeter20
1 files changed, 10 insertions, 10 deletions
diff --git a/test/lib/exeter b/test/lib/exeter
index 3b19bea..0b36159 100644
--- a/test/lib/exeter
+++ b/test/lib/exeter
@@ -13,18 +13,18 @@
# Copyright Red Hat
# Author: David Gibson <david@gibson.dropbear.id.au>
-EXETOOL="$BASEPATH/exeter/exetool/exetool"
+EXETOOL="${BASEPATH}/exeter/exetool/exetool"
# is_exeter() - Determine if a test file is an exeter program
# $@: Command line to invoke test program
is_exeter() {
- $EXETOOL probe -- "$@"
+ ${EXETOOL} probe -- "${@}"
}
# exeter() - Run each test in an exeter program, logging each test separately
# $@: Command line to invoke exeter test program
exeter() {
- STATESETUP="${STATEBASE}/$1"
+ STATESETUP="${STATEBASE}/${1}"
mkdir -p "${STATESETUP}"
context_setup_host host
@@ -32,9 +32,9 @@ exeter() {
cd test
- __ntests=$($EXETOOL list -- "$@" | wc -l)
- if [ $? != 0 ]; then
- info "Failed to get exeter manifest for $@"
+ __ntests=$(${EXETOOL} list -- "${@}" | wc -l)
+ if [ ${?} != 0 ]; then
+ info "Failed to get exeter manifest for ${@}"
pause_continue \
"Press any key to pause test session" \
"Resuming in " \
@@ -43,13 +43,13 @@ exeter() {
return
fi
- status_file_start "$* (exeter)" ${__ntests}
+ status_file_start "${*} (exeter)" ${__ntests}
[ ${CI} -eq 1 ] && video_link "${1}"
- for __testid in $($EXETOOL list -- "$@"); do
- __desc="$($EXETOOL desc -- "$@" -- ${__testid})"
+ for __testid in $(${EXETOOL} list -- "${@}"); do
+ __desc="$(${EXETOOL} desc -- "${@}" -- ${__testid})"
status_test_start "${__desc}"
- context_run host "$@" "${__testid}" && status_test_ok || status_test_fail
+ context_run host "${@}" "${__testid}" && status_test_ok || status_test_fail
done
cd ..