diff options
| author | David Gibson <david@gibson.dropbear.id.au> | 2025-10-10 14:45:47 +1100 |
|---|---|---|
| committer | Stefano Brivio <sbrivio@redhat.com> | 2025-10-15 23:16:38 +0200 |
| commit | 898cf50e94e596409869d57e58e29f1ff069664a (patch) | |
| tree | 8a051743fe50a2735e12169837087772546e504d | |
| parent | 5da0316f27c9b36b7ee4ba181d38a8dc358b2328 (diff) | |
| download | passt-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/exeter | 20 |
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 .. |
