[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor/master] Use setup_full_capture_of_logs for 21800



commit 230a33679814f3074c0ba43e42dc7b38b5342c10
Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date:   Fri Sep 22 10:22:47 2017 -0400

    Use setup_full_capture_of_logs for 21800
    
    setup_capture_of_logs() doesn't actually suppress warnings :p
---
 src/test/test_util.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/test/test_util.c b/src/test/test_util.c
index 1749617c2..49ebdf211 100644
--- a/src/test/test_util.c
+++ b/src/test/test_util.c
@@ -635,7 +635,7 @@ test_util_time(void *arg)
    * time_t */
   a_time.tm_year = 2039-1900;
 #if SIZEOF_TIME_T == 4
-  setup_capture_of_logs(LOG_WARN);
+  setup_full_capture_of_logs(LOG_WARN);
   tt_int_op((time_t) -1,OP_EQ, tor_timegm(&a_time));
   expect_single_log_msg_containing("Result does not fit in tor_timegm");
   teardown_capture_of_logs();
@@ -654,8 +654,7 @@ test_util_time(void *arg)
     setup_full_capture_of_logs(LOG_WARN);                       \
   } while (0)
 #define CHECK_TIMEGM_WARNING(msg) do { \
-    expect_log_msg_containing(msg);                                     \
-    tt_int_op(1, OP_EQ, smartlist_len(mock_saved_logs()));              \
+    expect_single_log_msg_containing(msg);                              \
     teardown_capture_of_logs();                                         \
   } while (0)
 
@@ -1226,7 +1225,7 @@ test_util_parse_http_time(void *arg)
 #if SIZEOF_TIME_T == 4
   /* parse_http_time should indicate failure on overflow, but it doesn't yet.
    * Hopefully #18480 will improve the failure semantics in this case. */
-  setup_capture_of_logs(LOG_WARN);
+  setup_full_capture_of_logs(LOG_WARN);
   tt_int_op(0,OP_EQ,parse_http_time("Wed, 17 Feb 2038 06:13:20 GMT", &a_time));
   tt_int_op((time_t)-1,OP_EQ, tor_timegm(&a_time));
   expect_single_log_msg_containing("does not fit in tor_timegm");

_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits