Re: [PATCH 9/9] tests/functional: fix some tests that require HMP

[Thomas Huth <th.huth+qemu@posteo.eu>]

Am Sun, 06 Sep 2026 12:42:40 +0400
schrieb Marc-André Lureau <marcandre.lureau@redhat.com>:

> Those tests use "info registers" command and connot be converted

s/connot/cannot/

> easily, for now skip them
> 
> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
>  tests/functional/m68k/test_nextcube.py | 1 +
>  tests/functional/ppc/test_ppe42.py     | 1 +
>  2 files changed, 2 insertions(+)
> 
> diff --git a/tests/functional/m68k/test_nextcube.py b/tests/functional/m68k/test_nextcube.py
> index dc99962eb75c..b0c0b653a7d0 100755
> --- a/tests/functional/m68k/test_nextcube.py
> +++ b/tests/functional/m68k/test_nextcube.py
> @@ -27,6 +27,7 @@ def check_bootrom_framebuffer(self, screenshot_path):
>  
>          self.vm.add_args('-bios', rom_path)
>          self.vm.launch()
> +        self.skipTestIfNoHMP() # FIXME: QMP x-query-registers
>  
>          self.log.info('VM launched, waiting for display')
>          # Wait for the FPU test to finish, then the display is available, too:
> diff --git a/tests/functional/ppc/test_ppe42.py b/tests/functional/ppc/test_ppe42.py
> index 53958a7938d1..a9c154f67328 100755
> --- a/tests/functional/ppc/test_ppe42.py
> +++ b/tests/functional/ppc/test_ppe42.py
> @@ -72,6 +72,7 @@ def test_ppe42_instructions(self):
>          self.vm.add_args('-device', 'loader,addr=0xfff80040,cpu-num=0')
>          self.vm.add_args('-action', 'panic=pause')
>          self.vm.launch()
> +        self.skipTestIfNoHMP() # FIXME: QMP x-query-registers
>          self._wait_pass_fail(self.timeout)

Why don't you use these as a decorator (with @) instead?

 Thomas