opendev
/
gerrit
Code Issues Proposed changes

DeleteDraftPatchSetIT: Only 1 statement should come after ExpectedException#expect 

Change-Id: I33f3a67134175d76ec7933b1b3ca54e22ea25904
This commit is contained in:
David Pursehouse 2018-07-18 09:51:52 +09:00
parent 82e543d121
commit 8296506e99
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/DeleteDraftPatchSetIT.java
View File

@ -60,9 +60,9 @@ public class DeleteDraftPatchSetIT extends AbstractDaemonTest {
assertThat(c.id).isEqualTo(triplet);
assertThat(c.status).isEqualTo(ChangeStatus.NEW);
setApiUser(admin);
exception.expect(ResourceConflictException.class);
exception.expectMessage("Patch set is not a draft");
setApiUser(admin);
deletePatchSet(changeId, ps);
}
@ -75,9 +75,9 @@ public class DeleteDraftPatchSetIT extends AbstractDaemonTest {
assertThat(c.id).isEqualTo(triplet);
assertThat(c.status).isEqualTo(ChangeStatus.DRAFT);
setApiUser(user);
exception.expect(ResourceNotFoundException.class);
exception.expectMessage("Not found: " + changeId);
setApiUser(user);
deletePatchSet(changeId, ps);
}