Replace the remaining test_assert instances

diff --git a/src/test/test.h b/src/test/test.h
index d3464111f5748ecec75079cb3af8a92016242ffb..05000c0c4639c6435ee0f2de3b5fa6b21a329461 100644 (file)
--- a/src/test/test.h
+++ b/src/test/test.h
@@ -22,8 +22,6 @@
 #define PRETTY_FUNCTION ""
 #endif
 
-#define test_assert(expr) tt_assert(expr)
-
 #define test_eq(expr1, expr2) tt_int_op((expr1), ==, (expr2))
 #define test_streq(expr1, expr2) tt_str_op((expr1), ==, (expr2))
 

diff --git a/src/test/test_addr.c b/src/test/test_addr.c
index 24e1ba2122ab803cd05b85bc5ce01c80289672f2..8900c46b04e9286a8fe262e6c1832e59622dbb55 100644 (file)
--- a/src/test/test_addr.c
+++ b/src/test/test_addr.c
@@ -651,13 +651,13 @@ test_addr_ip6_helpers(void)
 #if 0
   /* Try a mask with a wildcard. */
   r=tor_addr_parse_mask_ports("*/16",0,&t1, &mask, NULL, NULL);
-  test_assert(r == -1);
+  tt_assert(r == -1);
   r=tor_addr_parse_mask_ports("*4/16",TAPMP_EXTENDED_STAR,
                               &t1, &mask, NULL, NULL);
-  test_assert(r == -1);
+  tt_assert(r == -1);
   r=tor_addr_parse_mask_ports("*6/30",TAPMP_EXTENDED_STAR,
                               &t1, &mask, NULL, NULL);
-  test_assert(r == -1);
+  tt_assert(r == -1);
 #endif
   /* Basic mask tests*/
   r=tor_addr_parse_mask_ports("1.1.2.2/31",0,&t1, &mask, NULL, NULL);

diff --git a/src/test/test_dir.c b/src/test/test_dir.c
index 96acc7bf61ac72a8fc675fa304cbe85823bfe733..e3af699c23f7163170c615139814ceccd745ec37 100644 (file)
--- a/src/test/test_dir.c
+++ b/src/test/test_dir.c
@@ -194,7 +194,7 @@ test_dir_formats(void)
   tt_int_op(rp1->bandwidthcapacity,==, r1->bandwidthcapacity);
   tt_assert(crypto_pk_cmp_keys(rp1->onion_pkey, pk1) == 0);
   tt_assert(crypto_pk_cmp_keys(rp1->identity_pkey, pk2) == 0);
-  //test_assert(rp1->exit_policy == NULL);
+  //tt_assert(rp1->exit_policy == NULL);
   tor_free(buf);
 
   strlcpy(buf2,
@@ -1094,7 +1094,7 @@ test_vrs_for_v3ns(vote_routerstatus_t *vrs, int voter, time_t now)
      * Didn't expect this, but the old unit test only checked some of them,
      * so don't assert.
      */
-    /* test_assert(0); */
+    /* tt_assert(0); */
   }
 
  done:
@@ -2126,7 +2126,7 @@ test_consensus_for_umbw(networkstatus_t *con, time_t now)
 
   tt_assert(con);
   tt_assert(!con->cert);
-  // test_assert(con->consensus_method >= MIN_METHOD_TO_CLIP_UNMEASURED_BW_KB);
+  // tt_assert(con->consensus_method >= MIN_METHOD_TO_CLIP_UNMEASURED_BW_KB);
   tt_assert(con->consensus_method >= 16);
   tt_int_op(4,==, smartlist_len(con->routerstatus_list));
   /* There should be four listed routers; all voters saw the same in this */

diff --git a/src/test/test_util.c b/src/test/test_util.c
index d3ac0cc8b6466f40735467c6393091e1ee479dfe..4c901535346df1cc1be85541e9b9ef2eafc7a7b1 100644 (file)
--- a/src/test/test_util.c
+++ b/src/test/test_util.c
@@ -976,7 +976,7 @@ test_util_string_is_key_value(void *ptr)
   tt_assert(!string_is_key_value(LOG_WARN, "="));
 
   /* ??? */
-  /* test_assert(!string_is_key_value(LOG_WARN, "===")); */
+  /* tt_assert(!string_is_key_value(LOG_WARN, "===")); */
  done:
   ;
 }