From 5149f34902555852ff96a33ea377223d4b770ffe Mon Sep 17 00:00:00 2001 From: David Gibson Date: Fri, 10 Oct 2025 14:45:48 +1100 Subject: 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 Signed-off-by: Stefano Brivio --- test/lib/exeter | 4 ++-- 1 file 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 .. -- cgit v1.2.3