[tor-commits] [tor/master] pubsub: add test for items_out in builder_finalize()

[asn at torproject.org]

commit d976cda49f7346e8fab16c062973c38ec4da4f4b
Author: Nick Mathewson <nickm at torproject.org>
Date:   Wed Mar 13 16:30:56 2019 -0400

    pubsub: add test for items_out in builder_finalize()
---
 src/test/test_pubsub_build.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/test/test_pubsub_build.c b/src/test/test_pubsub_build.c
index 86b5f763a..19aa1f0e0 100644
--- a/src/test/test_pubsub_build.c
+++ b/src/test/test_pubsub_build.c
@@ -2,6 +2,7 @@
 /* See LICENSE for licensing information */
 
 #define DISPATCH_PRIVATE
+#define PUBSUB_PRIVATE
 
 #include "test/test.h"
 
@@ -119,13 +120,16 @@ test_pubsub_build_types_ok(void *arg)
   pubsub_builder_t *b = NULL;
   dispatch_t *dispatcher = NULL;
   pubsub_connector_t *c = NULL;
+  pubsub_items_t *items = NULL;
 
   b = pubsub_builder_new();
   seed_pubsub_builder_basic(b);
 
-  dispatcher = pubsub_builder_finalize(b, NULL);
+  dispatcher = pubsub_builder_finalize(b, &items);
   b = NULL;
   tt_assert(dispatcher);
+  tt_assert(items);
+  tt_int_op(smartlist_len(items->items), OP_EQ, 4);
 
   tt_int_op(dispatcher->n_types, OP_GE, 2);
   tt_assert(dispatcher->typefns);
@@ -138,6 +142,7 @@ test_pubsub_build_types_ok(void *arg)
   pubsub_connector_free(c);
   pubsub_builder_free(b);
   dispatch_free(dispatcher);
+  pubsub_items_free(items);
 }
 
 /* We fail if the same type is defined in two places with different functions.