Re: [PATCH 1/2] test: Add some missing quoting in exeter runner

[Stefano Brivio <sbrivio@redhat.com>]

On Thu,  9 Oct 2025 14:43:57 +1100
David Gibson <david@gibson.dropbear.id.au> wrote:

> 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.
> 
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> ---
>  test/lib/exeter | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/test/lib/exeter b/test/lib/exeter
> index 3b19beaa..530c6909 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

Nit: for consistency, given that @ and * are variable names, I would
enclose them in curly brackets as well, while at it.

-- 
Stefano