aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorQingping Hou <dave2008713@gmail.com>2014-02-04 19:54:09 -0500
committerQingping Hou <dave2008713@gmail.com>2014-02-06 16:13:55 -0500
commit39ff3b00cf965bab7ac82aa052ea3ea941d30a2c (patch)
treee236b76697d63046cb918cc566639b0ace539f64
parent0fbe7f31883787df9c6fe91a3869e5f3f6a2befe (diff)
downloadtor-39ff3b00cf965bab7ac82aa052ea3ea941d30a2c.tar
tor-39ff3b00cf965bab7ac82aa052ea3ea941d30a2c.tar.gz
add test for node_get_verbose_nickname_by_id
-rw-r--r--src/test/include.am1
-rw-r--r--src/test/test.c2
-rw-r--r--src/test/test_nodelist.c40
3 files changed, 43 insertions, 0 deletions
diff --git a/src/test/include.am b/src/test/include.am
index aa6a872af..5f978b518 100644
--- a/src/test/include.am
+++ b/src/test/include.am
@@ -39,6 +39,7 @@ src_test_test_SOURCES = \
src/test/test_util.c \
src/test/test_config.c \
src/test/test_hs.c \
+ src/test/test_nodelist.c \
src/ext/tinytest.c
src_test_test_CFLAGS = $(AM_CFLAGS) $(TEST_CFLAGS)
diff --git a/src/test/test.c b/src/test/test.c
index 3f8e9c64f..522f2af64 100644
--- a/src/test/test.c
+++ b/src/test/test.c
@@ -1626,6 +1626,7 @@ extern struct testcase_t controller_event_tests[];
extern struct testcase_t logging_tests[];
extern struct testcase_t backtrace_tests[];
extern struct testcase_t hs_tests[];
+extern struct testcase_t nodelist_tests[];
static struct testgroup_t testgroups[] = {
{ "", test_array },
@@ -1650,6 +1651,7 @@ static struct testgroup_t testgroups[] = {
{ "extorport/", extorport_tests },
{ "control/", controller_event_tests },
{ "hs/", hs_tests },
+ { "nodelist/", nodelist_tests },
END_OF_GROUPS
};
diff --git a/src/test/test_nodelist.c b/src/test/test_nodelist.c
new file mode 100644
index 000000000..b01ceed77
--- /dev/null
+++ b/src/test/test_nodelist.c
@@ -0,0 +1,40 @@
+/* Copyright (c) 2007-2013, The Tor Project, Inc. */
+/* See LICENSE for licensing information */
+
+/**
+ * \file test_nodelist.c
+ * \brief Unit tests for nodelist related functions.
+ **/
+
+#include "or.h"
+#include "nodelist.h"
+#include "test.h"
+
+/** Tese the case when node_get_by_id() returns NULL,
+ * node_get_verbose_nickname_by_id should return the base 16 encoding
+ * of the id.
+ */
+static void
+test_nodelist_node_get_verbose_nickname_by_id_null_node(void *arg)
+{
+ char vname[MAX_VERBOSE_NICKNAME_LEN+1];
+ const char ID[] = "\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA"
+ "\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA";
+ (void) arg;
+
+ /* make sure node_get_by_id returns NULL */
+ test_assert(!node_get_by_id(ID));
+ node_get_verbose_nickname_by_id(ID, vname);
+ test_streq(vname, "$AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA");
+ done:
+ return;
+}
+
+#define NODE(name, flags) \
+ { #name, test_nodelist_##name, (flags), NULL, NULL }
+
+struct testcase_t nodelist_tests[] = {
+ NODE(node_get_verbose_nickname_by_id_null_node, TT_FORK),
+ END_OF_TESTCASES
+};
+