[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 5/7] Acceptance Tests: set machine type
From: |
Cleber Rosa |
Subject: |
[Qemu-devel] [PATCH 5/7] Acceptance Tests: set machine type |
Date: |
Thu, 4 Oct 2018 11:14:27 -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>
---
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
- [Qemu-devel] [PATCH 0/7] Acceptance Tests: basic architecture support, Cleber Rosa, 2018/10/04
- [Qemu-devel] [PATCH 1/7] Acceptance Tests: improve docstring on pick_default_qemu_bin(), Cleber Rosa, 2018/10/04
- [Qemu-devel] [PATCH 2/7] Acceptance Tests: introduce arch parameter and attribute, Cleber Rosa, 2018/10/04
- [Qemu-devel] [PATCH 3/7] scripts/qemu.py: add method and private attribute for arch, Cleber Rosa, 2018/10/04
- [Qemu-devel] [PATCH 4/7] scripts/qemu.py: set predefined machine type based on arch, Cleber Rosa, 2018/10/04
- [Qemu-devel] [PATCH 5/7] Acceptance Tests: set machine type,
Cleber Rosa <=
- [Qemu-devel] [PATCH 6/7] Acceptance Tests: add variants definition for architectures, Cleber Rosa, 2018/10/04
[Qemu-devel] [PATCH 7/7] Acceptance Tests: change the handling of tests for specific archs, Cleber Rosa, 2018/10/04