[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 5/7] Acceptance Tests: set machine type
From: |
Cleber Rosa |
Subject: |
[Qemu-devel] [PATCH v2 5/7] Acceptance Tests: set machine type |
Date: |
Tue, 9 Oct 2018 19:26:05 -0400 |
By setting the machine type, even if it's the one that will be picked
based on the arch, it's possible to run the same tests with targets
that require a machine type (in addition to those that don't).
Given that only boot_linux_console.py contains code specific to x86_64
(an explicit reference to the kernel image that will be used) the
other tests can be used to test different targets:
$ avocado run -p arch=aarch64 --filter-by-tags='-x86_64' -- tests/acceptance/
Eventually, to reduce boiler plate code, the idea is to concentrate
the basic configuration (arch, machine, console) in either another
utility method, or make that happen by default. This is of course the
subject of a future discussion.
Signed-off-by: Cleber Rosa <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
---
tests/acceptance/boot_linux_console.py | 3 ++-
tests/acceptance/version.py | 2 ++
tests/acceptance/vnc.py | 5 +++++
3 files changed, 9 insertions(+), 1 deletion(-)
diff --git a/tests/acceptance/boot_linux_console.py
b/tests/acceptance/boot_linux_console.py
index 98324f7591..58032f971c 100644
--- a/tests/acceptance/boot_linux_console.py
+++ b/tests/acceptance/boot_linux_console.py
@@ -30,7 +30,8 @@ class BootLinuxConsole(Test):
kernel_hash = '238e083e114c48200f80d889f7e32eeb2793e02a'
kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash)
- self.vm.set_machine('pc')
+ self.vm.set_arch(self.arch)
+ self.vm.set_machine()
self.vm.set_console()
kernel_command_line = 'console=ttyS0'
self.vm.add_args('-kernel', kernel_path,
diff --git a/tests/acceptance/version.py b/tests/acceptance/version.py
index 13b0a7440d..7a3a20945f 100644
--- a/tests/acceptance/version.py
+++ b/tests/acceptance/version.py
@@ -18,6 +18,8 @@ class Version(Test):
:avocado: tags=quick
"""
def test_qmp_human_info_version(self):
+ self.vm.set_arch(self.arch)
+ self.vm.set_machine()
self.vm.launch()
res = self.vm.command('human-monitor-command',
command_line='info version')
diff --git a/tests/acceptance/vnc.py b/tests/acceptance/vnc.py
index b1ef9d71b1..4a8a83025f 100644
--- a/tests/acceptance/vnc.py
+++ b/tests/acceptance/vnc.py
@@ -16,6 +16,11 @@ class Vnc(Test):
:avocado: enable
:avocado: tags=vnc,quick
"""
+ def setUp(self):
+ super(Vnc, self).setUp()
+ self.vm.set_arch(self.arch)
+ self.vm.set_machine()
+
def test_no_vnc(self):
self.vm.add_args('-nodefaults', '-S')
self.vm.launch()
--
2.17.1
- Re: [Qemu-devel] [PATCH v2 4/7] scripts/qemu.py: set predefined machine type based on arch, (continued)
- Re: [Qemu-devel] [PATCH v2 4/7] scripts/qemu.py: set predefined machine type based on arch, Peter Maydell, 2018/10/11
- Re: [Qemu-devel] [PATCH v2 4/7] scripts/qemu.py: set predefined machine type based on arch, Cleber Rosa, 2018/10/10
- Re: [Qemu-devel] [PATCH v2 4/7] scripts/qemu.py: set predefined machine type based on arch, Eduardo Habkost, 2018/10/10
- Re: [Qemu-devel] [PATCH v2 4/7] scripts/qemu.py: set predefined machine type based on arch, Cleber Rosa, 2018/10/11
- Re: [Qemu-devel] [PATCH v2 4/7] scripts/qemu.py: set predefined machine type based on arch, Eduardo Habkost, 2018/10/11
[Qemu-devel] [PATCH v2 6/7] Acceptance Tests: add variants definition for architectures, Cleber Rosa, 2018/10/09
[Qemu-devel] [PATCH v2 5/7] Acceptance Tests: set machine type,
Cleber Rosa <=
[Qemu-devel] [PATCH v2 2/7] Acceptance Tests: introduce arch parameter and attribute, Cleber Rosa, 2018/10/09
[Qemu-devel] [PATCH v2 7/7] Acceptance Tests: change the handling of tests for specific archs, Cleber Rosa, 2018/10/09