ChangeIT: Remove redundant assertOkStatus() checks createChange() already invokes assertOkStatus() and the caller of createChange() don't need to do it again. Change-Id: I8df0ed4462dfbc021deaefbb02c8ec75fc4882de Signed-off-by: Edwin Kempin <ekempin@google.com>
diff --git a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/change/ChangeIT.java b/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/change/ChangeIT.java index 6701bb2..da8f4a7 100644 --- a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/change/ChangeIT.java +++ b/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/change/ChangeIT.java
@@ -614,7 +614,6 @@ @Test public void reviewAndMoveToWorkInProgress() throws Exception { PushOneCommit.Result r = createChange(); - r.assertOkStatus(); assertThat(r.getChange().change().isWorkInProgress()).isFalse(); ReviewInput in = ReviewInput.noScore().setWorkInProgress(true); @@ -628,7 +627,6 @@ @Test public void reviewAndSetWorkInProgressAndAddReviewerAndVote() throws Exception { PushOneCommit.Result r = createChange(); - r.assertOkStatus(); assertThat(r.getChange().change().isWorkInProgress()).isFalse(); ReviewInput in = @@ -645,7 +643,6 @@ @Test public void reviewWithWorkInProgressAndReadyReturnsError() throws Exception { PushOneCommit.Result r = createChange(); - r.assertOkStatus(); ReviewInput in = ReviewInput.noScore(); in.ready = true; in.workInProgress = true; @@ -656,7 +653,6 @@ @Test public void reviewWithWorkInProgressByNonOwnerReturnsError() throws Exception { PushOneCommit.Result r = createChange(); - r.assertOkStatus(); ReviewInput in = ReviewInput.noScore().setWorkInProgress(true); setApiUser(user); ReviewResult result = gApi.changes().id(r.getChangeId()).revision("current").review(in);