[GNUnet-SVN] r37611 - gnunet/src/hostlist

[gnunet]

Author: grothoff
Date: 2016-07-29 21:18:54 +0200 (Fri, 29 Jul 2016)
New Revision: 37611

Modified:
   gnunet/src/hostlist/Makefile.am
   gnunet/src/hostlist/test_gnunet_daemon_hostlist.c
   gnunet/src/hostlist/test_gnunet_daemon_hostlist_reconnect.c
Log:
convert tests to new transport API

Modified: gnunet/src/hostlist/Makefile.am
===================================================================
--- gnunet/src/hostlist/Makefile.am     2016-07-29 16:21:44 UTC (rev 37610)
+++ gnunet/src/hostlist/Makefile.am     2016-07-29 19:18:54 UTC (rev 37611)
@@ -59,17 +59,22 @@
  test_gnunet_daemon_hostlist \
  test_gnunet_daemon_hostlist_reconnect \
  test_gnunet_daemon_hostlist_learning
+else
+if HAVE_LIBCURL
+check_PROGRAMS = \
+ test_gnunet_daemon_hostlist \
+ test_gnunet_daemon_hostlist_reconnect \
+ test_gnunet_daemon_hostlist_learning
+endif
+endif
 
 if HAVE_MHD
 if ENABLE_TEST_RUN
 AM_TESTS_ENVIRONMENT=export 
GNUNET_PREFIX=$${GNUNET_PREFIX:address@hidden@};export 
PATH=$${GNUNET_PREFIX:address@hidden@}/bin:$$PATH;
 TESTS = \
- test_gnunet_daemon_hostlist \
- test_gnunet_daemon_hostlist_reconnect \
- test_gnunet_daemon_hostlist_learning
+ $(check_PROGRAMS)
 endif
 endif
-endif
 
 test_gnunet_daemon_hostlist_SOURCES = \
  test_gnunet_daemon_hostlist.c

Modified: gnunet/src/hostlist/test_gnunet_daemon_hostlist.c
===================================================================
--- gnunet/src/hostlist/test_gnunet_daemon_hostlist.c   2016-07-29 16:21:44 UTC 
(rev 37610)
+++ gnunet/src/hostlist/test_gnunet_daemon_hostlist.c   2016-07-29 19:18:54 UTC 
(rev 37611)
@@ -1,6 +1,6 @@
 /*
      This file is part of GNUnet
-     Copyright (C) 2009 GNUnet e.V.
+     Copyright (C) 2009, 2016 GNUnet e.V.
 
      GNUnet is free software; you can redistribute it and/or modify
      it under the terms of the GNU General Public License as published
@@ -26,6 +26,7 @@
 #include "gnunet_util_lib.h"
 #include "gnunet_arm_service.h"
 #include "gnunet_transport_service.h"
+#include "gnunet_transport_core_service.h"
 
 
 /**
@@ -40,7 +41,7 @@
 struct PeerContext
 {
   struct GNUNET_CONFIGURATION_Handle *cfg;
-  struct GNUNET_TRANSPORT_Handle *th;
+  struct GNUNET_TRANSPORT_CoreHandle *th;
   struct GNUNET_MessageHeader *hello;
   struct GNUNET_TRANSPORT_GetHelloHandle *ghh;
   struct GNUNET_OS_Process *arm_proc;
@@ -54,29 +55,30 @@
 static void
 clean_up (void *cls)
 {
-  if (p1.th != NULL)
+  if (NULL != p1.th)
   {
-    if (p1.ghh != NULL)
+    if (NULL != p1.ghh)
     {
       GNUNET_TRANSPORT_get_hello_cancel (p1.ghh);
       p1.ghh = NULL;
     }
-    GNUNET_TRANSPORT_disconnect (p1.th);
+    GNUNET_TRANSPORT_core_disconnect (p1.th);
     p1.th = NULL;
   }
-  if (p2.th != NULL)
+  if (NULL != p2.th)
   {
-    if (p2.ghh != NULL)
+    if (NULL != p2.ghh)
     {
       GNUNET_TRANSPORT_get_hello_cancel (p2.ghh);
       p2.ghh = NULL;
     }
-    GNUNET_TRANSPORT_disconnect (p2.th);
+    GNUNET_TRANSPORT_core_disconnect (p2.th);
     p2.th = NULL;
   }
   GNUNET_SCHEDULER_shutdown ();
 }
 
+
 /**
  * Timeout, give up.
  */
