Merge "Do not use deprecated waitForMerge option"

This commit is contained in:
David Pursehouse
2015-07-05 14:47:25 +00:00
committed by Gerrit Code Review
4 changed files with 0 additions and 4 deletions

View File

@@ -123,7 +123,6 @@ public class RevisionIT extends AbstractDaemonTest {
.review(ReviewInput.approve());
SubmitInput in = new SubmitInput();
in.onBehalfOf = admin2.email;
in.waitForMerge = true;
gApi.changes()
.id(project.get() + "~master~" + r.getChangeId())
.current()

View File

@@ -221,7 +221,6 @@ public abstract class AbstractSubmit extends AbstractDaemonTest {
private void submit(String changeId, int expectedStatus) throws Exception {
approve(changeId);
SubmitInput subm = new SubmitInput();
subm.waitForMerge = false;
RestResponse r =
adminSession.post("/changes/" + changeId + "/submit", subm);
assertThat(r.getStatusCode()).isEqualTo(expectedStatus);

View File

@@ -29,7 +29,6 @@ public interface RevisionApi {
void delete() throws RestApiException;
void review(ReviewInput in) throws RestApiException;
/** {@code submit} with {@link SubmitInput#waitForMerge} set to true. */
void submit() throws RestApiException;
void submit(SubmitInput in) throws RestApiException;
void publish() throws RestApiException;

View File

@@ -152,7 +152,6 @@ class RevisionApiImpl implements RevisionApi {
@Override
public void submit() throws RestApiException {
SubmitInput in = new SubmitInput();
in.waitForMerge = true;
submit(in);
}