While the 'qemuxmlconftest' was able to load capability variants the
output file name didn't include the variant thus it was not possible to
test the same input file both on the default variant and on an explicit
variant.
Include the variant in the output file name and adjust two output file
names.
Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
... hvf-aarch64-virt-headless.aarch64-latest+hvf.args} | 0
...> hvf-aarch64-virt-headless.aarch64-latest+hvf.xml} | 0
... => hvf-x86_64-q35-headless.x86_64-latest+hvf.args} | 0
...l => hvf-x86_64-q35-headless.x86_64-latest+hvf.xml} | 0
tests/qemuxmlconftest.c | 10 +++++++---
tests/testutilsqemu.c | 6 +++---
6 files changed, 10 insertions(+), 6 deletions(-)
rename tests/qemuxmlconfdata/{hvf-aarch64-virt-headless.aarch64-latest.args => hvf-aarch64-virt-headless.aarch64-latest+hvf.args} (100%)
rename tests/qemuxmlconfdata/{hvf-aarch64-virt-headless.aarch64-latest.xml => hvf-aarch64-virt-headless.aarch64-latest+hvf.xml} (100%)
rename tests/qemuxmlconfdata/{hvf-x86_64-q35-headless.x86_64-latest.args => hvf-x86_64-q35-headless.x86_64-latest+hvf.args} (100%)
rename tests/qemuxmlconfdata/{hvf-x86_64-q35-headless.x86_64-latest.xml => hvf-x86_64-q35-headless.x86_64-latest+hvf.xml} (100%)
diff --git a/tests/qemuxmlconfdata/hvf-aarch64-virt-headless.aarch64-latest.args b/tests/qemuxmlconfdata/hvf-aarch64-virt-headless.aarch64-latest+hvf.args
similarity index 100%
rename from tests/qemuxmlconfdata/hvf-aarch64-virt-headless.aarch64-latest.args
rename to tests/qemuxmlconfdata/hvf-aarch64-virt-headless.aarch64-latest+hvf.args
diff --git a/tests/qemuxmlconfdata/hvf-aarch64-virt-headless.aarch64-latest.xml b/tests/qemuxmlconfdata/hvf-aarch64-virt-headless.aarch64-latest+hvf.xml
similarity index 100%
rename from tests/qemuxmlconfdata/hvf-aarch64-virt-headless.aarch64-latest.xml
rename to tests/qemuxmlconfdata/hvf-aarch64-virt-headless.aarch64-latest+hvf.xml
diff --git a/tests/qemuxmlconfdata/hvf-x86_64-q35-headless.x86_64-latest.args b/tests/qemuxmlconfdata/hvf-x86_64-q35-headless.x86_64-latest+hvf.args
similarity index 100%
rename from tests/qemuxmlconfdata/hvf-x86_64-q35-headless.x86_64-latest.args
rename to tests/qemuxmlconfdata/hvf-x86_64-q35-headless.x86_64-latest+hvf.args
diff --git a/tests/qemuxmlconfdata/hvf-x86_64-q35-headless.x86_64-latest.xml b/tests/qemuxmlconfdata/hvf-x86_64-q35-headless.x86_64-latest+hvf.xml
similarity index 100%
rename from tests/qemuxmlconfdata/hvf-x86_64-q35-headless.x86_64-latest.xml
rename to tests/qemuxmlconfdata/hvf-x86_64-q35-headless.x86_64-latest+hvf.xml
diff --git a/tests/qemuxmlconftest.c b/tests/qemuxmlconftest.c
index 8632434760..84eea4967a 100644
--- a/tests/qemuxmlconftest.c
+++ b/tests/qemuxmlconftest.c
@@ -1006,14 +1006,18 @@ testRun(const char *name,
va_end(ap);
info->infile = g_strdup_printf("%s/qemuxmlconfdata/%s.xml", abs_srcdir, info->name);
+
if (info->flags & (FLAG_EXPECT_FAILURE | FLAG_EXPECT_PARSE_ERROR)) {
- info->errfile = g_strdup_printf("%s/qemuxmlconfdata/%s%s.err", abs_srcdir, info->name, suffix);
+ info->errfile = g_strdup_printf("%s/qemuxmlconfdata/%s%s%s.err",
+ abs_srcdir, info->name, suffix, info->args.capsvariant);
} else {
- info->outfile = g_strdup_printf("%s/qemuxmlconfdata/%s%s.args", abs_srcdir, info->name, suffix);
+ info->outfile = g_strdup_printf("%s/qemuxmlconfdata/%s%s%s.args",
+ abs_srcdir, info->name, suffix, info->args.capsvariant);
}
if (!(info->flags & FLAG_EXPECT_PARSE_ERROR))
- info->out_xml_inactive = g_strdup_printf("%s/qemuxmlconfdata/%s%s.xml", abs_srcdir, info->name, suffix);
+ info->out_xml_inactive = g_strdup_printf("%s/qemuxmlconfdata/%s%s%s.xml",
+ abs_srcdir, info->name, suffix, info->args.capsvariant);
virTestRunLog(ret, name_parse, testXMLParse, info);
virTestRunLog(ret, name_xml, testCompareDef2XML, info);
diff --git a/tests/testutilsqemu.c b/tests/testutilsqemu.c
index 44011c2b36..20409a6535 100644
--- a/tests/testutilsqemu.c
+++ b/tests/testutilsqemu.c
@@ -754,6 +754,9 @@ testQemuInfoSetArgs(testQemuInfo *info,
if (info->args.invalidarg)
break;
}
+
+ if (!info->args.capsvariant)
+ info->args.capsvariant = "";
}
@@ -913,9 +916,6 @@ testQemuInfoInitArgs(testQemuInfo *info)
info->args.newargs = false;
- if (!info->args.capsvariant)
- info->args.capsvariant = "";
-
if (info->args.invalidarg) {
fprintf(stderr, "Invalid argument encountered by 'testQemuInfoSetArgs'\n");
return -1;
--
2.48.1
