Re: [meta-virtualization][PATCH] test_vdkr_registry: fix test_image_requires_subcommand

Bruce Ashfield <[email protected]> Tue, 28 Apr 2026 11:47:43 +0000
Newsgroups org.yoctoproject.lists.meta-virtualization
Message-ID <[email protected]>
merged.

Bruce

In message: [meta-virtualization][PATCH] test_vdkr_registry: fix test_image_requires_subcommand
on 27/04/2026 Tim Orling via lists.yoctoproject.org wrote:

> vdkr.run() merges stderr into stdout (see conftest.py), so the
> error message ends up in result.stdout even though the script
> writes it to stderr (>&2).
> 
> Signed-off-by: Tim Orling <[email protected]>
> ---
>  tests/test_vdkr_registry.py | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/test_vdkr_registry.py b/tests/test_vdkr_registry.py
> index 9073444d..72a27169 100644
> --- a/tests/test_vdkr_registry.py
> +++ b/tests/test_vdkr_registry.py
> @@ -259,7 +259,11 @@ class TestImageCompoundCommands:
>          """Test that 'image' without subcommand shows error."""
>          result = self.vdkr.run("image", check=False)
>          assert result.returncode != 0
> -        assert "subcommand" in result.stderr.lower() or "requires" in result.stderr.lower()
> +        # vdkr.run() merges stderr into stdout (see conftest.py), so the
> +        # error message ends up in result.stdout even though the script
> +        # writes it to stderr (>&2).
> +        combined = (result.stdout + result.stderr).lower()
> +        assert "subcommand" in combined or "requires" in combined
>  
>  
>  class TestRegistryCLIOverride:
> -- 
> 2.50.1 (Apple Git-155)
> 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#9748): https://lists.yoctoproject.org/g/meta-virtualization/message/9748
> Mute This Topic: https://lists.yoctoproject.org/mt/119041586/1050810
> Group Owner: [email protected]
> Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [[email protected]]
> -=-=-=-=-=-=-=-=-=-=-=-
>