summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Pursehouse <david.pursehouse@sonymobile.com>2015-08-03 11:38:42 +0900
committerDavid Pursehouse <david.pursehouse@sonymobile.com>2015-08-03 11:38:42 +0900
commit4d13333d608dac49357f1bbc0b5d2aa5b28eacf2 (patch)
treee8c004492d30f9d29f42cc9c8499ac7090d490c0
parent89caa4212ada74a48a458442d1b270da5cc00368 (diff)
PushOneCommit: Fix usage of assertThat
The expected and actual values were passed in the wrong order, resulting in confusing failure messages. Change-Id: Ic283261fd5d6afd3c9988c04349785ea9d43da8f
-rw-r--r--gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/PushOneCommit.java6
1 files changed, 3 insertions, 3 deletions
diff --git a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/PushOneCommit.java b/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/PushOneCommit.java
index 6cd8031b92..67b6f518c2 100644
--- a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/PushOneCommit.java
+++ b/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/PushOneCommit.java
@@ -282,10 +282,10 @@ public class PushOneCommit {
private void assertStatus(Status expectedStatus, String expectedMessage) {
RemoteRefUpdate refUpdate = result.getRemoteUpdate(ref);
- assertThat(expectedStatus)
+ assertThat(refUpdate.getStatus())
.named(message(refUpdate))
- .isEqualTo(refUpdate.getStatus());
- assertThat(expectedMessage).isEqualTo(refUpdate.getMessage());
+ .isEqualTo(expectedStatus);
+ assertThat(refUpdate.getMessage()).isEqualTo(expectedMessage);
}
public void assertMessage(String expectedMessage) {