commit 0a8ecb41fd9a9fb67c1cc63a0e1eab3e940c6a35
author Thomas Huth <thuth@redhat.com> Thu Sep 05 21:14:29 2024 +0200
committer Thomas Huth <thuth@redhat.com> Tue Sep 17 09:53:17 2024 +0200
tree 802ffc40756217cc59565a4604b365031d652d05
parent d822b883d7c5e0bbee4fa5ea1c14546e5ce16019

tests/qtest/boot-order-test: Make the machine name mandatory in this test

Let's make sure that we always pass a machine name to the test_boot_orders()
function, so we can check whether the machine is available in the binary
and skip the test in case it is not included in the build.

Message-ID: <20240905191434.694440-4-thuth@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>

tests/qtest/boot-order-test.c

diff --git a/tests/qtest/boot-order-test.c b/tests/qtest/boot-order-test.c
index 8f2b6ef..c67b8cf 100644
--- a/tests/qtest/boot-order-test.c
+++ b/tests/qtest/boot-order-test.c
@@ -31,7 +31,7 @@
     uint64_t actual;
     QTestState *qts;
 
-    if (machine && !qtest_has_machine(machine)) {
+    if (!qtest_has_machine(machine)) {
         g_test_skip("Machine is not available");
         return;
     }
@@ -107,7 +107,7 @@
 
 static void test_pc_boot_order(void)
 {
-    test_boot_orders(NULL, read_boot_order_pc, test_cases_pc);
+    test_boot_orders("pc", read_boot_order_pc, test_cases_pc);
 }
 
 static uint64_t read_boot_order_pmac(QTestState *qts)