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

[libvirt] [PATCH v2 02/19] Rename virtTestDifference to virTestDifference.



This function doesn't follow our convention of naming functions.
---
 tests/commandtest.c            | 16 ++++++++--------
 tests/esxutilstest.c           |  8 ++++----
 tests/jsontest.c               |  6 +++---
 tests/nwfilterebiptablestest.c | 14 +++++++-------
 tests/openvzutilstest.c        |  4 ++--
 tests/qemumonitortest.c        |  4 ++--
 tests/sockettest.c             |  2 +-
 tests/testutils.c              |  6 +++---
 tests/testutils.h              |  6 +++---
 tests/utiltest.c               |  2 +-
 tests/vboxsnapshotxmltest.c    |  2 +-
 tests/virbuftest.c             | 10 +++++-----
 tests/virfirewalltest.c        | 20 ++++++++++----------
 tests/virkmodtest.c            |  2 +-
 tests/virnetdevbandwidthtest.c |  6 +++---
 tests/virnetsockettest.c       |  2 +-
 tests/virshtest.c              |  2 +-
 tests/virstoragetest.c         |  2 +-
 18 files changed, 57 insertions(+), 57 deletions(-)

diff --git a/tests/commandtest.c b/tests/commandtest.c
index 38ec70a..f433ad7 100644
--- a/tests/commandtest.c
+++ b/tests/commandtest.c
@@ -98,7 +98,7 @@ static int checkoutput(const char *testname,
     }
 
     if (STRNEQ(expectlog, actuallog)) {
-        virtTestDifference(stderr, expectlog, actuallog);
+        virTestDifference(stderr, expectlog, actuallog);
         goto cleanup;
     }
 
@@ -504,7 +504,7 @@ static int test13(const void *unused ATTRIBUTE_UNUSED)
     cmd = NULL;
 
     if (STRNEQ(outactual, outexpect)) {
-        virtTestDifference(stderr, outexpect, outactual);
+        virTestDifference(stderr, outexpect, outactual);
         goto cleanup;
     }
 
@@ -566,15 +566,15 @@ static int test14(const void *unused ATTRIBUTE_UNUSED)
         goto cleanup;
 
     if (STRNEQ(outactual, outexpect)) {
-        virtTestDifference(stderr, outexpect, outactual);
+        virTestDifference(stderr, outexpect, outactual);
         goto cleanup;
     }
     if (STRNEQ(erractual, errexpect)) {
-        virtTestDifference(stderr, errexpect, erractual);
+        virTestDifference(stderr, errexpect, erractual);
         goto cleanup;
     }
     if (STRNEQ(jointactual, jointexpect)) {
-        virtTestDifference(stderr, jointexpect, jointactual);
+        virTestDifference(stderr, jointexpect, jointactual);
         goto cleanup;
     }
 
@@ -650,7 +650,7 @@ static int test16(const void *unused ATTRIBUTE_UNUSED)
     }
 
     if (STRNEQ(outactual, outexpect)) {
-        virtTestDifference(stderr, outexpect, outactual);
+        virTestDifference(stderr, outexpect, outactual);
         goto cleanup;
     }
 
@@ -880,12 +880,12 @@ static int test21(const void *unused ATTRIBUTE_UNUSED)
         printf("STDOUT:%s\nSTDERR:%s\n", NULLSTR(outbuf), NULLSTR(errbuf));
 
     if (STRNEQ(outbuf, outbufExpected)) {
-        virtTestDifference(stderr, outbufExpected, outbuf);
+        virTestDifference(stderr, outbufExpected, outbuf);
         goto cleanup;
     }
 
     if (STRNEQ(errbuf, errbufExpected)) {
-        virtTestDifference(stderr, errbufExpected, errbuf);
+        virTestDifference(stderr, errbufExpected, errbuf);
         goto cleanup;
     }
 
