Change parameter order in assertEquals (#30247)

akka · May 25, 2021 · 48e4f11 · 48e4f11
1 parent b989c4c
commit 48e4f11
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/akka-actor-tests/src/test/java/akka/actor/StashJavaAPITestActors.java b/akka-actor-tests/src/test/java/akka/actor/StashJavaAPITestActors.java
@@ -25,8 +25,8 @@ private static int testReceive(
         return count + 1;
       }
     } else if (msg instanceof Integer) {
-      int value = ((Integer) msg).intValue();
-      assertEquals(value, 5);
+      int value = (Integer) msg;
+      assertEquals(5, value);
     }
     return count;
   }

diff --git a/akka-actor-tests/src/test/java/akka/actor/setup/ActorSystemSetupTest.java b/akka-actor-tests/src/test/java/akka/actor/setup/ActorSystemSetupTest.java
@@ -28,6 +28,6 @@ public void apiMustBeUsableFromJava() {
         ActorSystemSetup.create().withSetup(javaSetting).get(JavaSetup.class);
 
     assertTrue(result.isPresent());
-    assertEquals(result.get(), javaSetting);
+    assertEquals(javaSetting, result.get());
   }
 }
diff --git a/...istence-typed-tests/src/test/java/akka/persistence/typed/ReplicatedEventSourcingTest.java b/...istence-typed-tests/src/test/java/akka/persistence/typed/ReplicatedEventSourcingTest.java
@@ -180,26 +180,26 @@ public void replicatedEventSourcingReplicationTest() {
           replicaA.tell(new TestBehavior.GetState(probe.ref().narrow()));
           TestBehavior.State reply = probe.expectMessageClass(TestBehavior.State.class);
           assertEquals(
-              reply.texts,
-              new HashSet<String>(Arrays.asList("stored-to-a", "stored-to-b", "stored-to-c")));
+              new HashSet<>(Arrays.asList("stored-to-a", "stored-to-b", "stored-to-c")),
+              reply.texts);
           return null;
         });
     probe.awaitAssert(
         () -> {
           replicaB.tell(new TestBehavior.GetState(probe.ref().narrow()));
           TestBehavior.State reply = probe.expectMessageClass(TestBehavior.State.class);
           assertEquals(
-              reply.texts,
-              new HashSet<String>(Arrays.asList("stored-to-a", "stored-to-b", "stored-to-c")));
+              new HashSet<>(Arrays.asList("stored-to-a", "stored-to-b", "stored-to-c")),
+              reply.texts);
           return null;
         });
     probe.awaitAssert(
         () -> {
           replicaC.tell(new TestBehavior.GetState(probe.ref().narrow()));
           TestBehavior.State reply = probe.expectMessageClass(TestBehavior.State.class);
           assertEquals(
-              reply.texts,
-              new HashSet<String>(Arrays.asList("stored-to-a", "stored-to-b", "stored-to-c")));
+              new HashSet<>(Arrays.asList("stored-to-a", "stored-to-b", "stored-to-c")),
+              reply.texts);
           return null;
         });
   }

diff --git a/...-typed-tests/src/test/java/jdocs/akka/persistence/typed/ReplicatedAuctionExampleTest.java b/...-typed-tests/src/test/java/jdocs/akka/persistence/typed/ReplicatedAuctionExampleTest.java
@@ -70,7 +70,7 @@ public void auctionExample() {
         () -> {
           replicaA.tell(new GetHighestBid(replyProbe.ref()));
           Bid bid = replyProbe.expectMessageClass(Bid.class);
-          assertEquals(bid.offer, 202);
+          assertEquals(202, bid.offer);
           return bid;
         });