opendev/gerrit
Code Issues Proposed changes

AbstractSubmit: Include error message when submit fails

Browse Source 
Change-Id: Idf7501017d8457fdd42f7e933a6e906c282426ee
This commit is contained in:
Dave Borowitz 2016-01-12 13:54:01 -05:00
parent b009b9d9e2
commit c51d432769
1 changed files with 3 additions and 1 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/AbstractSubmit.java
View File

@ -211,7 +211,9 @@ public abstract class AbstractSubmit extends AbstractDaemonTest {
SubmitInput subm = new SubmitInput(); SubmitInput subm = new SubmitInput();
RestResponse r = RestResponse r =
adminSession.post("/changes/" + changeId + "/submit", subm); adminSession.post("/changes/" + changeId + "/submit", subm);
assertThat(r.getStatusCode()).isEqualTo(expectedStatus); assertThat(r.getStatusCode())
.named("Status code [" + r.getEntityContent() + "]")
.isEqualTo(expectedStatus);
if (expectedStatus == HttpStatus.SC_OK) { if (expectedStatus == HttpStatus.SC_OK) {
checkArgument(msg == null, "msg must be null for successful submits"); checkArgument(msg == null, "msg must be null for successful submits");
ChangeInfo change = ChangeInfo change =