On 1/27/20 2:05 PM, Thomas Huth wrote:
On 21/01/2020 00.51, Philippe Mathieu-Daudé wrote:
Avocado tags are handy to automatically select tests matching
the tags. Since this test also runs U-Boot, tag it.
We can run all the tests using U-Boot as once with:
$ avocado --show=app run -t u-boot tests/acceptance/
JOB LOG : avocado/job-results/job-2020-01-21T00.16-ee9344e/job.log
(1/3)
tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_arm_emcraft_sf2:
PASS (16.59 s)
(2/3)
tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_arm_raspi2_uboot:
PASS (0.47 s)
(3/3)
tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_aarch64_raspi3_uboot:
PASS (2.43 s)
RESULTS : PASS 3 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 |
CANCEL 0
JOB TIME : 19.78 s
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org>
---
tests/acceptance/boot_linux_console.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/acceptance/boot_linux_console.py
b/tests/acceptance/boot_linux_console.py
index 22b360118d..4a4cf9d0ea 100644
--- a/tests/acceptance/boot_linux_console.py
+++ b/tests/acceptance/boot_linux_console.py
@@ -305,6 +305,7 @@ class BootLinuxConsole(Test):
:avocado: tags=arch:arm
:avocado: tags=machine:emcraft-sf2
:avocado: tags=endian:little
+ :avocado: tags=u-boot
"""
uboot_url = ('https://raw.githubusercontent.com/'
'Subbaraya-Sundeep/qemu-test-binaries/'
We should maybe move that test to another file, too, since it is not
using Linux. Or should we maybe rename boot_linux_console.py to simply
boot_console.py ?
Thinking aloud: maybe create a directory 'boot' (or 'boot_tests') and
group tests in files according to the 'arch' tag.
Example:
[wainersm@localhost boot]$ tree
.
└── arm.py
0 directories, 1 file
[wainersm@localhost boot]$ cat arm.py
from avocado_qemu import Test
class BootArmTest(Test):
"""
:avocado: tags=arch:arm
"""
def test_linux_virt()
"""
:avocado: tags=machine:virt
:avocado: tags=linux-boot
"""
# Test here
pass
def test_uboot_emcraft_sf2()
"""
:avocado: tags=machine:emcraft-sf2
:avocado: tags=endian:little
:avocado: tags=u-boot
"""
# Test here
pass
--------------
Back to this patch...
Reviewed-by: Wainer dos Santos Moschetta <waine...@redhat.com>
Thomas