Re: [PATCH v2 2/5] python: use avocado's "new" runner

[Beraldo Leal]

On Wed, Jan 19, 2022 at 02:39:13PM -0500, John Snow wrote:
> The old legacy runner no longer seems to work with output logging, so we
> can't see failure logs when a test case fails. The new runner doesn't
> (seem to) support Coverage.py yet, but seeing error output is a more
> important feature.
> 
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
>  python/avocado.cfg | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/python/avocado.cfg b/python/avocado.cfg
> index c7722e7ecd..a460420059 100644
> --- a/python/avocado.cfg
> +++ b/python/avocado.cfg
> @@ -1,5 +1,5 @@
>  [run]
> -test_runner = runner
> +test_runner = nrunner
>  
>  [simpletests]
>  # Don't show stdout/stderr in the test *summary*

Since Avocado 82, the new one is the default. So, you could remove the
the "[run]" section.

In any case:

Reviewed-by: Beraldo Leal <bleal@redhat.com>

--
Beraldo