[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [Qemu-devel] [PATCH 2/2] BootLinuxConsoleTest: Test the S
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-arm] [Qemu-devel] [PATCH 2/2] BootLinuxConsoleTest: Test the SmartFusion2 board |
Date: |
Wed, 5 Jun 2019 17:43:23 -0300 |
On Wed, Jun 05, 2019 at 12:06:59PM -0400, Cleber Rosa wrote:
> On Tue, May 21, 2019 at 12:06:35AM +0200, Philippe Mathieu-Daudé wrote:
> > Similar to the x86_64/pc test, it boots a Linux kernel on an
> > Emcraft board and verify the serial is working.
> >
> > If ARM is a target being built, "make check-acceptance" will
> > automatically include this test by the use of the "arch:arm" tags.
> >
> > Alternatively, this test can be run using:
> >
> > $ avocado run -t arch:arm tests/acceptance
> > $ avocado run -t machine:emcraft-sf2 tests/acceptance
> >
>
> The pattern accepted as a tag currently doesn't include a dash, see:
>
>
> https://avocado-framework.readthedocs.io/en/68.0/ReferenceGuide.html#docstring-directives-rules
>
Why isn't this causing test jobs to fail?
This patch is already queued on python-next, should I remove it?
> My suggestion is to replace the dash for an underline. This was done
> on the s390 test:
>
> :avocado: tags=machine:s390_ccw_virtio
I would love if we stopped using the docstring to store
machine-readable data and used Python class attributes or
methods, like most Python APIs do. e.g.:
class MyTest(avocado.Test):
tags = {'machine':'s390-ccw-virtio'}
def test_my_feature(self):
...
I understand the Avocado runner currently tries to not run any
test code at the "list" step. But can we have a mode where it
will actually load the Python module and look at the class
objects directly, instead of trying to parse the Python code
without evaluating it?
> [...]
--
Eduardo