test: clean up test table and add logging in test env rule (#2594)

Author: mutianf

google-cloud-bigtable/src/test/java/com/google/cloud/bigtable/admin/v2/it/BigtableLogicalViewIT.java

@@ -35,6 +35,7 @@
 import io.grpc.StatusRuntimeException;
 import java.util.List;
 import java.util.logging.Logger;
+import org.junit.After;
 import org.junit.Before;
 import org.junit.BeforeClass;
 import org.junit.ClassRule;
@@ -71,6 +72,13 @@ public void setUp() throws InterruptedException {
     testTable = createTestTable(testEnvRule.env().getTableAdminClient());
   }
 
+  @After
+  public void tearDown() throws InterruptedException {
+    if (testTable != null) {
+      testEnvRule.env().getTableAdminClient().deleteTable(testTable.getId());
+    }
+  }
+
   @Test
   public void createLogicalViewAndGetLogicalViewTest() {
     String logicalViewId = prefixGenerator.newPrefix();

google-cloud-bigtable/src/test/java/com/google/cloud/bigtable/data/v2/it/SampleRowsIT.java

@@ -104,6 +104,8 @@ public void testOnAuthorizedView()
         .env()
         .getTableAdminClient()
         .deleteAuthorizedView(testAuthorizedView.getTableId(), testAuthorizedView.getId());
+
+    testEnvRule.env().getTableAdminClient().deleteTable(testAuthorizedView.getTableId());
   }
 
   private static AuthorizedView createPreSplitTableAndAuthorizedView() {

google-cloud-bigtable/src/test/java/com/google/cloud/bigtable/test_helpers/env/TestEnvRule.java

@@ -167,14 +167,17 @@ void cleanUpStale() throws ExecutionException, InterruptedException, IOException
   private void cleanupStaleTables(String stalePrefix) {
     for (String tableId : env().getTableAdminClient().listTables()) {
       if (!tableId.startsWith(PrefixGenerator.PREFIX)) {
+        LOGGER.info("Skip cleaning up table: " + tableId);
         continue;
       }
       if (stalePrefix.compareTo(tableId) > 0) {
+        LOGGER.info("Preparing stale table for delete: " + tableId);
         prepTableForDelete(tableId);
         try {
+          LOGGER.info("Deleting stable table: " + tableId);
           env().getTableAdminClient().deleteTable(tableId);
-        } catch (NotFoundException ignored) {
-
+        } catch (NotFoundException e) {
+          LOGGER.log(Level.WARNING, "Deleting stale table failed: " + tableId, e);
         }
       }
     }