commit 8296506e99e23898f56fa399b5b7b7ef7f1f02bd
author David Pursehouse <dpursehouse@collab.net> Wed Jul 18 09:51:52 2018 +0900
committer David Pursehouse <dpursehouse@collab.net> Wed Jul 18 09:51:52 2018 +0900
tree c05d9ca8120ec8aa71669af049887aee7947a36b
parent 82e543d121eeb2b67b5b59cc6289d26625d3909f

DeleteDraftPatchSetIT: Only 1 statement should come after ExpectedException#expect

Change-Id: I33f3a67134175d76ec7933b1b3ca54e22ea25904

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

diff --git a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/DeleteDraftPatchSetIT.java b/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/DeleteDraftPatchSetIT.java
index 11ff612..38f73c4 100644
--- a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/DeleteDraftPatchSetIT.java
+++ b/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/DeleteDraftPatchSetIT.java
@@ -60,9 +60,9 @@
     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 @@
     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);
   }