Replace wrong order of assertion

apache · Nov 29, 2024 · b259997 · b259997
1 parent 7c7b4ba
commit b259997
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
-Original file line number
+Diff line change
@@ Expand Up / @@ -49,7 +49,7 @@ public void testFilterAndRemovePrefix() { @@
         Properties actual = JdbcUtil.filterAndRemovePrefix(input, "jdbc.");
-        assertThat(expected).isEqualTo(actual);
+        assertThat(actual).isEqualTo(expected);
       }
       @Test
@@ Expand Down @@