Core, GCS, Spark: Replace wrong order of assertion (#11677)

apache · Nov 29, 2024 · e770fac · e770fac
1 parent 7c7b4ba
commit e770fac
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/core/src/test/java/org/apache/iceberg/jdbc/TestJdbcUtil.java b/core/src/test/java/org/apache/iceberg/jdbc/TestJdbcUtil.java
@@ -49,7 +49,7 @@ public void testFilterAndRemovePrefix() {
 
     Properties actual = JdbcUtil.filterAndRemovePrefix(input, "jdbc.");
 
-    assertThat(expected).isEqualTo(actual);
+    assertThat(actual).isEqualTo(expected);
   }
 
   @Test

diff --git a/gcp/src/test/java/org/apache/iceberg/gcp/gcs/GCSFileIOTest.java b/gcp/src/test/java/org/apache/iceberg/gcp/gcs/GCSFileIOTest.java
@@ -106,7 +106,7 @@ public void newInputFile() throws IOException {
       IOUtil.readFully(is, actual, 0, actual.length);
     }
 
-    assertThat(expected).isEqualTo(actual);
+    assertThat(actual).isEqualTo(expected);
 
     io.deleteFile(in);
 

diff --git a/...v3.5/spark/src/test/java/org/apache/iceberg/spark/source/TestIcebergSourceTablesBase.java b/...v3.5/spark/src/test/java/org/apache/iceberg/spark/source/TestIcebergSourceTablesBase.java
@@ -2054,9 +2054,9 @@ public void testFilesTablePartitionId() {
             .map(r -> (Integer) r.getAs(DataFile.SPEC_ID.name()))
             .collect(Collectors.toList());
 
-    assertThat(ImmutableList.of(spec0, spec1))
+    assertThat(actual)
         .as("Should have two partition specs")
-        .isEqualTo(actual);
+        .isEqualTo(ImmutableList.of(spec0, spec1));
   }
 
   @Test