@@ -96,24 +98,24 @@
  *
  * @param cls closure
  * @param peer the peer that connected
- * @param latency current latency of the connection
- * @param distance in overlay hops, as given by transport plugin
+ * @param mq message queue to send messages to the peer
  */
-static void
+static void *
 notify_connect (void *cls,
-               const struct GNUNET_PeerIdentity *peer)
+               const struct GNUNET_PeerIdentity *peer,
+               struct GNUNET_MQ_Handle *mq)
 {
-  if (peer == NULL)
-    return;
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
              "Peers connected, shutting down.\n");
   ok = 0;
-  if (timeout_task != NULL)
+  if (NULL != timeout_task)
   {
     GNUNET_SCHEDULER_cancel (timeout_task);
     timeout_task = NULL;
   }
-  GNUNET_SCHEDULER_add_now (&clean_up, NULL);
+  GNUNET_SCHEDULER_add_now (&clean_up,
+                           NULL);
+  return NULL;
 }
 
 
@@ -131,7 +133,8 @@
 
 
 static void
-setup_peer (struct PeerContext *p, const char *cfgname)
+setup_peer (struct PeerContext *p,
+           const char *cfgname)
 {
   char *binary;
 
@@ -138,16 +141,30 @@
   binary = GNUNET_OS_get_libexec_binary_path ("gnunet-service-arm");
   p->cfg = GNUNET_CONFIGURATION_create ();
   p->arm_proc =
-    GNUNET_OS_start_process (GNUNET_YES, GNUNET_OS_INHERIT_STD_OUT_AND_ERR,
-                             NULL, NULL, NULL,
+    GNUNET_OS_start_process (GNUNET_YES,
+                            GNUNET_OS_INHERIT_STD_OUT_AND_ERR,
+                             NULL,
+                            NULL,
+                            NULL,
                              binary,
                              "gnunet-service-arm",
-                             "-c", cfgname, NULL);
-  GNUNET_assert (GNUNET_OK == GNUNET_CONFIGURATION_load (p->cfg, cfgname));
-  p->th =
-      GNUNET_TRANSPORT_connect (p->cfg, NULL, p, NULL, &notify_connect, NULL);
+                             "-c",
+                            cfgname,
+                            NULL);
+  GNUNET_assert (GNUNET_OK ==
+                GNUNET_CONFIGURATION_load (p->cfg,
+                                           cfgname));
+  p->th = GNUNET_TRANSPORT_core_connect (p->cfg,
+                                        NULL,
+                                        NULL,
+                                        p,
+                                        &notify_connect,
+                                        NULL,
+                                        NULL);
   GNUNET_assert (p->th != NULL);
-  p->ghh = GNUNET_TRANSPORT_get_hello (p->cfg, &process_hello, p);
+  p->ghh = GNUNET_TRANSPORT_get_hello (p->cfg,
+                                      &process_hello,
+                                      p);
   GNUNET_free (binary);
 }
 
