Re: [PATCH v5 23/44] tests/functional: Convert the riscv_opensbi avocado test into a standalone test

["Philippe Mathieu-Daudé" <philmd@linaro.org>]

On 30/8/24 15:38, Thomas Huth wrote:
> The avocado test defined test functions for both, riscv32 and riscv64.
> Since we can run the whole file with multiple targets in the new
> framework, we can now consolidate the functions so we have to only
> define one function per machine now.
> 
> However, we have to mark the "spike" test as flaky now since it sometimes
> fails when running the tests in parallel (the failure happens also with the
> Avocado test when the host is under load, so the problem is not related
> to this conversion to the functional framework).
> 
> Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   MAINTAINERS                            |  1 +
>   tests/avocado/riscv_opensbi.py         | 63 --------------------------
>   tests/functional/meson.build           |  8 ++++
>   tests/functional/test_riscv_opensbi.py | 40 ++++++++++++++++
>   4 files changed, 49 insertions(+), 63 deletions(-)
>   delete mode 100644 tests/avocado/riscv_opensbi.py
>   create mode 100755 tests/functional/test_riscv_opensbi.py

Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>