diff --git a/tests/esxutilstest.c b/tests/esxutilstest.c
index ff34a7f..ab7f583 100644
--- a/tests/esxutilstest.c
+++ b/tests/esxutilstest.c
@@ -58,18 +58,18 @@ testParseDatastorePath(const void *data ATTRIBUTE_UNUSED)
             continue;
 
         if (STRNEQ(paths[i].datastoreName, datastoreName)) {
-            virtTestDifference(stderr, paths[i].datastoreName, datastoreName);
+            virTestDifference(stderr, paths[i].datastoreName, datastoreName);
             goto failure;
         }
 
         if (STRNEQ(paths[i].directoryName, directoryName)) {
-            virtTestDifference(stderr, paths[i].directoryName, directoryName);
+            virTestDifference(stderr, paths[i].directoryName, directoryName);
             goto failure;
         }
 
         if (STRNEQ(paths[i].directoryAndFileName, directoryAndFileName)) {
-            virtTestDifference(stderr, paths[i].directoryAndFileName,
-                               directoryAndFileName);
+            virTestDifference(stderr, paths[i].directoryAndFileName,
+                              directoryAndFileName);
             goto failure;
         }
     }
diff --git a/tests/jsontest.c b/tests/jsontest.c
index 2a6fcde..8e44f1a 100644
--- a/tests/jsontest.c
+++ b/tests/jsontest.c
@@ -105,7 +105,7 @@ testJSONAddRemove(const void *data)
         goto cleanup;
     }
     if (STRNEQ(info->expect, result)) {
-        virtTestDifference(stderr, info->expect, result);
+        virTestDifference(stderr, info->expect, result);
         goto cleanup;
     }
     ret = 0;
@@ -270,7 +270,7 @@ testJSONCopy(const void *data)
 
     if (STRNEQ(result, resultCopy)) {
         if (virTestGetVerbose())
-            virtTestDifference(stderr, result, resultCopy);
+            virTestDifference(stderr, result, resultCopy);
         ret = -1;
         goto cleanup;
     }
@@ -296,7 +296,7 @@ testJSONCopy(const void *data)
 
     if (STRNEQ(result, resultCopy)) {
         if (virTestGetVerbose())
-            virtTestDifference(stderr, result, resultCopy);
+            virTestDifference(stderr, result, resultCopy);
         ret = -1;
         goto cleanup;
     }
