[tor-commits] [tor/master] Fix a bug in connection/download_status.. tests

nickm at torproject.org nickm at torproject.org
Thu Sep 8 14:56:55 UTC 2016


commit d626ffe29ce63950db3593e1a3a61e38b844ae31
Author: Nick Mathewson <nickm at torproject.org>
Date:   Thu Sep 8 10:48:22 2016 -0400

    Fix a bug in connection/download_status.. tests
---
 src/test/test_connection.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/test/test_connection.c b/src/test/test_connection.c
index bf95b0b..92f1f68 100644
--- a/src/test/test_connection.c
+++ b/src/test/test_connection.c
@@ -340,10 +340,7 @@ test_conn_get_rsrc_teardown(const struct testcase_t *tc, void *arg)
 static void *
 test_conn_download_status_setup(const struct testcase_t *tc)
 {
-  (void)tc;
-
-  /* Don't return NULL, that causes the test to fail */
-  return (void*)"ok";
+  return (void*)tc;
 }
 
 static int
@@ -655,7 +652,8 @@ test_conn_download_status(void *arg)
   dir_connection_t *conn4 = NULL;
   connection_t *ap_conn = NULL;
 
-  consensus_flavor_t usable_flavor = (consensus_flavor_t)arg;
+  const struct testcase_t *tc = arg;
+  consensus_flavor_t usable_flavor = (consensus_flavor_t)tc->setup_data;
 
   /* The "other flavor" trick only works if there are two flavors */
   tor_assert(N_CONSENSUS_FLAVORS == 2);





More information about the tor-commits mailing list