Parquet: Junit5 upgrade

build.gradle

@@ -731,6 +731,9 @@ project(':iceberg-orc') {
 }
 
 project(':iceberg-parquet') {
+  test {
+    useJUnitPlatform()
+  }
   dependencies {
     implementation project(path: ':iceberg-bundled-guava', configuration: 'shadow')
     api project(':iceberg-api')

parquet/src/test/java/org/apache/iceberg/TestHelpers.java

@@ -18,11 +18,12 @@
  */
 package org.apache.iceberg;
 
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
+
 import java.util.concurrent.Callable;
 import org.apache.avro.AvroRuntimeException;
 import org.apache.avro.generic.GenericRecord;
 import org.assertj.core.api.AbstractThrowableAssert;
-import org.assertj.core.api.Assertions;
 
 public class TestHelpers {
 
@@ -42,7 +43,7 @@ public static void assertThrows(
       String containedInMessage,
       Callable callable) {
     AbstractThrowableAssert<?, ? extends Throwable> check =
-        Assertions.assertThatThrownBy(callable::call).as(message).isInstanceOf(expected);
+        assertThatThrownBy(callable::call).as(message).isInstanceOf(expected);
     if (null != containedInMessage) {
       check.hasMessageContaining(containedInMessage);
     }
@@ -62,7 +63,7 @@ public static void assertThrows(
       String containedInMessage,
       Runnable runnable) {
     AbstractThrowableAssert<?, ? extends Throwable> check =
-        Assertions.assertThatThrownBy(runnable::run).as(message).isInstanceOf(expected);
+        assertThatThrownBy(runnable::run).as(message).isInstanceOf(expected);
     if (null != containedInMessage) {
       check.hasMessageContaining(containedInMessage);
     }

parquet/src/test/java/org/apache/iceberg/avro/TestParquetReadProjection.java

@@ -32,7 +32,7 @@ public class TestParquetReadProjection extends TestReadProjection {
   protected GenericData.Record writeAndRead(
       String desc, Schema writeSchema, Schema readSchema, GenericData.Record record)
       throws IOException {
-    File file = temp.newFile(desc + ".parquet");
+    File file = temp.resolve(desc + ".parquet").toFile();
     file.delete();
 
     try (FileAppender<GenericData.Record> appender =

parquet/src/test/java/org/apache/iceberg/parquet/ParquetWritingTestUtils.java

@@ -23,6 +23,7 @@
 import java.io.Closeable;
 import java.io.File;
 import java.io.IOException;
+import java.nio.file.Path;
 import java.util.Collections;
 import java.util.Map;
 import java.util.UUID;
@@ -33,29 +34,25 @@
 import org.apache.iceberg.io.FileAppender;
 import org.apache.iceberg.relocated.com.google.common.collect.Lists;
 import org.apache.parquet.schema.MessageType;
-import org.junit.rules.TemporaryFolder;
 
 /** Utilities for tests that need to write Parquet files. */
 class ParquetWritingTestUtils {
 
   private ParquetWritingTestUtils() {}
 
-  static File writeRecords(TemporaryFolder temp, Schema schema, GenericData.Record... records)
+  static File writeRecords(Path temp, Schema schema, GenericData.Record... records)
       throws IOException {
     return writeRecords(temp, schema, Collections.emptyMap(), null, records);
   }
 
   static File writeRecords(
-      TemporaryFolder temp,
-      Schema schema,
-      Map<String, String> properties,
-      GenericData.Record... records)
+      Path temp, Schema schema, Map<String, String> properties, GenericData.Record... records)
       throws IOException {
     return writeRecords(temp, schema, properties, null, records);
   }
 
   static File writeRecords(
-      TemporaryFolder temp,
+      Path temp,
       Schema schema,
       Map<String, String> properties,
       Function<MessageType, ParquetValueWriter<?>> createWriterFunc,
@@ -97,8 +94,8 @@ static long write(
     return len;
   }
 
-  static File createTempFile(TemporaryFolder temp) throws IOException {
-    File tmpFolder = temp.newFolder("parquet");
+  static File createTempFile(Path temp) throws IOException {
+    File tmpFolder = temp.resolve("parquet").toFile();
     String filename = UUID.randomUUID().toString();
     return new File(tmpFolder, FileFormat.PARQUET.addExtension(filename));
   }