]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
tests: string: Remove pointless test for virStringListFreeCount
authorPeter Krempa <pkrempa@redhat.com>
Tue, 23 Mar 2021 17:33:26 +0000 (18:33 +0100)
committerPeter Krempa <pkrempa@redhat.com>
Mon, 12 Apr 2021 13:55:10 +0000 (15:55 +0200)
It's way more useful to run valgrind against the rest of the code than
this test to see whether virStringListFreeCount works. Remove the test.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
tests/virstringtest.c

index 82e8e2106f4c22dc446ca5962f961b4ae3f995f8..83b883524d1c245965b028b961f508602e6827b0 100644 (file)
@@ -392,24 +392,6 @@ testStringToDouble(const void *opaque)
     return ret;
 }
 
-/* The point of this test is to check whether all members of the array are
- * freed. The test has to be checked using valgrind. */
-static int
-testVirStringListFreeCount(const void *opaque G_GNUC_UNUSED)
-{
-    char **list;
-
-    list = g_new0(char *, 4);
-
-    list[0] = g_strdup("test1");
-    list[2] = g_strdup("test2");
-    list[3] = g_strdup("test3");
-
-    virStringListFreeCount(list, 4);
-
-    return 0;
-}
-
 
 struct testStripData {
     const char *string;
@@ -718,11 +700,6 @@ mymain(void)
                 NULL,
                 3.141592653589793238462643383279502884197169399375105);
 
-    /* test virStringListFreeCount */
-    if (virTestRun("virStringListFreeCount", testVirStringListFreeCount,
-                   NULL) < 0)
-        ret = -1;
-
 #define TEST_STRIP_IPV6_BRACKETS(str, res) \
     do { \
         struct testStripData stripData = { \