diff options
| author | David Gibson <david@gibson.dropbear.id.au> | 2025-10-10 14:45:48 +1100 |
|---|---|---|
| committer | Stefano Brivio <sbrivio@redhat.com> | 2025-10-15 23:16:38 +0200 |
| commit | 5149f34902555852ff96a33ea377223d4b770ffe (patch) | |
| tree | d6e721e923c8ad841e58c8107f1930c5923b329d | |
| parent | 898cf50e94e596409869d57e58e29f1ff069664a (diff) | |
| download | passt-5149f34902555852ff96a33ea377223d4b770ffe.tar passt-5149f34902555852ff96a33ea377223d4b770ffe.tar.gz passt-5149f34902555852ff96a33ea377223d4b770ffe.tar.bz2 passt-5149f34902555852ff96a33ea377223d4b770ffe.tar.lz passt-5149f34902555852ff96a33ea377223d4b770ffe.tar.xz passt-5149f34902555852ff96a33ea377223d4b770ffe.tar.zst passt-5149f34902555852ff96a33ea377223d4b770ffe.zip | |
test: Add some missing quoting in exeter runner
exeter() quoted ${__testid}, but in some places we use it there's an
extra level of shell, which needs another layer of quoting. This breaks
if testids include ';', which is quite common in exeter tests created as
a composition/pipeline of two functions. Add the required extra quoting.
While we're there, improve consistency with other scripts by always using
the optional {} around variable names.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
| -rw-r--r-- | test/lib/exeter | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/lib/exeter b/test/lib/exeter index 0b36159..7ea084a 100644 --- a/test/lib/exeter +++ b/test/lib/exeter @@ -47,9 +47,9 @@ exeter() { [ ${CI} -eq 1 ] && video_link "${1}" for __testid in $(${EXETOOL} list -- "${@}"); do - __desc="$(${EXETOOL} desc -- "${@}" -- ${__testid})" + __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 .. |