Should close admin in afterEach() in the repair integration tests (#767)

scalar-labs · Dec 26, 2022 · f1cf367 · f1cf367
1 parent 13acd02
commit f1cf367
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/...rc/main/java/com/scalar/db/api/DistributedStorageAdminRepairTableIntegrationTestBase.java b/...rc/main/java/com/scalar/db/api/DistributedStorageAdminRepairTableIntegrationTestBase.java
@@ -105,12 +105,11 @@ protected void beforeEach() throws Exception {
   @AfterEach
   protected void afterEach() throws Exception {
     dropTable();
+    admin.close();
   }
 
   @AfterAll
-  protected void afterAll() throws Exception {
-    admin.close();
-  }
+  protected void afterAll() throws Exception {}
 
   @Test
   public void repairTable_ForDeletedMetadataTable_ShouldRepairProperly() throws Exception {

diff --git a/...ain/java/com/scalar/db/api/DistributedTransactionAdminRepairTableIntegrationTestBase.java b/...ain/java/com/scalar/db/api/DistributedTransactionAdminRepairTableIntegrationTestBase.java
@@ -106,12 +106,11 @@ protected void beforeEach() throws Exception {
   @AfterEach
   protected void afterEach() throws Exception {
     dropTable();
+    admin.close();
   }
 
   @AfterAll
-  protected void afterAll() throws Exception {
-    admin.close();
-  }
+  protected void afterAll() throws Exception {}
 
   @Test
   public void repairTableAndCoordinatorTable_ForDeletedMetadataTable_ShouldRepairProperly()