Skip to content

Commit 061a124

Browse files
committed
Un-ignore tests now that #1147 was merged.
1 parent b88ec46 commit 061a124

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

spark3/src/test/java/org/apache/iceberg/spark/sql/TestAlterTable.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@
3232
import org.junit.Assert;
3333
import org.junit.Assume;
3434
import org.junit.Before;
35-
import org.junit.Ignore;
3635
import org.junit.Test;
3736

3837
public class TestAlterTable extends SparkCatalogTestBase {
@@ -189,7 +188,7 @@ public void testAlterColumnPositionFirst() {
189188
expectedSchema, validationCatalog.loadTable(tableIdent).schema().asStruct());
190189
}
191190

192-
@Ignore // requires catalog names from #1147
191+
@Test
193192
public void testTableRename() {
194193
Assume.assumeFalse("Hadoop catalog does not support rename", validationCatalog instanceof HadoopCatalog);
195194

spark3/src/test/java/org/apache/iceberg/spark/sql/TestCreateTable.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@
3535
import org.junit.After;
3636
import org.junit.Assert;
3737
import org.junit.Assume;
38-
import org.junit.Ignore;
3938
import org.junit.Test;
4039

4140
public class TestCreateTable extends SparkCatalogTestBase {
@@ -66,7 +65,7 @@ public void testCreateTable() {
6665
table.properties().get(TableProperties.DEFAULT_FILE_FORMAT));
6766
}
6867

69-
@Ignore // requires table properties fix in #1147
68+
@Test
7069
public void testCreateTableUsingParquet() {
7170
Assume.assumeTrue(
7271
"Not working with session catalog because Spark will not use v2 for a Parquet table",

0 commit comments

Comments
 (0)