diff options
| author | Thomas Huth <thuth@redhat.com> | 2024-09-05 21:14:29 +0200 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2024-09-17 09:53:17 +0200 |
| commit | 0a8ecb41fd9a9fb67c1cc63a0e1eab3e940c6a35 (patch) | |
| tree | 802ffc40756217cc59565a4604b365031d652d05 | |
| parent | d822b883d7c5e0bbee4fa5ea1c14546e5ce16019 (diff) | |
| download | qemu-0a8ecb41fd9a9fb67c1cc63a0e1eab3e940c6a35.zip qemu-0a8ecb41fd9a9fb67c1cc63a0e1eab3e940c6a35.tar.gz qemu-0a8ecb41fd9a9fb67c1cc63a0e1eab3e940c6a35.tar.bz2 | |
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>
| -rw-r--r-- | tests/qtest/boot-order-test.c | 4 |
1 files changed, 2 insertions, 2 deletions
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 @@ static void test_a_boot_order(const char *machine, 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 const boot_order_test test_cases_pc[] = { 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) |