Re: [PATCH] tests/avocado: Move common Avocado tags to class

[Thomas Huth]

On 19/07/2024 17.13, Philippe Mathieu-Daudé wrote:
> When Avocado tags apply to all tests in a class,
> we can define them once in the class: they will
> be applied to all test methods.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   tests/avocado/boot_linux.py           |  2 +-
>   tests/avocado/boot_xen.py             | 17 +----------------
>   tests/avocado/machine_aarch64_virt.py | 10 ++++------
>   3 files changed, 6 insertions(+), 23 deletions(-)
>
> diff --git a/tests/avocado/boot_linux.py b/tests/avocado/boot_linux.py
> index cdce4cbcba..df6cf209ef 100644
> --- a/tests/avocado/boot_linux.py
> +++ b/tests/avocado/boot_linux.py
> @@ -64,12 +64,12 @@ def test_pc_q35_kvm(self):
>   class BootLinuxAarch64(LinuxTest):
>       """
>       :avocado: tags=arch:aarch64
> -    :avocado: tags=machine:virt
>       """
>       timeout = 720
>   
>       def test_virt_kvm(self):
>           """
> +        :avocado: tags=machine:virt
>           :avocado: tags=accel:kvm
>           :avocado: tags=cpu:host
>           """

This change seems to contradict your patch description?

 Thomas