From a28b5f2fbb70ab35f22cc3d14c03d62d3cd8bccc Mon Sep 17 00:00:00 2001
From: levlam <levlam@telegram.org>
Date: Sun, 21 Nov 2021 21:41:11 +0300
Subject: [PATCH] Remove explicit verbosity level changes in tests.

---
 tdactor/test/actors_bugs.cpp | 1 -
 tdutils/test/misc.cpp        | 2 --
 test/db.cpp                  | 2 --
 test/http.cpp                | 1 -
 test/mtproto.cpp             | 2 --
 test/secret.cpp              | 1 -
 test/tdclient.cpp            | 5 ++---
 7 files changed, 2 insertions(+), 12 deletions(-)

diff --git a/tdactor/test/actors_bugs.cpp b/tdactor/test/actors_bugs.cpp
index aaa1252ad..8d6f3c468 100644
--- a/tdactor/test/actors_bugs.cpp
+++ b/tdactor/test/actors_bugs.cpp
@@ -91,7 +91,6 @@ class TimeoutManager final : public td::Actor {
 td::int32 TimeoutManager::count;
 
 TEST(MultiTimeout, Destroy) {
-  SET_VERBOSITY_LEVEL(VERBOSITY_NAME(ERROR));
   td::ConcurrentScheduler sched;
   int threads_n = 0;
   sched.init(threads_n);
diff --git a/tdutils/test/misc.cpp b/tdutils/test/misc.cpp
index 3cfd9885d..5a67f3939 100644
--- a/tdutils/test/misc.cpp
+++ b/tdutils/test/misc.cpp
@@ -72,7 +72,6 @@ static td::ExitGuard exit_guard;
 
 #if TD_LINUX || TD_DARWIN
 TEST(Misc, update_atime_saves_mtime) {
-  SET_VERBOSITY_LEVEL(VERBOSITY_NAME(ERROR));
   td::string name = "test_file";
   td::unlink(name).ignore();
   auto r_file = td::FileFd::open(name, td::FileFd::Read | td::FileFd::Flags::Create | td::FileFd::Flags::Truncate);
@@ -102,7 +101,6 @@ TEST(Misc, update_atime_saves_mtime) {
 }
 
 TEST(Misc, update_atime_change_atime) {
-  SET_VERBOSITY_LEVEL(VERBOSITY_NAME(ERROR));
   td::string name = "test_file";
   td::unlink(name).ignore();
   auto r_file = td::FileFd::open(name, td::FileFd::Read | td::FileFd::Flags::Create | td::FileFd::Flags::Truncate);
diff --git a/test/db.cpp b/test/db.cpp
index b48903950..0d19db8ed 100644
--- a/test/db.cpp
+++ b/test/db.cpp
@@ -340,7 +340,6 @@ class BaselineKV {
 };
 
 TEST(DB, key_value) {
-  SET_VERBOSITY_LEVEL(VERBOSITY_NAME(ERROR));
   std::vector<std::string> keys;
   std::vector<std::string> values;
 
@@ -517,7 +516,6 @@ TEST(DB, persistent_key_value) {
   using KeyValue = BinlogKeyValue<ConcurrentBinlog>;
   // using KeyValue = PersistentKeyValue;
   // using KeyValue = SqliteKV;
-  SET_VERBOSITY_LEVEL(VERBOSITY_NAME(ERROR));
   std::vector<std::string> keys;
   std::vector<std::string> values;
   CSlice path = "test_pmc";
diff --git a/test/http.cpp b/test/http.cpp
index 3696ea2cb..a9be5dffa 100644
--- a/test/http.cpp
+++ b/test/http.cpp
@@ -140,7 +140,6 @@ TEST(Http, reader) {
   return;
 #endif
   clear_thread_locals();
-  SET_VERBOSITY_LEVEL(VERBOSITY_NAME(ERROR));
   auto start_mem = BufferAllocator::get_buffer_mem();
   auto start_size = BufferAllocator::get_buffer_slice_size();
   {
diff --git a/test/mtproto.cpp b/test/mtproto.cpp
index a9a0cb09c..c886f5f60 100644
--- a/test/mtproto.cpp
+++ b/test/mtproto.cpp
@@ -49,7 +49,6 @@
 using namespace td;
 
 TEST(Mtproto, GetHostByNameActor) {
-  SET_VERBOSITY_LEVEL(VERBOSITY_NAME(ERROR));
   ConcurrentScheduler sched;
   int threads_n = 1;
   sched.init(threads_n);
@@ -667,7 +666,6 @@ TEST(Mtproto, Grease) {
 }
 
 TEST(Mtproto, TlsTransport) {
-  SET_VERBOSITY_LEVEL(VERBOSITY_NAME(ERROR));
   ConcurrentScheduler sched;
   int threads_n = 1;
   sched.init(threads_n);
diff --git a/test/secret.cpp b/test/secret.cpp
index 5dd40afb3..4b34d65f2 100644
--- a/test/secret.cpp
+++ b/test/secret.cpp
@@ -1001,7 +1001,6 @@ void FakeSecretChatContext::on_read_message(int64, Promise<> promise) {
 
 TEST(Secret, go) {
   return;
-  SET_VERBOSITY_LEVEL(VERBOSITY_NAME(ERROR));
   ConcurrentScheduler sched;
   int threads_n = 0;
   sched.init(threads_n);
diff --git a/test/tdclient.cpp b/test/tdclient.cpp
index c351e7358..a914c254f 100644
--- a/test/tdclient.cpp
+++ b/test/tdclient.cpp
@@ -828,7 +828,6 @@ class Tdclient_login final : public td::Test {
   using Test::Test;
   bool step() final {
     if (!is_inited_) {
-      SET_VERBOSITY_LEVEL(VERBOSITY_NAME(DEBUG) + 2);
       sched_.init(4);
       sched_.create_actor_unsafe<LoginTestActor>(0, "LoginTestActor", &result_).release();
       sched_.start();
@@ -936,7 +935,7 @@ TEST(Client, Multi) {
 TEST(Client, Manager) {
   td::vector<td::thread> threads;
   td::ClientManager client;
-#if !TD_EVENTFD_UNSUPPORTED // Client must be used from a single thread if there is no EventFd
+#if !TD_EVENTFD_UNSUPPORTED  // Client must be used from a single thread if there is no EventFd
   int threads_n = 4;
 #else
   int threads_n = 1;
@@ -973,7 +972,7 @@ TEST(Client, Manager) {
   }
 }
 
-#if !TD_EVENTFD_UNSUPPORTED // Client must be used from a single thread if there is no EventFd
+#if !TD_EVENTFD_UNSUPPORTED  // Client must be used from a single thread if there is no EventFd
 TEST(Client, Close) {
   std::atomic<bool> stop_send{false};
   std::atomic<bool> can_stop_receive{false};