From: Peter Krempa Date: Tue, 15 Aug 2023 12:43:20 +0000 (+0200) Subject: qemuxml2xmloutdata: Workaround wrong detection of 'disk-cdrom-empty-network-invalid... X-Git-Tag: v9.7.0-rc1~96 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=72a6c2ad9fd22b41de904e2c8eb8988c649b0e1d;p=thirdparty%2Flibvirt.git qemuxml2xmloutdata: Workaround wrong detection of 'disk-cdrom-empty-network-invalid' in virschematest The 'disk-cdrom-empty-network-invalid' is a special case were the input XML is invalid according to the schema, but after processing a valid XML is produced. This corner case doesn't play well with 'virschematest' which uses the file suffix to determine whether the file is invalid. Upcoming patch will change the 'virschematest' condition, which would start detecting this XML as invalid. Use the '-active'/'-inactive' suffix for the file, which is possible with qemuxml2xmltest so that an upcoming patch will not cause test failure. Signed-off-by: Peter Krempa Reviewed-by: Ján Tomko --- diff --git a/tests/qemuxml2xmloutdata/disk-cdrom-empty-network-invalid.x86_64-latest.xml b/tests/qemuxml2xmloutdata/disk-cdrom-empty-network-invalid-active.x86_64-latest.xml similarity index 100% rename from tests/qemuxml2xmloutdata/disk-cdrom-empty-network-invalid.x86_64-latest.xml rename to tests/qemuxml2xmloutdata/disk-cdrom-empty-network-invalid-active.x86_64-latest.xml diff --git a/tests/qemuxml2xmloutdata/disk-cdrom-empty-network-invalid-inactive.x86_64-latest.xml b/tests/qemuxml2xmloutdata/disk-cdrom-empty-network-invalid-inactive.x86_64-latest.xml new file mode 120000 index 0000000000..9733ff06a7 --- /dev/null +++ b/tests/qemuxml2xmloutdata/disk-cdrom-empty-network-invalid-inactive.x86_64-latest.xml @@ -0,0 +1 @@ +disk-cdrom-empty-network-invalid-active.x86_64-latest.xml \ No newline at end of file diff --git a/tests/qemuxml2xmltest.c b/tests/qemuxml2xmltest.c index 50e6860dfc..e9ab6d8a45 100644 --- a/tests/qemuxml2xmltest.c +++ b/tests/qemuxml2xmltest.c @@ -287,6 +287,9 @@ mymain(void) DO_TEST_NOCAPS("disk-aio"); DO_TEST_CAPS_LATEST("disk-aio-io_uring"); DO_TEST_NOCAPS("disk-cdrom"); + /* The 'disk-cdrom-empty-network-invalid' test case uses separate + * '-active'/'-inactive' output files to work around 'virschematest' + * thinking the output file is invalid XML */ DO_TEST_CAPS_LATEST("disk-cdrom-empty-network-invalid"); DO_TEST_CAPS_LATEST("disk-cdrom-network"); DO_TEST("disk-cdrom-bus-other", QEMU_CAPS_DEVICE_USB_STORAGE);