fix non deterministic test
authorSchanzenbach, Martin <mschanzenbach@posteo.de>
Fri, 11 Oct 2019 17:11:06 +0000 (19:11 +0200)
committerSchanzenbach, Martin <mschanzenbach@posteo.de>
Fri, 11 Oct 2019 17:11:06 +0000 (19:11 +0200)
src/peerstore/test_peerstore_api_data.conf
src/peerstore/test_peerstore_api_sync.c

index 3ebda50eb2d1464a0010c080b4369c8d6811c949..614d0cf5b66c9d861d5ef55757e50754db0c2e05 100644 (file)
@@ -5,8 +5,9 @@ GNUNET_TEST_HOME = $GNUNET_TMP/test-gnunet-peerstore
 START_ON_DEMAND = YES
 BINARY = gnunet-service-peerstore
 UNIXPATH = $GNUNET_TMP/gnunet-service-peerstore.sock
-HOME = $SERVICEHOME
 DATABASE = sqlite
+UNIX_MATCH_UID = NO
+UNIX_MATCH_GID = YES
 #PREFIX = xterm -e gdb --args 
 
 [peerstore-sqlite]
index b2ac860b754dae2022e222aa84f8256e076e0a59..503cb2f1e485eaefe584bec4cf760cc4641df74f 100644 (file)
@@ -109,7 +109,6 @@ iterate_cb (void *cls,
 static void
 test_cont (void *cls)
 {
-  h = GNUNET_PEERSTORE_connect (cfg);
   GNUNET_PEERSTORE_iterate (h,
                             subsystem,
                             &pid, key,
@@ -117,6 +116,19 @@ test_cont (void *cls)
                             NULL);
 }
 
+static void
+store_cont (void *cls, int success)
+{
+  ok = success;
+  /* We need to wait a little bit to give the disconnect
+     a chance to actually finish the operation; otherwise,
+     the test may fail non-deterministically if the new
+     connection is faster than the cleanup routine of the
+     old one. */
+  GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_SECONDS,
+                                &test_cont,
+                                NULL);
+}
 
 /**
  * Actually run the test.
@@ -132,18 +144,7 @@ test1 ()
                           val, strlen (val) + 1,
                           GNUNET_TIME_UNIT_FOREVER_ABS,
                           GNUNET_PEERSTORE_STOREOPTION_REPLACE,
-                          NULL, NULL);
-  GNUNET_PEERSTORE_disconnect (h,
-                               GNUNET_YES);
-  h = NULL;
-  /* We need to wait a little bit to give the disconnect
-     a chance to actually finish the operation; otherwise,
-     the test may fail non-deterministically if the new
-     connection is faster than the cleanup routine of the
-     old one. */
-  GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_SECONDS,
-                                &test_cont,
-                                NULL);
+                          &store_cont, NULL);
 }
 
 
@@ -171,7 +172,7 @@ main (int argc, char *argv[])
   if (0 !=
       GNUNET_TESTING_service_run ("test-gnunet-peerstore-sync",
                                   "peerstore",
-                                  "test_peerstore_api_data.conf",
+                                  "peerstore.conf",
                                   &run, NULL))
     return 1;
   if (0 != ok)