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

[tor-commits] [tor/master] Update tests to handle new interpretation of quiet_level.



commit 98c3b3bb4ca7679a5045f285db7fd30b5d069615
Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date:   Thu Oct 17 13:23:11 2019 -0400

    Update tests to handle new interpretation of quiet_level.
    
    Two things needed to be changed.  First, we used to set quiet_level
    to the default (QUIET_NONE) when running tests, since we would not
    call anything that acted based upon it.  But since we sometimes call
    options_init_logs(), we need to pre-set quiet_level to QUIET_SILENT
    in the logs so that we don't add the default logs.  This did not
    cause test failure: just unwanted logs.
    
    Second, we had a test that checked whether options_validate was
    messing with options->Logs correctly.  Since options_validate no
    longer messes with the logs, we no longer want a test for this.
---
 src/test/test_options.c   | 6 ++----
 src/test/testing_common.c | 3 +++
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/test/test_options.c b/src/test/test_options.c
index 69407a999..0a7cd03e5 100644
--- a/src/test/test_options.c
+++ b/src/test/test_options.c
@@ -678,8 +678,7 @@ test_options_validate__logs(void *ignored)
   tdata->opt->RunAsDaemon = 0;
 
   ret = options_validate(tdata->old_opt, tdata->opt, tdata->def_opt, 0, &msg);
-  tt_str_op(tdata->opt->Logs->key, OP_EQ, "Log");
-  tt_str_op(tdata->opt->Logs->value, OP_EQ, "notice stdout");
+  tt_assert(!tdata->opt->Logs);
   tor_free(msg);
   tt_int_op(ret, OP_EQ, -1);
 
@@ -689,8 +688,7 @@ test_options_validate__logs(void *ignored)
   tdata->opt->RunAsDaemon = 0;
   quiet_level = 1;
   ret = options_validate(tdata->old_opt, tdata->opt, tdata->def_opt, 0, &msg);
-  tt_str_op(tdata->opt->Logs->key, OP_EQ, "Log");
-  tt_str_op(tdata->opt->Logs->value, OP_EQ, "warn stdout");
+  tt_assert(!tdata->opt->Logs);
   tor_free(msg);
   tt_int_op(ret, OP_EQ, -1);
 
diff --git a/src/test/testing_common.c b/src/test/testing_common.c
index ff6028ddb..e14a16940 100644
--- a/src/test/testing_common.c
+++ b/src/test/testing_common.c
@@ -272,6 +272,9 @@ main(int c, const char **v)
 
   control_initialize_event_queue();
 
+  /* Don't add default logs; the tests manage their own. */
+  quiet_level = QUIET_SILENT;
+
   for (i_out = i = 1; i < c; ++i) {
     if (!strcmp(v[i], "--warn")) {
       loglevel = LOG_WARN;



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