@@ -157,12 +174,18 @@
 {
   struct PeerContext *p = cls;
 
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Killing ARM process.\n");
-  if (0 != GNUNET_OS_process_kill (p->arm_proc, GNUNET_TERM_SIG))
-    GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING, "kill");
-  if (GNUNET_OS_process_wait (p->arm_proc) != GNUNET_OK)
-    GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING, "waitpid");
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "ARM process %u stopped\n",
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "Killing ARM process.\n");
+  if (0 != GNUNET_OS_process_kill (p->arm_proc,
+                                  GNUNET_TERM_SIG))
+    GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING,
+                        "kill");
+  if (GNUNET_OK !=
+      GNUNET_OS_process_wait (p->arm_proc))
+    GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING,
+                        "waitpid");
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "ARM process %u stopped\n",
               GNUNET_OS_process_get_pid (p->arm_proc));
   GNUNET_OS_process_destroy (p->arm_proc);
   p->arm_proc = NULL;
@@ -176,7 +199,8 @@
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
              "Asking ARM to stop core service\n");
   GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_SECONDS,
-                               &waitpid_task, p);
+                               &waitpid_task,
+                               p);
 }
 
 
@@ -204,8 +228,10 @@
                                               NULL);
   GNUNET_SCHEDULER_add_shutdown (&shutdown_task,
                                 NULL);
-  setup_peer (&p1, "test_gnunet_daemon_hostlist_peer1.conf");
-  setup_peer (&p2, "test_gnunet_daemon_hostlist_peer2.conf");
+  setup_peer (&p1,
+             "test_gnunet_daemon_hostlist_peer1.conf");
+  setup_peer (&p2,
+             "test_gnunet_daemon_hostlist_peer2.conf");
 }
 
 
@@ -212,7 +238,8 @@
 static int
 check ()
 {
-  char *const argv[] = { "test-gnunet-daemon-hostlist",
+  char *const argv[] = {
+    "test-gnunet-daemon-hostlist",
     "-c", "test_gnunet_daemon_hostlist_data.conf",
     NULL
   };
@@ -220,8 +247,12 @@
     GNUNET_GETOPT_OPTION_END
   };
   ok = 1;
-  GNUNET_PROGRAM_run ((sizeof (argv) / sizeof (char *)) - 1, argv,
-                      "test-gnunet-daemon-hostlist", "nohelp", options, &run,
+  GNUNET_PROGRAM_run ((sizeof (argv) / sizeof (char *)) - 1,
+                     argv,
+                      "test-gnunet-daemon-hostlist",
+                     "nohelp",
+                     options,
+                     &run,
                       &ok);
   return ok;
 }

Modified: gnunet/src/hostlist/test_gnunet_daemon_hostlist_reconnect.c
===================================================================
--- gnunet/src/hostlist/test_gnunet_daemon_hostlist_reconnect.c 2016-07-29 
16:21:44 UTC (rev 37610)
+++ gnunet/src/hostlist/test_gnunet_daemon_hostlist_reconnect.c 2016-07-29 
19:18:54 UTC (rev 37611)
@@ -1,6 +1,6 @@
 /*
      This file is part of GNUnet
-     Copyright (C) 2010 GNUnet e.V.
+     Copyright (C) 2010, 2016 GNUnet e.V.
 
      GNUnet is free software; you can redistribute it and/or modify
      it under the terms of the GNU General Public License as published
@@ -19,7 +19,7 @@
 */
 /**
  * @file hostlist/test_gnunet_daemon_hostlist_reconnect.c
- * @brief test for gnunet_daemon_hostslist.c; tries to re-start the peers
+ * @brief test for gnunet-daemon-hostslist.c; tries to re-start the peers
  *        and connect a second time
  * @author Christian Grothoff
  */
@@ -26,7 +26,7 @@
 #include "platform.h"
 #include "gnunet_util_lib.h"
 #include "gnunet_arm_service.h"
-#include "gnunet_transport_service.h"
+#include "gnunet_transport_core_service.h"
 
 /**
  * How long until we give up on transmitting the message?
@@ -35,12 +35,12 @@
 
 static int ok;
 
-static struct GNUNET_SCHEDULER_Task * timeout_task;
+static struct GNUNET_SCHEDULER_Task *timeout_task;
 
 struct PeerContext
 {
   struct GNUNET_CONFIGURATION_Handle *cfg;
-  struct GNUNET_TRANSPORT_Handle *th;
+  struct GNUNET_TRANSPORT_CoreHandle *th;
   struct GNUNET_MessageHeader *hello;
   struct GNUNET_TRANSPORT_GetHelloHandle *ghh;
   struct GNUNET_OS_Process *arm_proc;
@@ -70,19 +70,19 @@
  *
  * @param cls closure
  * @param peer the peer that connected
- * @param latency current latency of the connection
- * @param distance in overlay hops, as given by transport plugin
+ * @param mq message queue to send to @a peer
+ * @return NULL
  */
 static void
 notify_connect (void *cls,
-               const struct GNUNET_PeerIdentity *peer)
+               const struct GNUNET_PeerIdentity *peer,
+               struct GNUNET_MQ_Handle *mq)
 {
-  if (peer == NULL)
-    return;
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
              "Peers connected, shutting down.\n");
   ok = 0;
   GNUNET_SCHEDULER_shutdown ();
+  return NULL;
 }
 
 
@@ -100,7 +100,8 @@
 
 
 static void
-setup_peer (struct PeerContext *p, const char *cfgname)
+setup_peer (struct PeerContext *p,
+           const char *cfgname)
 {
   char *binary;
 
@@ -107,16 +108,30 @@
   binary = GNUNET_OS_get_libexec_binary_path ("gnunet-service-arm");
   p->cfg = GNUNET_CONFIGURATION_create ();
   p->arm_proc =
-    GNUNET_OS_start_process (GNUNET_YES, GNUNET_OS_INHERIT_STD_OUT_AND_ERR,
-                             NULL, NULL, NULL,
+    GNUNET_OS_start_process (GNUNET_YES,
+                            GNUNET_OS_INHERIT_STD_OUT_AND_ERR,
+                             NULL,
+                            NULL,
+                            NULL,
                              binary,
                              "gnunet-service-arm",
-                             "-c", cfgname, NULL);
-  GNUNET_assert (GNUNET_OK == GNUNET_CONFIGURATION_load (p->cfg, cfgname));
-  p->th =
-      GNUNET_TRANSPORT_connect (p->cfg, NULL, p, NULL, &notify_connect, NULL);
-  GNUNET_assert (p->th != NULL);
-  p->ghh = GNUNET_TRANSPORT_get_hello (p->cfg, &process_hello, p);
+                             "-c",
+                            cfgname,
+                            NULL);
+  GNUNET_assert (GNUNET_OK ==
+                GNUNET_CONFIGURATION_load (p->cfg,
+                                           cfgname));
+  p->th = GNUNET_TRANSPORT_core_connect (p->cfg,
+                                        NULL,
+                                        NULL,
+                                        p,
+                                        &notify_connect,
+                                        NULL,
+                                        NULL);
+  GNUNET_assert (NULL != p->th);
+  p->ghh = GNUNET_TRANSPORT_get_hello (p->cfg,
+                                      &process_hello,
+                                      p);
   GNUNET_free (binary);
 }
 
@@ -126,12 +141,18 @@
 {
   struct PeerContext *p = cls;
 
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Killing ARM process.\n");
-  if (0 != GNUNET_OS_process_kill (p->arm_proc, GNUNET_TERM_SIG))
-    GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING, "kill");
-  if (GNUNET_OS_process_wait (p->arm_proc) != GNUNET_OK)
-    GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING, "waitpid");
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "ARM process %u stopped\n",
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "Killing ARM process.\n");
+  if (0 != GNUNET_OS_process_kill (p->arm_proc,
+                                  GNUNET_TERM_SIG))
+    GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING,
+                        "kill");
+  if (GNUNET_OK !=
+      GNUNET_OS_process_wait (p->arm_proc))
+    GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING,
+                        "waitpid");
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "ARM process %u stopped\n",
               GNUNET_OS_process_get_pid (p->arm_proc));
   GNUNET_OS_process_destroy (p->arm_proc);
   p->arm_proc = NULL;
