opendev/gerrit
Code Issues Proposed changes

RevisionIT: Only 1 statement should come after ExpectedException#expect

Browse Source 
Change-Id: I5af2c5e9dfa4392baa824faf9c30f2c6b5e33c23
Signed-off-by: Edwin Kempin <ekempin@google.com>
This commit is contained in:
Edwin Kempin
2017-04-03 15:11:21 +02:00
parent 078ba5c96b
commit d5cebda445
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/revision/RevisionIT.java
View File

@@ -996,9 +996,9 @@ public class RevisionIT extends AbstractDaemonTest {
recommend(r.getChangeId()); recommend(r.getChangeId());
// check if it's blocked to delete a vote on a non-current patch set. // check if it's blocked to delete a vote on a non-current patch set.
setApiUser(admin);
exception.expect(MethodNotAllowedException.class); exception.expect(MethodNotAllowedException.class);
exception.expectMessage("Cannot access on non-current patch set"); exception.expectMessage("Cannot access on non-current patch set");
setApiUser(admin);
gApi.changes() gApi.changes()
.id(r.getChangeId()) .id(r.getChangeId())
.revision(r.getCommit().getName()) .revision(r.getCommit().getName())