summary refs log tree commit diff stats
path: root/tests/functional/test_virtio_gpu.py
diff options
context:
space:
mode:
authorDaniel P. Berrangé <berrange@redhat.com>2024-11-21 16:57:40 +0000
committerAlex Bennée <alex.bennee@linaro.org>2024-11-25 10:13:41 +0000
commit7699e37b2ac32ba4b905b4b22024e93f33337ce7 (patch)
tree4e012463c1b61dccd0d94337cdcd0412a487eede /tests/functional/test_virtio_gpu.py
parent6f0942b723df9441fe3304e8ab6d87bb17f88a1e (diff)
downloadfocaccia-qemu-7699e37b2ac32ba4b905b4b22024e93f33337ce7.tar.gz
focaccia-qemu-7699e37b2ac32ba4b905b4b22024e93f33337ce7.zip
tests/functional: don't try to wait for the empty string
Telling exec_command_wand_wait_for_pattern to wait for the empty
string does not make any conceptual sense, as a check for empty
string will always succeed. It makes even less sense when followed
by a call to wait_for_console_pattern() with a real match.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Message-Id: <20241121154218.1423005-14-berrange@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20241121165806.476008-14-alex.bennee@linaro.org>
Diffstat (limited to 'tests/functional/test_virtio_gpu.py')
-rwxr-xr-xtests/functional/test_virtio_gpu.py3
1 files changed, 1 insertions, 2 deletions
diff --git a/tests/functional/test_virtio_gpu.py b/tests/functional/test_virtio_gpu.py
index 441cbdcf2d..d5027487ac 100755
--- a/tests/functional/test_virtio_gpu.py
+++ b/tests/functional/test_virtio_gpu.py
@@ -80,9 +80,8 @@ class VirtioGPUx86(QemuSystemTest):
 
         self.wait_for_console_pattern("as init process")
         exec_command_and_wait_for_pattern(
-            self, "/usr/sbin/modprobe virtio_gpu", ""
+            self, "/usr/sbin/modprobe virtio_gpu", "features: +virgl +edid"
         )
-        self.wait_for_console_pattern("features: +virgl +edid")
 
     def test_vhost_user_vga_virgl(self):
         # FIXME: should check presence of vhost-user-gpu, virgl, memfd etc