[PATCH 3/6] qemuxmlconftest: Check also output files for usage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Include also the output files in the validation of used files.

Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
 tests/qemuxmlconftest.c | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/tests/qemuxmlconftest.c b/tests/qemuxmlconftest.c
index c3acfb59d5..cb19e7b27d 100644
--- a/tests/qemuxmlconftest.c
+++ b/tests/qemuxmlconftest.c
@@ -633,6 +633,12 @@ testQemuConfXMLCommon(testQemuInfo *info,

     /* mark test case as used */
     ignore_value(g_hash_table_remove(info->conf->existingTestCases, info->infile));
+    if (info->outfile)
+        ignore_value(g_hash_table_remove(info->conf->existingTestCases, info->outfile));
+    if (info->errfile)
+        ignore_value(g_hash_table_remove(info->conf->existingTestCases, info->errfile));
+    if (info->out_xml_inactive)
+        ignore_value(g_hash_table_remove(info->conf->existingTestCases, info->out_xml_inactive));

     if (testQemuInfoInitArgs((testQemuInfo *) info) < 0)
         goto cleanup;
@@ -925,7 +931,7 @@ testConfXMLCheck(GHashTable *existingTestCases)
         if (ret == 0)
             fprintf(stderr, "\n");

-        fprintf(stderr, "unused input file: %s\n", (const char *) items[i].key);
+        fprintf(stderr, "unused file: %s\n", (const char *) items[i].key);
         ret = -1;
     }

@@ -949,7 +955,9 @@ testConfXMLEnumerate(GHashTable *existingTestCases)
         return -1;

     while ((rc = virDirRead(dir, &ent, abs_srcdir "/qemuxml2argvdata")) > 0) {
-        if (virStringHasSuffix(ent->d_name, ".xml")) {
+        if (virStringHasSuffix(ent->d_name, ".xml") ||
+            virStringHasSuffix(ent->d_name, ".args") ||
+            virStringHasSuffix(ent->d_name, ".err")) {
             g_hash_table_insert(existingTestCases,
                                 g_strdup_printf(abs_srcdir "/qemuxml2argvdata/%s", ent->d_name),
                                 NULL);
-- 
2.43.0
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux