test(bigtable): relax logging tests (#8062)

Author: dbolduc

google/cloud/bigtable/admin/integration_tests/instance_admin_integration_test.cc

@@ -478,7 +478,7 @@ TEST_F(InstanceAdminIntegrationTest,
   auto no_logging_client =
       BigtableInstanceAdminClient(MakeBigtableInstanceAdminConnection());
   (void)no_logging_client.ListInstances(project_name);
-  EXPECT_THAT(log.ExtractLines(), IsEmpty());
+  EXPECT_THAT(log.ExtractLines(), Not(Contains(HasSubstr("ListInstances"))));
 }
 
 TEST_F(InstanceAdminIntegrationTest, CustomWorkers) {

google/cloud/bigtable/admin/integration_tests/table_admin_integration_test.cc

@@ -41,7 +41,6 @@ namespace {
 using ::google::cloud::bigtable::testing::TableTestEnvironment;
 using ::testing::Contains;
 using ::testing::HasSubstr;
-using ::testing::IsEmpty;
 using ::testing::IsSupersetOf;
 using ::testing::Not;
 
@@ -497,7 +496,7 @@ TEST_F(TableAdminIntegrationTest, CreateListGetDeleteTableWithLogging) {
   auto no_logging_client =
       BigtableTableAdminClient(MakeBigtableTableAdminConnection());
   (void)no_logging_client.ListTables(instance_name);
-  EXPECT_THAT(log.ExtractLines(), IsEmpty());
+  EXPECT_THAT(log.ExtractLines(), Not(Contains(HasSubstr("ListTables"))));
 }
 
 }  // namespace

google/cloud/bigtable/tests/admin_integration_test.cc

@@ -34,7 +34,6 @@ namespace {
 
 using ::testing::Contains;
 using ::testing::HasSubstr;
-using ::testing::IsEmpty;
 using ::testing::IsSupersetOf;
 using ::testing::Not;
 
@@ -385,7 +384,7 @@ TEST_F(AdminIntegrationTest, CreateListGetDeleteTableWithLogging) {
   auto no_logging_client =
       TableAdmin(MakeAdminClient(project_id()), instance_id());
   (void)no_logging_client.ListTables(btadmin::Table::NAME_ONLY);
-  EXPECT_THAT(log.ExtractLines(), IsEmpty());
+  EXPECT_THAT(log.ExtractLines(), Not(Contains(HasSubstr("ListTables"))));
 }
 
 }  // namespace

google/cloud/bigtable/tests/data_integration_test.cc

@@ -33,7 +33,6 @@ using ::std::chrono::microseconds;
 using ::std::chrono::milliseconds;
 using ::testing::Contains;
 using ::testing::HasSubstr;
-using ::testing::IsEmpty;
 
 using DataIntegrationTest =
     ::google::cloud::bigtable::testing::TableIntegrationTest;
@@ -538,7 +537,7 @@ TEST_F(DataIntegrationTest, TableApplyWithLogging) {
   auto no_logging_client =
       Table(MakeDataClient(project_id(), instance_id()), table_id);
   Apply(no_logging_client, row_key, created);
-  EXPECT_THAT(log.ExtractLines(), IsEmpty());
+  EXPECT_THAT(log.ExtractLines(), Not(Contains(HasSubstr("MutateRow"))));
 }
 
 TEST(ConnectionRefresh, Disabled) {

google/cloud/bigtable/tests/instance_admin_integration_test.cc

@@ -39,7 +39,6 @@ using ::google::cloud::internal::GetEnv;
 using ::google::cloud::testing_util::ContainsOnce;
 using ::testing::Contains;
 using ::testing::HasSubstr;
-using ::testing::IsEmpty;
 using ::testing::Not;
 namespace btadmin = ::google::bigtable::admin::v2;
 
@@ -473,7 +472,7 @@ TEST_F(InstanceAdminIntegrationTest,
   // Verify that a normal client does not log.
   auto no_logging_client = InstanceAdmin(MakeInstanceAdminClient(project_id_));
   (void)no_logging_client.ListInstances();
-  EXPECT_THAT(log.ExtractLines(), IsEmpty());
+  EXPECT_THAT(log.ExtractLines(), Not(Contains(HasSubstr("ListInstances"))));
 }
 
 TEST_F(InstanceAdminIntegrationTest, CustomWorkers) {