@@ -142,9 +163,11 @@
 static void
 stop_arm (struct PeerContext *p)
 {
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Asking ARM to stop core service\n");
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "Asking ARM to stop core service\n");
   GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_SECONDS,
-                               &waitpid_task, p);
+                               &waitpid_task,
+                               p);
 }
 
 
@@ -164,9 +187,9 @@
     GNUNET_TRANSPORT_get_hello_cancel (p1.ghh);
     p1.ghh = NULL;
   }
-  if (p1.th != NULL)
+  if (NULL != p1.th)
   {
-    GNUNET_TRANSPORT_disconnect (p1.th);
+    GNUNET_TRANSPORT_core_disconnect (p1.th);
     p1.th = NULL;
   }
   if (NULL != p2.ghh)
@@ -174,9 +197,9 @@
     GNUNET_TRANSPORT_get_hello_cancel (p2.ghh);
     p2.ghh = NULL;
   }
-  if (p2.th != NULL)
+  if (NULL != p2.th)
   {
-    GNUNET_TRANSPORT_disconnect (p2.th);
+    GNUNET_TRANSPORT_core_disconnect (p2.th);
     p2.th = NULL;
   }
   stop_arm (&p1);
@@ -185,22 +208,28 @@
 
 
 static void
-run (void *cls, char *const *args, const char *cfgfile,
+run (void *cls,
+     char *const *args,
+     const char *cfgfile,
      const struct GNUNET_CONFIGURATION_Handle *cfg)
 {
   GNUNET_assert (ok == 1);
   ok++;
   timeout_task = GNUNET_SCHEDULER_add_delayed (TIMEOUT,
-                                              &timeout_error, NULL);
+                                              &timeout_error,
+                                              NULL);
   GNUNET_SCHEDULER_add_shutdown (&shutdown_task,
                                  NULL);
-  setup_peer (&p1, "test_gnunet_daemon_hostlist_peer1.conf");
-  setup_peer (&p2, "test_gnunet_daemon_hostlist_peer2.conf");
+  setup_peer (&p1,
+             "test_gnunet_daemon_hostlist_peer1.conf");
+  setup_peer (&p2,
+             "test_gnunet_daemon_hostlist_peer2.conf");
 }
 
 
 int
-main (int argcx, char *argvx[])
+main (int argcx,
+      char *argvx[])
 {
   static char *const argv[] = {
     "test-gnunet-daemon-hostlist",
@@ -218,8 +247,12 @@
                     "WARNING",
                     NULL);
   ok = 1;
-  GNUNET_PROGRAM_run ((sizeof (argv) / sizeof (char *)) - 1, argv,
-                      "test-gnunet-daemon-hostlist", "nohelp", options, &run,
+  GNUNET_PROGRAM_run ((sizeof (argv) / sizeof (char *)) - 1,
+                     argv,
+                      "test-gnunet-daemon-hostlist",
+                     "nohelp",
+                     options,
+                     &run,
                       &ok);
   if (0 == ok)
   {
@@ -226,10 +259,16 @@
     FPRINTF (stderr, "%s",  ".");
     /* now do it again */
     ok = 1;
-    GNUNET_PROGRAM_run ((sizeof (argv) / sizeof (char *)) - 1, argv,
-                       "test-gnunet-daemon-hostlist", "nohelp", options, &run,
+    GNUNET_PROGRAM_run ((sizeof (argv) / sizeof (char *)) - 1,
+                       argv,
+                       "test-gnunet-daemon-hostlist",
+                       "nohelp",
+                       options,
+                       &run,
                        &ok);
-    FPRINTF (stderr, "%s",  ".\n");
+    FPRINTF (stderr,
+            "%s",
+            ".\n");
   }
   GNUNET_DISK_directory_remove ("/tmp/test-gnunet-hostlist-peer-1");
   GNUNET_DISK_directory_remove ("/tmp/test-gnunet-hostlist-peer-2");