diff options
| author | Daniel P. Berrangé <berrange@redhat.com> | 2024-11-21 16:57:40 +0000 |
|---|---|---|
| committer | Alex Bennée <alex.bennee@linaro.org> | 2024-11-25 10:13:41 +0000 |
| commit | 7699e37b2ac32ba4b905b4b22024e93f33337ce7 (patch) | |
| tree | 4e012463c1b61dccd0d94337cdcd0412a487eede | |
| parent | 6f0942b723df9441fe3304e8ab6d87bb17f88a1e (diff) | |
| download | qemu-7699e37b2ac32ba4b905b4b22024e93f33337ce7.zip qemu-7699e37b2ac32ba4b905b4b22024e93f33337ce7.tar.gz qemu-7699e37b2ac32ba4b905b4b22024e93f33337ce7.tar.bz2 | |
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>
| -rwxr-xr-x | tests/functional/test_virtio_gpu.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/tests/functional/test_virtio_gpu.py b/tests/functional/test_virtio_gpu.py index 441cbdc..d502748 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 |