diff options
author | David Pursehouse <david.pursehouse@sonymobile.com> | 2015-08-03 11:38:42 +0900 |
---|---|---|
committer | David Pursehouse <david.pursehouse@sonymobile.com> | 2015-08-03 11:38:42 +0900 |
commit | 4d13333d608dac49357f1bbc0b5d2aa5b28eacf2 (patch) | |
tree | e8c004492d30f9d29f42cc9c8499ac7090d490c0 | |
parent | 89caa4212ada74a48a458442d1b270da5cc00368 (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.java | 6 |
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) { |