Make tests independent so they don't interfere

prestodb · Apr 9, 2024 · ae0be4f · ae0be4f
1 parent fb1a4c8
commit ae0be4f
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/presto-hive/src/test/java/com/facebook/presto/hive/AbstractTestHiveClient.java b/presto-hive/src/test/java/com/facebook/presto/hive/AbstractTestHiveClient.java
@@ -3007,13 +3007,23 @@ public void testBucketSortedTables()
             throws Exception
     {
         SchemaTableName table = temporaryTable("create_sorted");
-        SchemaTableName tableWithTempPath = temporaryTable("create_sorted_with_temp_path");
         try {
             doTestBucketSortedTables(table, false, ORC);
-            doTestBucketSortedTables(tableWithTempPath, true, ORC);
         }
         finally {
             dropTable(table);
+        }
+    }
+
+    @Test
+    public void testBucketSortedTablesTempPath()
+            throws Exception
+    {
+        SchemaTableName tableWithTempPath = temporaryTable("create_sorted_with_temp_path");
+        try {
+            doTestBucketSortedTables(tableWithTempPath, true, ORC);
+        }
+        finally {
             dropTable(tableWithTempPath);
         }
     }
@@ -3077,7 +3087,6 @@ private void doTestBucketSortedTables(SchemaTableName table, boolean useTempPath
                     table.getTableName(),
                     outputHandle.getLocationHandle().getTargetPath().toString(),
                     true);
-            Set<String> files = listAllDataFiles(context, path);
             assertThat(listAllDataFiles(context, path))
                     .filteredOn(file -> file.contains(".tmp-sort"))
                     .size().isGreaterThan(bucketCount * getHiveClientConfig().getMaxOpenSortFiles() * 2);