From 34bf9b36905c015f0faf45ee79e0539a0e9e7a3e Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Mon, 15 Sep 2014 21:29:48 -0400 Subject: [PATCH] Replace the remaining test_assert instances --- src/test/test.h | 2 -- src/test/test_addr.c | 6 +++--- src/test/test_dir.c | 6 +++--- src/test/test_util.c | 2 +- 4 files changed, 7 insertions(+), 9 deletions(-) diff --git a/src/test/test.h b/src/test/test.h index d3464111f5..05000c0c46 100644 --- 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 24e1ba2122..8900c46b04 100644 --- 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 96acc7bf61..e3af699c23 100644 --- 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 d3ac0cc8b6..4c90153534 100644 --- 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: ; }