]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commitdiff
test: Fix actual vs. expected in virtTestCompareFiles
authorLaine Stump <laine@laine.org>
Mon, 27 Apr 2015 17:43:06 +0000 (13:43 -0400)
committerLaine Stump <laine@laine.org>
Mon, 27 Apr 2015 19:12:23 +0000 (15:12 -0400)
Commit ca329299 added a utility function virtTestCompareFiles() to
eliminate repetitive code in several test programs. It unfortunately
calls virtTestDifference() with the arguments in the wrong order -
strcontent is the "actual" output gathered by the test rig, while
filecontent is the "expected", and virtTestDifference() wants expected
(filecontent) followed by actual (strcontent), but
virtTestCompareFiles() does the opposite, which can make the output a
bit confusing when there is a failure.

tests/testutils.c

index acb2ef106497771259e7abf6a1ece33779797390..6a8fe6a5324a2690c52bcc77749c8478f8eb8ce0 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * testutils.c: basic test utils
  *
- * Copyright (C) 2005-2014 Red Hat, Inc.
+ * Copyright (C) 2005-2015 Red Hat, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -641,7 +641,7 @@ virtTestCompareToFile(const char *strcontent,
                 goto failure;
             goto out;
         }
-        virtTestDifference(stderr, strcontent, filecontent);
+        virtTestDifference(stderr, filecontent, strcontent);
         goto failure;
     }