diff options
Diffstat (limited to 'gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/SubmitByCherryPickIT.java')
-rw-r--r-- | gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/SubmitByCherryPickIT.java | 461 |
1 files changed, 0 insertions, 461 deletions
diff --git a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/SubmitByCherryPickIT.java b/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/SubmitByCherryPickIT.java deleted file mode 100644 index a3859325b3..0000000000 --- a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/SubmitByCherryPickIT.java +++ /dev/null @@ -1,461 +0,0 @@ -// Copyright (C) 2013 The Android Open Source Project -// -// Licensed under the Apache License, Version 2.0 (the "License"); -// you may not use this file except in compliance with the License. -// You may obtain a copy of the License at -// -// http://www.apache.org/licenses/LICENSE-2.0 -// -// Unless required by applicable law or agreed to in writing, software -// distributed under the License is distributed on an "AS IS" BASIS, -// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -// See the License for the specific language governing permissions and -// limitations under the License. - -package com.google.gerrit.acceptance.rest.change; - -import static com.google.common.truth.Truth.assertThat; -import static com.google.common.truth.TruthJUnit.assume; - -import com.google.common.collect.Iterables; -import com.google.gerrit.acceptance.PushOneCommit; -import com.google.gerrit.acceptance.TestProjectInput; -import com.google.gerrit.common.FooterConstants; -import com.google.gerrit.extensions.api.changes.SubmitInput; -import com.google.gerrit.extensions.client.ChangeStatus; -import com.google.gerrit.extensions.client.InheritableBoolean; -import com.google.gerrit.extensions.client.ListChangesOption; -import com.google.gerrit.extensions.client.SubmitType; -import com.google.gerrit.extensions.common.ChangeInfo; -import com.google.gerrit.extensions.registration.DynamicSet; -import com.google.gerrit.extensions.registration.RegistrationHandle; -import com.google.gerrit.extensions.restapi.ResourceConflictException; -import com.google.gerrit.reviewdb.client.Branch; -import com.google.gerrit.reviewdb.client.Change; -import com.google.gerrit.reviewdb.client.PatchSet; -import com.google.gerrit.server.change.Submit.TestSubmitInput; -import com.google.gerrit.server.git.ChangeMessageModifier; -import com.google.gerrit.server.git.strategy.CommitMergeStatus; -import com.google.inject.Inject; -import java.util.List; -import org.eclipse.jgit.lib.ObjectId; -import org.eclipse.jgit.lib.Repository; -import org.eclipse.jgit.revwalk.RevCommit; -import org.eclipse.jgit.revwalk.RevWalk; -import org.junit.Test; - -public class SubmitByCherryPickIT extends AbstractSubmit { - @Inject private DynamicSet<ChangeMessageModifier> changeMessageModifiers; - - @Override - protected SubmitType getSubmitType() { - return SubmitType.CHERRY_PICK; - } - - @Test - public void submitWithCherryPickIfFastForwardPossible() throws Exception { - RevCommit initialHead = getRemoteHead(); - PushOneCommit.Result change = createChange(); - submit(change.getChangeId()); - assertCherryPick(testRepo, false); - RevCommit newHead = getRemoteHead(); - assertThat(newHead.getParent(0)).isEqualTo(change.getCommit().getParent(0)); - - assertRefUpdatedEvents(initialHead, newHead); - assertChangeMergedEvents(change.getChangeId(), newHead.name()); - } - - @Test - public void submitWithCherryPick() throws Exception { - RevCommit initialHead = getRemoteHead(); - PushOneCommit.Result change = createChange("Change 1", "a.txt", "content"); - submit(change.getChangeId()); - - RevCommit headAfterFirstSubmit = getRemoteHead(); - testRepo.reset(initialHead); - PushOneCommit.Result change2 = createChange("Change 2", "b.txt", "other content"); - submit(change2.getChangeId()); - assertCherryPick(testRepo, false); - RevCommit newHead = getRemoteHead(); - assertThat(newHead.getParentCount()).isEqualTo(1); - assertThat(newHead.getParent(0)).isEqualTo(headAfterFirstSubmit); - assertCurrentRevision(change2.getChangeId(), 2, newHead); - assertSubmitter(change2.getChangeId(), 1); - assertSubmitter(change2.getChangeId(), 2); - assertPersonEquals(admin.getIdent(), newHead.getAuthorIdent()); - assertPersonEquals(admin.getIdent(), newHead.getCommitterIdent()); - - assertRefUpdatedEvents(initialHead, headAfterFirstSubmit, headAfterFirstSubmit, newHead); - assertChangeMergedEvents( - change.getChangeId(), headAfterFirstSubmit.name(), change2.getChangeId(), newHead.name()); - } - - @Test - public void changeMessageOnSubmit() throws Exception { - PushOneCommit.Result change = createChange(); - RegistrationHandle handle = - changeMessageModifiers.add( - new ChangeMessageModifier() { - @Override - public String onSubmit( - String newCommitMessage, - RevCommit original, - RevCommit mergeTip, - Branch.NameKey destination) { - return newCommitMessage + "Custom: " + destination.get(); - } - }); - try { - submit(change.getChangeId()); - } finally { - handle.remove(); - } - testRepo.git().fetch().setRemote("origin").call(); - ChangeInfo info = get(change.getChangeId()); - RevCommit c = testRepo.getRevWalk().parseCommit(ObjectId.fromString(info.currentRevision)); - testRepo.getRevWalk().parseBody(c); - assertThat(c.getFooterLines("Custom")).containsExactly("refs/heads/master"); - assertThat(c.getFooterLines(FooterConstants.REVIEWED_ON)).hasSize(1); - } - - @Test - @TestProjectInput(useContentMerge = InheritableBoolean.TRUE) - public void submitWithContentMerge() throws Exception { - RevCommit initialHead = getRemoteHead(); - PushOneCommit.Result change = createChange("Change 1", "a.txt", "aaa\nbbb\nccc\n"); - submit(change.getChangeId()); - RevCommit headAfterFirstSubmit = getRemoteHead(); - PushOneCommit.Result change2 = createChange("Change 2", "a.txt", "aaa\nbbb\nccc\nddd\n"); - submit(change2.getChangeId()); - RevCommit headAfterSecondSubmit = getRemoteHead(); - - testRepo.reset(change.getCommit()); - PushOneCommit.Result change3 = createChange("Change 3", "a.txt", "bbb\nccc\n"); - submit(change3.getChangeId()); - assertCherryPick(testRepo, true); - RevCommit headAfterThirdSubmit = getRemoteHead(); - assertThat(headAfterThirdSubmit.getParent(0)).isEqualTo(headAfterSecondSubmit); - assertApproved(change3.getChangeId()); - assertCurrentRevision(change3.getChangeId(), 2, headAfterThirdSubmit); - assertSubmitter(change2.getChangeId(), 1); - assertSubmitter(change2.getChangeId(), 2); - - assertRefUpdatedEvents( - initialHead, - headAfterFirstSubmit, - headAfterFirstSubmit, - headAfterSecondSubmit, - headAfterSecondSubmit, - headAfterThirdSubmit); - assertChangeMergedEvents( - change.getChangeId(), - headAfterFirstSubmit.name(), - change2.getChangeId(), - headAfterSecondSubmit.name(), - change3.getChangeId(), - headAfterThirdSubmit.name()); - } - - @Test - @TestProjectInput(useContentMerge = InheritableBoolean.TRUE) - public void submitWithContentMerge_Conflict() throws Exception { - RevCommit initialHead = getRemoteHead(); - PushOneCommit.Result change = createChange("Change 1", "a.txt", "content"); - submit(change.getChangeId()); - - RevCommit newHead = getRemoteHead(); - testRepo.reset(initialHead); - PushOneCommit.Result change2 = createChange("Change 2", "a.txt", "other content"); - submitWithConflict( - change2.getChangeId(), - "Failed to submit 1 change due to the following problems:\n" - + "Change " - + change2.getChange().getId() - + ": Change could not be " - + "merged due to a path conflict. Please rebase the change locally and " - + "upload the rebased commit for review."); - - assertThat(getRemoteHead()).isEqualTo(newHead); - assertCurrentRevision(change2.getChangeId(), 1, change2.getCommit()); - assertNoSubmitter(change2.getChangeId(), 1); - - assertRefUpdatedEvents(initialHead, newHead); - assertChangeMergedEvents(change.getChangeId(), newHead.name()); - } - - @Test - public void submitOutOfOrder() throws Exception { - RevCommit initialHead = getRemoteHead(); - PushOneCommit.Result change = createChange("Change 1", "a.txt", "content"); - submit(change.getChangeId()); - - RevCommit headAfterFirstSubmit = getRemoteHead(); - testRepo.reset(initialHead); - createChange("Change 2", "b.txt", "other content"); - PushOneCommit.Result change3 = createChange("Change 3", "c.txt", "different content"); - submit(change3.getChangeId()); - assertCherryPick(testRepo, false); - RevCommit headAfterSecondSubmit = getRemoteHead(); - assertThat(headAfterSecondSubmit.getParent(0)).isEqualTo(headAfterFirstSubmit); - assertApproved(change3.getChangeId()); - assertCurrentRevision(change3.getChangeId(), 2, headAfterSecondSubmit); - assertSubmitter(change3.getChangeId(), 1); - assertSubmitter(change3.getChangeId(), 2); - - assertRefUpdatedEvents( - initialHead, headAfterFirstSubmit, headAfterFirstSubmit, headAfterSecondSubmit); - assertChangeMergedEvents( - change.getChangeId(), - headAfterFirstSubmit.name(), - change3.getChangeId(), - headAfterSecondSubmit.name()); - } - - @Test - public void submitOutOfOrder_Conflict() throws Exception { - RevCommit initialHead = getRemoteHead(); - PushOneCommit.Result change = createChange("Change 1", "a.txt", "content"); - submit(change.getChangeId()); - - RevCommit newHead = getRemoteHead(); - testRepo.reset(initialHead); - createChange("Change 2", "b.txt", "other content"); - PushOneCommit.Result change3 = createChange("Change 3", "b.txt", "different content"); - submitWithConflict( - change3.getChangeId(), - "Failed to submit 1 change due to the following problems:\n" - + "Change " - + change3.getChange().getId() - + ": Change could not be " - + "merged due to a path conflict. Please rebase the change locally and " - + "upload the rebased commit for review."); - - assertThat(getRemoteHead()).isEqualTo(newHead); - assertCurrentRevision(change3.getChangeId(), 1, change3.getCommit()); - assertNoSubmitter(change3.getChangeId(), 1); - - assertRefUpdatedEvents(initialHead, newHead); - assertChangeMergedEvents(change.getChangeId(), newHead.name()); - } - - @Test - public void submitMultipleChanges() throws Exception { - RevCommit initialHead = getRemoteHead(); - - testRepo.reset(initialHead); - PushOneCommit.Result change = createChange("Change 1", "b", "b"); - - testRepo.reset(initialHead); - PushOneCommit.Result change2 = createChange("Change 2", "c", "c"); - - testRepo.reset(initialHead); - PushOneCommit.Result change3 = createChange("Change 3", "d", "d"); - - approve(change.getChangeId()); - approve(change2.getChangeId()); - submit(change3.getChangeId()); - - List<RevCommit> log = getRemoteLog(); - assertThat(log.get(0).getShortMessage()).isEqualTo(change3.getCommit().getShortMessage()); - assertThat(log.get(1).getId()).isEqualTo(initialHead.getId()); - - assertNew(change.getChangeId()); - assertNew(change2.getChangeId()); - - assertRefUpdatedEvents(initialHead, log.get(0)); - assertChangeMergedEvents(change3.getChangeId(), log.get(0).name()); - } - - @Test - public void submitDependentNonConflictingChangesOutOfOrder() throws Exception { - RevCommit initialHead = getRemoteHead(); - - testRepo.reset(initialHead); - PushOneCommit.Result change = createChange("Change 1", "b", "b"); - PushOneCommit.Result change2 = createChange("Change 2", "c", "c"); - assertThat(change2.getCommit().getParent(0)).isEqualTo(change.getCommit()); - - // Submit succeeds; change2 is successfully cherry-picked onto head. - submit(change2.getChangeId()); - RevCommit headAfterFirstSubmit = getRemoteHead(); - // Submit succeeds; change is successfully cherry-picked onto head - // (which was change2's cherry-pick). - submit(change.getChangeId()); - RevCommit headAfterSecondSubmit = getRemoteHead(); - - // change is the new tip. - List<RevCommit> log = getRemoteLog(); - assertThat(log.get(0).getShortMessage()).isEqualTo(change.getCommit().getShortMessage()); - assertThat(log.get(0).getParent(0)).isEqualTo(log.get(1)); - - assertThat(log.get(1).getShortMessage()).isEqualTo(change2.getCommit().getShortMessage()); - assertThat(log.get(1).getParent(0)).isEqualTo(log.get(2)); - - assertThat(log.get(2).getId()).isEqualTo(initialHead.getId()); - - assertRefUpdatedEvents( - initialHead, headAfterFirstSubmit, headAfterFirstSubmit, headAfterSecondSubmit); - assertChangeMergedEvents( - change2.getChangeId(), - headAfterFirstSubmit.name(), - change.getChangeId(), - headAfterSecondSubmit.name()); - } - - @Test - public void submitDependentConflictingChangesOutOfOrder() throws Exception { - RevCommit initialHead = getRemoteHead(); - - testRepo.reset(initialHead); - PushOneCommit.Result change = createChange("Change 1", "b", "b1"); - PushOneCommit.Result change2 = createChange("Change 2", "b", "b2"); - assertThat(change2.getCommit().getParent(0)).isEqualTo(change.getCommit()); - - // Submit fails; change2 contains the delta "b1" -> "b2", which cannot be - // applied against tip. - submitWithConflict( - change2.getChangeId(), - "Failed to submit 1 change due to the following problems:\n" - + "Change " - + change2.getChange().getId() - + ": Change could not be " - + "merged due to a path conflict. Please rebase the change locally and " - + "upload the rebased commit for review."); - - ChangeInfo info3 = get(change2.getChangeId(), ListChangesOption.MESSAGES); - assertThat(info3.status).isEqualTo(ChangeStatus.NEW); - - // Tip has not changed. - List<RevCommit> log = getRemoteLog(); - assertThat(log.get(0)).isEqualTo(initialHead.getId()); - assertNoSubmitter(change2.getChangeId(), 1); - - assertRefUpdatedEvents(); - assertChangeMergedEvents(); - } - - @Test - public void submitSubsetOfDependentChanges() throws Exception { - RevCommit initialHead = getRemoteHead(); - - testRepo.reset(initialHead); - PushOneCommit.Result change = createChange("Change 1", "b", "b"); - PushOneCommit.Result change2 = createChange("Change 2", "c", "c"); - PushOneCommit.Result change3 = createChange("Change 3", "e", "e"); - - // Out of the above, only submit change 3. Changes 1 and 2 are not - // related to change 3 by topic or ancestor (due to cherrypicking!) - approve(change2.getChangeId()); - submit(change3.getChangeId()); - RevCommit newHead = getRemoteHead(); - - assertNew(change.getChangeId()); - assertNew(change2.getChangeId()); - - assertRefUpdatedEvents(initialHead, newHead); - assertChangeMergedEvents(change3.getChangeId(), newHead.name()); - } - - @Test - @TestProjectInput(useContentMerge = InheritableBoolean.TRUE) - public void submitIdenticalTree() throws Exception { - RevCommit initialHead = getRemoteHead(); - - PushOneCommit.Result change1 = createChange("Change 1", "a.txt", "a"); - - testRepo.reset(initialHead); - PushOneCommit.Result change2 = createChange("Change 2", "a.txt", "a"); - - submit(change1.getChangeId()); - RevCommit headAfterFirstSubmit = getRemoteHead(); - assertThat(headAfterFirstSubmit.getShortMessage()).isEqualTo("Change 1"); - - submit(change2.getChangeId(), new SubmitInput(), null, null); - - assertThat(getRemoteHead()).isEqualTo(headAfterFirstSubmit); - - ChangeInfo info2 = get(change2.getChangeId()); - assertThat(info2.status).isEqualTo(ChangeStatus.MERGED); - assertThat(Iterables.getLast(info2.messages).message) - .isEqualTo(CommitMergeStatus.SKIPPED_IDENTICAL_TREE.getMessage()); - - assertRefUpdatedEvents(initialHead, headAfterFirstSubmit); - assertChangeMergedEvents( - change1.getChangeId(), - headAfterFirstSubmit.name(), - change2.getChangeId(), - headAfterFirstSubmit.name()); - } - - @Test - public void repairChangeStateAfterFailure() throws Exception { - // In NoteDb-only mode, repo and meta updates are atomic (at least in InMemoryRepository). - assume().that(notesMigration.disableChangeReviewDb()).isFalse(); - - RevCommit initialHead = getRemoteHead(); - PushOneCommit.Result change = createChange("Change 1", "a.txt", "content"); - submit(change.getChangeId()); - - RevCommit headAfterFirstSubmit = getRemoteHead(); - testRepo.reset(initialHead); - PushOneCommit.Result change2 = createChange("Change 2", "b.txt", "other content"); - Change.Id id2 = change2.getChange().getId(); - TestSubmitInput failInput = new TestSubmitInput(); - failInput.failAfterRefUpdates = true; - submit( - change2.getChangeId(), - failInput, - ResourceConflictException.class, - "Failing after ref updates"); - RevCommit headAfterFailedSubmit = getRemoteHead(); - - // Bad: ref advanced but change wasn't updated. - PatchSet.Id psId1 = new PatchSet.Id(id2, 1); - PatchSet.Id psId2 = new PatchSet.Id(id2, 2); - ChangeInfo info = gApi.changes().id(id2.get()).get(); - assertThat(info.status).isEqualTo(ChangeStatus.NEW); - assertThat(info.revisions.get(info.currentRevision)._number).isEqualTo(1); - assertThat(getPatchSet(psId2)).isNull(); - - ObjectId rev2; - try (Repository repo = repoManager.openRepository(project); - RevWalk rw = new RevWalk(repo)) { - ObjectId rev1 = repo.exactRef(psId1.toRefName()).getObjectId(); - assertThat(rev1).isNotNull(); - - rev2 = repo.exactRef(psId2.toRefName()).getObjectId(); - assertThat(rev2).isNotNull(); - assertThat(rev2).isNotEqualTo(rev1); - assertThat(rw.parseCommit(rev2).getParent(0)).isEqualTo(headAfterFirstSubmit); - - assertThat(repo.exactRef("refs/heads/master").getObjectId()).isEqualTo(rev2); - } - - submit(change2.getChangeId()); - - // Change status and patch set entities were updated, and branch tip stayed - // the same. - RevCommit headAfterSecondSubmit = getRemoteHead(); - assertThat(headAfterSecondSubmit).isEqualTo(headAfterFailedSubmit); - info = gApi.changes().id(id2.get()).get(); - assertThat(info.status).isEqualTo(ChangeStatus.MERGED); - assertThat(info.revisions.get(info.currentRevision)._number).isEqualTo(2); - PatchSet ps2 = getPatchSet(psId2); - assertThat(ps2).isNotNull(); - assertThat(ps2.getRevision().get()).isEqualTo(rev2.name()); - assertThat(Iterables.getLast(info.messages).message) - .isEqualTo( - "Change has been successfully cherry-picked as " + rev2.name() + " by Administrator"); - - try (Repository repo = repoManager.openRepository(project)) { - assertThat(repo.exactRef("refs/heads/master").getObjectId()).isEqualTo(rev2); - } - - assertRefUpdatedEvents(initialHead, headAfterFirstSubmit); - assertChangeMergedEvents( - change.getChangeId(), - headAfterFirstSubmit.name(), - change2.getChangeId(), - headAfterSecondSubmit.name()); - } -} |