test: Wrap ExecuteQuery in try/catch block to make sure the deletion always happen

trollyxia · trollyxia · commit 53d489bc6a1b · 2025-11-12T14:33:35.000-05:00
Change-Id: I50e92785affc8ea931253190b8389617c7e327e1
diff --git a/google-cloud-bigtable/src/test/java/com/google/cloud/bigtable/data/v2/it/ExecuteQueryIT.java b/google-cloud-bigtable/src/test/java/com/google/cloud/bigtable/data/v2/it/ExecuteQueryIT.java
@@ -232,8 +232,11 @@ public void allTypes() throws Exception {
       assertThat(rs.getProtoEnum("enumCol", Genre::forNumber)).isEqualTo(Genre.JAZZ);
       assertThat(rs.getProtoEnum(12, Genre::forNumber)).isEqualTo(Genre.JAZZ);
       assertThat(rs.next()).isFalse();
+    } catch (AssertionError e) {
+      throw e;
+    } finally {
+      deleteTestSchemaBundle();
     }
-    deleteTestSchemaBundle();
   }
 
   @Test

Original file line number	Diff line number	Diff line change
`@@ -232,8 +232,11 @@ public void allTypes() throws Exception {`
`232`	`232`	`assertThat(rs.getProtoEnum("enumCol", Genre::forNumber)).isEqualTo(Genre.JAZZ);`
`233`	`233`	`assertThat(rs.getProtoEnum(12, Genre::forNumber)).isEqualTo(Genre.JAZZ);`
`234`	`234`	`assertThat(rs.next()).isFalse();`
	`235`	`+ } catch (AssertionError e) {`
	`236`	`+ throw e;`
	`237`	`+ } finally {`
	`238`	`+ deleteTestSchemaBundle();`
`235`	`239`	`}`
`236`		`- deleteTestSchemaBundle();`
`237`	`240`	`}`
`238`	`241`
`239`	`242`	`@Test`