diff --git a/tests/nwfilterebiptablestest.c b/tests/nwfilterebiptablestest.c
index 0e7b694..84e5ed2 100644
--- a/tests/nwfilterebiptablestest.c
+++ b/tests/nwfilterebiptablestest.c
@@ -116,7 +116,7 @@ testNWFilterEBIPTablesAllTeardown(const void *opaque ATTRIBUTE_UNUSED)
     virtTestClearCommandPath(actual);
 
     if (STRNEQ_NULLABLE(actual, expected)) {
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -187,7 +187,7 @@ testNWFilterEBIPTablesTearOldRules(const void *opaque ATTRIBUTE_UNUSED)
     virtTestClearCommandPath(actual);
 
     if (STRNEQ_NULLABLE(actual, expected)) {
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -236,7 +236,7 @@ testNWFilterEBIPTablesRemoveBasicRules(const void *opaque ATTRIBUTE_UNUSED)
     virtTestClearCommandPath(actual);
 
     if (STRNEQ_NULLABLE(actual, expected)) {
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -270,7 +270,7 @@ testNWFilterEBIPTablesTearNewRules(const void *opaque ATTRIBUTE_UNUSED)
     virtTestClearCommandPath(actual);
 
     if (STRNEQ_NULLABLE(actual, expected)) {
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -342,7 +342,7 @@ testNWFilterEBIPTablesApplyBasicRules(const void *opaque ATTRIBUTE_UNUSED)
     virtTestClearCommandPath(actual);
 
     if (STRNEQ_NULLABLE(actual, expected)) {
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -432,7 +432,7 @@ testNWFilterEBIPTablesApplyDHCPOnlyRules(const void *opaque ATTRIBUTE_UNUSED)
     virtTestClearCommandPath(actual);
 
     if (STRNEQ_NULLABLE(actual, expected)) {
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -505,7 +505,7 @@ testNWFilterEBIPTablesApplyDropAllRules(const void *opaque ATTRIBUTE_UNUSED)
     virtTestClearCommandPath(actual);
 
     if (STRNEQ_NULLABLE(actual, expected)) {
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
diff --git a/tests/openvzutilstest.c b/tests/openvzutilstest.c
index fc5bee0..c7336e7 100644
--- a/tests/openvzutilstest.c
+++ b/tests/openvzutilstest.c
@@ -55,7 +55,7 @@ testReadConfigParam(const void *data ATTRIBUTE_UNUSED)
             continue;
 
         if (STRNEQ(configParams[i].value, value)) {
-            virtTestDifference(stderr, configParams[i].value, value);
+            virTestDifference(stderr, configParams[i].value, value);
             goto cleanup;
         }
     }
@@ -121,7 +121,7 @@ testReadNetworkConf(const void *data ATTRIBUTE_UNUSED)
     }
 
     if (STRNEQ(expected, actual)) {
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
diff --git a/tests/qemumonitortest.c b/tests/qemumonitortest.c
index 5eb3f78..48f7ad2 100644
--- a/tests/qemumonitortest.c
+++ b/tests/qemumonitortest.c
@@ -52,7 +52,7 @@ static int testEscapeArg(const void *data ATTRIBUTE_UNUSED)
             return -1;
         }
         if (STRNEQ(escapeStrings[i].escaped, escaped)) {
-            virtTestDifference(stderr, escapeStrings[i].escaped, escaped);
+            virTestDifference(stderr, escapeStrings[i].escaped, escaped);
             VIR_FREE(escaped);
             return -1;
         }
@@ -77,7 +77,7 @@ static int testUnescapeArg(const void *data ATTRIBUTE_UNUSED)
             return -1;
         }
         if (STRNEQ(escapeStrings[i].unescaped, unescaped)) {
-            virtTestDifference(stderr, escapeStrings[i].unescaped, unescaped);
+            virTestDifference(stderr, escapeStrings[i].unescaped, unescaped);
             VIR_FREE(unescaped);
             return -1;
         }
diff --git a/tests/sockettest.c b/tests/sockettest.c
index 0094a57..23b98c4 100644
--- a/tests/sockettest.c
+++ b/tests/sockettest.c
@@ -53,7 +53,7 @@ static int testFormat(virSocketAddr *addr, const char *addrstr, bool pass)
         return pass ? -1 : 0;
 
     if (STRNEQ(newaddrstr, addrstr)) {
-        virtTestDifference(stderr, newaddrstr, addrstr);
+        virTestDifference(stderr, newaddrstr, addrstr);
         VIR_FREE(newaddrstr);
         return pass ? -1 : 0;
     } else {
diff --git a/tests/testutils.c b/tests/testutils.c
index 23e8b13..c95df5d 100644
--- a/tests/testutils.c
+++ b/tests/testutils.c
@@ -607,9 +607,9 @@ virtTestDifferenceFullNoRegenerate(FILE *stream,
  * first and last characters at which differences occur
  */
 int
-virtTestDifference(FILE *stream,
-                   const char *expect,
-                   const char *actual)
+virTestDifference(FILE *stream,
+                  const char *expect,
+                  const char *actual)
 {
     return virtTestDifferenceFullNoRegenerate(stream,
                                               expect, NULL,
diff --git a/tests/testutils.h b/tests/testutils.h
index 47c64b0..6a8eeb9 100644
--- a/tests/testutils.h
+++ b/tests/testutils.h
@@ -56,9 +56,9 @@ int virtTestCaptureProgramOutput(const char *const argv[], char **buf, int maxle
 
 void virtTestClearCommandPath(char *cmdset);
 
-int virtTestDifference(FILE *stream,
-                       const char *expect,
-                       const char *actual);
+int virTestDifference(FILE *stream,
+                      const char *expect,
+                      const char *actual);
 int virtTestDifferenceFull(FILE *stream,
                            const char *expect,
                            const char *expectName,
diff --git a/tests/utiltest.c b/tests/utiltest.c
index 68cb178..7172cc3 100644
--- a/tests/utiltest.c
+++ b/tests/utiltest.c
@@ -52,7 +52,7 @@ testIndexToDiskName(const void *data ATTRIBUTE_UNUSED)
         diskName = virIndexToDiskName(i, "sd");
 
         if (STRNEQ(diskNames[i], diskName)) {
-            virtTestDifference(stderr, diskNames[i], diskName);
+            virTestDifference(stderr, diskNames[i], diskName);
             VIR_FREE(diskName);
 
             return -1;
diff --git a/tests/vboxsnapshotxmltest.c b/tests/vboxsnapshotxmltest.c
index 5457bc4..5e040bd 100644
--- a/tests/vboxsnapshotxmltest.c
+++ b/tests/vboxsnapshotxmltest.c
@@ -85,7 +85,7 @@ testCompareXMLtoXMLFiles(const char *xml)
         goto cleanup;
 
     if (STRNEQ(actual, xmlData)) {
-        virtTestDifference(stderr, xmlData, actual);
+        virTestDifference(stderr, xmlData, actual);
         goto cleanup;
     }
 
diff --git a/tests/virbuftest.c b/tests/virbuftest.c
index 1e4ce18..22407ab 100644
--- a/tests/virbuftest.c
+++ b/tests/virbuftest.c
@@ -138,7 +138,7 @@ static int testBufAutoIndent(const void *data ATTRIBUTE_UNUSED)
 
     result = virBufferContentAndReset(buf);
     if (!result || STRNEQ(result, expected)) {
-        virtTestDifference(stderr, expected, result);
+        virTestDifference(stderr, expected, result);
         ret = -1;
     }
     VIR_FREE(result);
@@ -175,7 +175,7 @@ static int testBufTrim(const void *data ATTRIBUTE_UNUSED)
 
     result = virBufferContentAndReset(buf);
     if (!result || STRNEQ(result, expected)) {
-        virtTestDifference(stderr, expected, result);
+        virTestDifference(stderr, expected, result);
         goto cleanup;
     }
 
@@ -292,7 +292,7 @@ static int testBufAddBuffer(const void *data ATTRIBUTE_UNUSED)
 
     result = virBufferContentAndReset(&buf1);
     if (STRNEQ_NULLABLE(result, expected)) {
-        virtTestDifference(stderr, expected, result);
+        virTestDifference(stderr, expected, result);
         goto cleanup;
     }
 
@@ -330,7 +330,7 @@ testBufAddStr(const void *opaque ATTRIBUTE_UNUSED)
 
     if (STRNEQ_NULLABLE(actual, data->expect)) {
         VIR_TEST_DEBUG("testBufAddStr(): Strings don't match:\n");
-        virtTestDifference(stderr, data->expect, actual);
+        virTestDifference(stderr, data->expect, actual);
         goto cleanup;
     }
 
@@ -363,7 +363,7 @@ testBufEscapeStr(const void *opaque ATTRIBUTE_UNUSED)
 
     if (STRNEQ_NULLABLE(actual, data->expect)) {
         VIR_TEST_DEBUG("testBufEscapeStr(): Strings don't match:\n");
-        virtTestDifference(stderr, data->expect, actual);
+        virTestDifference(stderr, data->expect, actual);
         goto cleanup;
     }
 
diff --git a/tests/virfirewalltest.c b/tests/virfirewalltest.c
index 68c2dc5..338f714 100644
--- a/tests/virfirewalltest.c
+++ b/tests/virfirewalltest.c
@@ -241,7 +241,7 @@ testFirewallSingleGroup(const void *opaque)
 
     if (STRNEQ_NULLABLE(expected, actual)) {
         fprintf(stderr, "Unexected command execution\n");
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -307,7 +307,7 @@ testFirewallRemoveRule(const void *opaque)
 
     if (STRNEQ_NULLABLE(expected, actual)) {
         fprintf(stderr, "Unexected command execution\n");
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -380,7 +380,7 @@ testFirewallManyGroups(const void *opaque ATTRIBUTE_UNUSED)
 
     if (STRNEQ_NULLABLE(expected, actual)) {
         fprintf(stderr, "Unexected command execution\n");
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -476,7 +476,7 @@ testFirewallIgnoreFailGroup(const void *opaque ATTRIBUTE_UNUSED)
 
     if (STRNEQ_NULLABLE(expected, actual)) {
         fprintf(stderr, "Unexected command execution\n");
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -550,7 +550,7 @@ testFirewallIgnoreFailRule(const void *opaque ATTRIBUTE_UNUSED)
 
     if (STRNEQ_NULLABLE(expected, actual)) {
         fprintf(stderr, "Unexected command execution\n");
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -621,7 +621,7 @@ testFirewallNoRollback(const void *opaque ATTRIBUTE_UNUSED)
 
     if (STRNEQ_NULLABLE(expected, actual)) {
         fprintf(stderr, "Unexected command execution\n");
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -711,7 +711,7 @@ testFirewallSingleRollback(const void *opaque ATTRIBUTE_UNUSED)
 
     if (STRNEQ_NULLABLE(expected, actual)) {
         fprintf(stderr, "Unexected command execution\n");
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -804,7 +804,7 @@ testFirewallManyRollback(const void *opaque ATTRIBUTE_UNUSED)
 
     if (STRNEQ_NULLABLE(expected, actual)) {
         fprintf(stderr, "Unexected command execution\n");
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -927,7 +927,7 @@ testFirewallChainedRollback(const void *opaque ATTRIBUTE_UNUSED)
 
     if (STRNEQ_NULLABLE(expected, actual)) {
         fprintf(stderr, "Unexected command execution\n");
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
@@ -1110,7 +1110,7 @@ testFirewallQuery(const void *opaque ATTRIBUTE_UNUSED)
 
     if (STRNEQ_NULLABLE(expected, actual)) {
         fprintf(stderr, "Unexected command execution\n");
-        virtTestDifference(stderr, expected, actual);
+        virTestDifference(stderr, expected, actual);
         goto cleanup;
     }
 
diff --git a/tests/virkmodtest.c b/tests/virkmodtest.c
index b86cf02..b98edf9 100644
--- a/tests/virkmodtest.c
+++ b/tests/virkmodtest.c
@@ -74,7 +74,7 @@ checkOutput(virBufferPtr buf, const char *exp_cmd)
     }
 
     if (STRNEQ(exp_cmd, actual_cmd)) {
-        virtTestDifference(stderr, exp_cmd, actual_cmd);
+        virTestDifference(stderr, exp_cmd, actual_cmd);
         goto cleanup;
     }
 
diff --git a/tests/virnetdevbandwidthtest.c b/tests/virnetdevbandwidthtest.c
index 14eced4..49283fe 100644
--- a/tests/virnetdevbandwidthtest.c
+++ b/tests/virnetdevbandwidthtest.c
@@ -95,9 +95,9 @@ testVirNetDevBandwidthSet(const void *data)
     }
 
     if (STRNEQ_NULLABLE(info->exp_cmd, actual_cmd)) {
-        virtTestDifference(stderr,
-                           NULLSTR(info->exp_cmd),
-                           NULLSTR(actual_cmd));
+        virTestDifference(stderr,
+                          NULLSTR(info->exp_cmd),
+                          NULLSTR(actual_cmd));
         goto cleanup;
     }
 
diff --git a/tests/virnetsockettest.c b/tests/virnetsockettest.c
index 89965d5..bbecb3c 100644
--- a/tests/virnetsockettest.c
+++ b/tests/virnetsockettest.c
@@ -407,7 +407,7 @@ static int testSocketSSH(const void *opaque)
         buf[rv] = '\0';
 
         if (STRNEQ(buf, data->expectOut)) {
-            virtTestDifference(stderr, data->expectOut, buf);
+            virTestDifference(stderr, data->expectOut, buf);
             goto cleanup;
         }
 
diff --git a/tests/virshtest.c b/tests/virshtest.c
index b675873..f330aa8 100644
--- a/tests/virshtest.c
+++ b/tests/virshtest.c
@@ -72,7 +72,7 @@ testCompareOutputLit(const char *expectData,
         goto cleanup;
 
     if (STRNEQ(expectData, actualData)) {
-        virtTestDifference(stderr, expectData, actualData);
+        virTestDifference(stderr, expectData, actualData);
         goto cleanup;
     }
 
diff --git a/tests/virstoragetest.c b/tests/virstoragetest.c
index fc9a3da..01c1248 100644
--- a/tests/virstoragetest.c
+++ b/tests/virstoragetest.c
@@ -394,7 +394,7 @@ testStorageChain(const void *args)
             goto cleanup;
         }
         if (STRNEQ(expect, actual)) {
-            virtTestDifference(stderr, expect, actual);
+            virTestDifference(stderr, expect, actual);
             VIR_FREE(expect);
             VIR_FREE(actual);
             goto cleanup;
-- 
2.5.5


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