[libvirt] [PATCH 01/12] tests: qemuxml2xml: Break out testInfoSet*Paths

Cole Robinson crobinso at redhat.com
Mon Apr 1 16:47:17 UTC 2019


These will need to be separate to share testInfo with qemuxml2argv

Signed-off-by: Cole Robinson <crobinso at redhat.com>
---
 tests/qemuxml2xmltest.c | 25 +++++++++----------------
 1 file changed, 9 insertions(+), 16 deletions(-)

diff --git a/tests/qemuxml2xmltest.c b/tests/qemuxml2xmltest.c
index 83a0d1cf7b..2dfa9e628b 100644
--- a/tests/qemuxml2xmltest.c
+++ b/tests/qemuxml2xmltest.c
@@ -129,16 +129,11 @@ testInfoSetCommon(struct testInfo *info,
     return -1;
 }
 
-
 static int
-testInfoSet(struct testInfo *info,
-            const char *name,
-            int when,
-            int gic)
+testInfoSetPaths(struct testInfo *info,
+                 const char *name,
+                 int when)
 {
-    if (testInfoSetCommon(info, gic) < 0)
-        return -1;
-
     if (virAsprintf(&info->inName, "%s/qemuxml2argvdata/%s.xml",
                     abs_srcdir, name) < 0)
         goto error;
@@ -186,13 +181,9 @@ testInfoSet(struct testInfo *info,
 static const char *statusPath = abs_srcdir "/qemustatusxml2xmldata/";
 
 static int
-testInfoSetStatus(struct testInfo *info,
-                  const char *name,
-                  int gic)
+testInfoSetStatusPaths(struct testInfo *info,
+                       const char *name)
 {
-    if (testInfoSetCommon(info, gic) < 0)
-        return -1;
-
     if (virAsprintf(&info->inName, "%s%s-in.xml", statusPath, name) < 0 ||
         virAsprintf(&info->outActiveName, "%s%s-out.xml", statusPath, name) < 0)
         goto error;
@@ -236,7 +227,8 @@ mymain(void)
 
 # define DO_TEST_FULL(name, when, gic, ...) \
     do { \
-        if (testInfoSet(&info, name, when, gic) < 0) { \
+        if (testInfoSetCommon(&info, gic) < 0 || \
+            testInfoSetPaths(&info, name, when) < 0) { \
             VIR_TEST_DEBUG("Failed to generate test data for '%s'", name); \
             return -1; \
         } \
@@ -1241,7 +1233,8 @@ mymain(void)
 
 # define DO_TEST_STATUS(name) \
     do { \
-        if (testInfoSetStatus(&info, name, GIC_NONE) < 0) { \
+        if (testInfoSetCommon(&info, GIC_NONE) < 0 || \
+            testInfoSetStatusPaths(&info, name) < 0) { \
             VIR_TEST_DEBUG("Failed to generate status test data for '%s'", name); \
             return -1; \
         } \
-- 
2.21.0




More information about the libvir-list mailing list