diff --git a/spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAlterTablePartitionFields.java b/spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAlterTablePartitionFields.java index 15161446a868..0e978e52e570 100644 --- a/spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAlterTablePartitionFields.java +++ b/spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestAlterTablePartitionFields.java @@ -27,11 +27,11 @@ import org.apache.spark.sql.connector.catalog.CatalogManager; import org.apache.spark.sql.connector.catalog.Identifier; import org.apache.spark.sql.connector.catalog.TableCatalog; +import org.assertj.core.api.Assertions; import org.junit.After; import org.junit.Assert; import org.junit.Test; import org.junit.runners.Parameterized; -import org.assertj.core.api.Assertions; public class TestAlterTablePartitionFields extends SparkExtensionsTestBase { @@ -180,7 +180,7 @@ public void testAddHoursPartition() { Assert.assertEquals("Should have new spec field", expected, table.spec()); } -@Test + @Test public void testAddYearPartition() { createTable("id bigint NOT NULL, category string, ts timestamp, data string"); Table table = validationCatalog.loadTable(tableIdent); @@ -196,9 +196,7 @@ public void testAddYearPartition() { PartitionSpec expected = PartitionSpec.builderFor(table.schema()).withSpecId(1).year("ts").build(); - Assertions.assertThat(table.spec()) - .as("Should have new spec field") - .isEqualTo(expected); + Assertions.assertThat(table.spec()).as("Should have new spec field").isEqualTo(expected); } @Test @@ -217,9 +215,7 @@ public void testAddMonthPartition() { PartitionSpec expected = PartitionSpec.builderFor(table.schema()).withSpecId(1).month("ts").build(); - Assertions.assertThat(table.spec()) - .as("Should have new spec field") - .isEqualTo(expected); + Assertions.assertThat(table.spec()).as("Should have new spec field").isEqualTo(expected); } @Test @@ -238,9 +234,7 @@ public void testAddDayPartition() { PartitionSpec expected = PartitionSpec.builderFor(table.schema()).withSpecId(1).day("ts").build(); - Assertions.assertThat(table.spec()) - .as("Should have new spec field") - .isEqualTo(expected); + Assertions.assertThat(table.spec()).as("Should have new spec field").isEqualTo(expected); } @Test @@ -259,9 +253,7 @@ public void testAddHourPartition() { PartitionSpec expected = PartitionSpec.builderFor(table.schema()).withSpecId(1).hour("ts").build(); - Assertions.assertThat(table.spec()) - .as("Should have new spec field") - .isEqualTo(expected); + Assertions.assertThat(table.spec()).as("Should have new spec field").isEqualTo(expected); } @Test