]> xenbits.xensource.com Git - libvirt.git/commitdiff
tests: lxcconf2xml: Drop XML blanking
authorCole Robinson <crobinso@redhat.com>
Tue, 16 Feb 2016 17:07:32 +0000 (12:07 -0500)
committerCole Robinson <crobinso@redhat.com>
Wed, 17 Feb 2016 16:07:21 +0000 (11:07 -0500)
Hardcode a UUID like we did for qemuargv2xml, so we can use standard
comparison helpers, which gives us VIR_TEST_REGENERATE_OUTPUT support

tests/lxcconf2xmltest.c

index fc071f751cfb95438579b077d50cd7890896d05e..83895cd4ad1c61dc564603395a31785d32e1ca1b 100644 (file)
 static virCapsPtr caps;
 static virDomainXMLOptionPtr xmlopt;
 
-static int
-blankProblemElements(char *data)
+static int testSanitizeDef(virDomainDefPtr vmdef)
 {
-    if (virtTestClearLineRegex("<uuid>([[:alnum:]]|-)+</uuid>", data) < 0)
+    /* Remove UUID randomness */
+    if (virUUIDParse("c7a5fdbd-edaf-9455-926a-d65c16db1809", vmdef->uuid) < 0)
         return -1;
     return 0;
 }
 
 static int
-testCompareXMLToConfigFiles(const char *xml,
+testCompareXMLToConfigFiles(const char *xmlfile,
                             const char *configfile,
                             bool expectError)
 {
     int ret = -1;
     char *config = NULL;
-    char *expectxml = NULL;
     char *actualxml = NULL;
     virDomainDefPtr vmdef = NULL;
 
@@ -40,26 +39,19 @@ testCompareXMLToConfigFiles(const char *xml,
         goto fail;
 
     if (vmdef) {
-        if (!(actualxml = virDomainDefFormat(vmdef, caps, 0)))
+        if (testSanitizeDef(vmdef) < 0)
             goto fail;
 
-        if (virtTestLoadFile(xml, &expectxml) < 0)
-            goto fail;
-
-        if (blankProblemElements(expectxml) < 0 ||
-            blankProblemElements(actualxml) < 0)
+        if (!(actualxml = virDomainDefFormat(vmdef, caps, 0)))
             goto fail;
 
-        if (STRNEQ(expectxml, actualxml)) {
-            virtTestDifferenceFull(stderr, expectxml, xml, actualxml, NULL);
+        if (virtTestCompareToFile(actualxml, xmlfile) < 0)
             goto fail;
-        }
     }
 
     ret = 0;
 
  fail:
-    VIR_FREE(expectxml);
     VIR_FREE(actualxml);
     VIR_FREE(config);
     virDomainDefFree(vmdef);