blob: 7cc72af11ef49426ead662af27fd04cb8e5b3664 [file] [log] [blame]
// Copyright (C) 2015 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.gerrit.acceptance.GitUtil.pushHead;
import static com.google.gerrit.acceptance.testsuite.project.TestProjectUpdate.allow;
import static com.google.gerrit.acceptance.testsuite.project.TestProjectUpdate.allowLabel;
import static com.google.gerrit.acceptance.testsuite.project.TestProjectUpdate.block;
import static com.google.gerrit.server.group.SystemGroupBackend.REGISTERED_USERS;
import static com.google.gerrit.server.project.testing.TestLabels.value;
import static com.google.gerrit.testing.GerritJUnit.assertThrows;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Iterables;
import com.google.gerrit.acceptance.AbstractDaemonTest;
import com.google.gerrit.acceptance.GitUtil;
import com.google.gerrit.acceptance.NoHttpd;
import com.google.gerrit.acceptance.PushOneCommit;
import com.google.gerrit.acceptance.TestAccount;
import com.google.gerrit.acceptance.config.GerritConfig;
import com.google.gerrit.acceptance.testsuite.project.ProjectOperations;
import com.google.gerrit.acceptance.testsuite.request.RequestScopeOperations;
import com.google.gerrit.entities.BranchNameKey;
import com.google.gerrit.entities.LabelFunction;
import com.google.gerrit.entities.LabelId;
import com.google.gerrit.entities.LabelType;
import com.google.gerrit.entities.Permission;
import com.google.gerrit.extensions.api.changes.MoveInput;
import com.google.gerrit.extensions.api.changes.ReviewInput;
import com.google.gerrit.extensions.api.projects.BranchInput;
import com.google.gerrit.extensions.restapi.AuthException;
import com.google.gerrit.extensions.restapi.BadRequestException;
import com.google.gerrit.extensions.restapi.MethodNotAllowedException;
import com.google.gerrit.extensions.restapi.ResourceConflictException;
import com.google.gerrit.extensions.restapi.RestApiException;
import com.google.gerrit.server.project.testing.TestLabels;
import com.google.inject.Inject;
import java.util.Arrays;
import org.eclipse.jgit.junit.TestRepository;
import org.eclipse.jgit.lib.PersonIdent;
import org.eclipse.jgit.revwalk.RevCommit;
import org.junit.Test;
@NoHttpd
public class MoveChangeIT extends AbstractDaemonTest {
@Inject private ProjectOperations projectOperations;
@Inject private RequestScopeOperations requestScopeOperations;
@Test
public void moveChangeWithShortRef() throws Exception {
// Move change to a different branch using short ref name
PushOneCommit.Result r = createChange();
BranchNameKey newBranch = BranchNameKey.create(r.getChange().change().getProject(), "moveTest");
createBranch(newBranch);
move(r.getChangeId(), newBranch.shortName());
assertThat(r.getChange().change().getDest()).isEqualTo(newBranch);
}
@Test
public void moveChangeWithFullRef() throws Exception {
// Move change to a different branch using full ref name
PushOneCommit.Result r = createChange();
BranchNameKey newBranch = BranchNameKey.create(r.getChange().change().getProject(), "moveTest");
createBranch(newBranch);
move(r.getChangeId(), newBranch.branch());
assertThat(r.getChange().change().getDest()).isEqualTo(newBranch);
}
@Test
public void moveChangeWithMessage() throws Exception {
// Provide a message using --message flag
PushOneCommit.Result r = createChange();
BranchNameKey newBranch = BranchNameKey.create(r.getChange().change().getProject(), "moveTest");
createBranch(newBranch);
String moveMessage = "Moving for the move test";
move(r.getChangeId(), newBranch.branch(), moveMessage);
assertThat(r.getChange().change().getDest()).isEqualTo(newBranch);
StringBuilder expectedMessage = new StringBuilder();
expectedMessage.append("Change destination moved from master to moveTest");
expectedMessage.append("\n\n");
expectedMessage.append(moveMessage);
assertThat(Iterables.getLast(gApi.changes().id(r.getChangeId()).get().messages).message)
.isEqualTo(expectedMessage.toString());
}
@Test
public void moveChangeToSameRefAsCurrent() throws Exception {
// Move change to the branch same as change's destination
PushOneCommit.Result r = createChange();
ResourceConflictException thrown =
assertThrows(
ResourceConflictException.class,
() -> move(r.getChangeId(), r.getChange().change().getDest().branch()));
assertThat(thrown)
.hasMessageThat()
.contains("Change is already destined for the specified branch");
}
@Test
public void moveChangeToSameChangeId() throws Exception {
// Move change to a branch with existing change with same change ID
PushOneCommit.Result r = createChange();
BranchNameKey newBranch = BranchNameKey.create(r.getChange().change().getProject(), "moveTest");
createBranch(newBranch);
int changeNum = r.getChange().change().getChangeId();
createChange(newBranch.branch(), r.getChangeId());
ResourceConflictException thrown =
assertThrows(ResourceConflictException.class, () -> move(changeNum, newBranch.branch()));
assertThat(thrown)
.hasMessageThat()
.contains(
"Destination "
+ newBranch.shortName()
+ " has a different change with same change key "
+ r.getChangeId());
}
@Test
public void moveChangeToNonExistentRef() throws Exception {
// Move change to a non-existing branch
PushOneCommit.Result r = createChange();
BranchNameKey newBranch =
BranchNameKey.create(r.getChange().change().getProject(), "does_not_exist");
ResourceConflictException thrown =
assertThrows(
ResourceConflictException.class, () -> move(r.getChangeId(), newBranch.branch()));
assertThat(thrown)
.hasMessageThat()
.contains("Destination " + newBranch.branch() + " not found in the project");
}
@Test
public void moveClosedChange() throws Exception {
// Move a change which is not open
PushOneCommit.Result r = createChange();
BranchNameKey newBranch = BranchNameKey.create(r.getChange().change().getProject(), "moveTest");
createBranch(newBranch);
merge(r);
ResourceConflictException thrown =
assertThrows(
ResourceConflictException.class, () -> move(r.getChangeId(), newBranch.branch()));
assertThat(thrown).hasMessageThat().contains("Change is merged");
}
@Test
public void moveMergeCommitChange() throws Exception {
// Move a change which has a merge commit as the current PS
// Create a merge commit and push for review
PushOneCommit.Result r1 = createChange();
PushOneCommit.Result r2 = createChange();
TestRepository<?>.CommitBuilder commitBuilder =
testRepo.branch("HEAD").commit().insertChangeId();
commitBuilder
.parent(r1.getCommit())
.parent(r2.getCommit())
.message("Move change Merge Commit")
.author(admin.newIdent())
.committer(new PersonIdent(admin.newIdent(), testRepo.getDate()));
RevCommit c = commitBuilder.create();
pushHead(testRepo, "refs/for/master", false, false);
// Try to move the merge commit to another branch
BranchNameKey newBranch =
BranchNameKey.create(r1.getChange().change().getProject(), "moveTest");
createBranch(newBranch);
String changeId = GitUtil.getChangeId(testRepo, c).get();
move(changeId, newBranch.branch());
assertThat(gApi.changes().id(changeId).get().branch).isEqualTo("moveTest");
}
@Test
public void moveChangeToBranchWithoutUploadPerms() throws Exception {
// Move change to a destination where user doesn't have upload permissions
PushOneCommit.Result r = createChange();
BranchNameKey newBranch =
BranchNameKey.create(r.getChange().change().getProject(), "blocked_branch");
createBranch(newBranch);
projectOperations
.project(project)
.forUpdate()
.add(block(Permission.PUSH).ref("refs/for/" + newBranch.branch()).group(REGISTERED_USERS))
.update();
AuthException thrown =
assertThrows(AuthException.class, () -> move(r.getChangeId(), newBranch.branch()));
assertThat(thrown).hasMessageThat().isEqualTo("move not permitted");
}
@Test
public void moveChangeFromBranchWithoutAbandonPerms() throws Exception {
// Move change for which user does not have abandon permissions
PushOneCommit.Result r = createChange();
BranchNameKey newBranch = BranchNameKey.create(r.getChange().change().getProject(), "moveTest");
createBranch(newBranch);
projectOperations
.project(project)
.forUpdate()
.add(
block(Permission.ABANDON)
.ref(r.getChange().change().getDest().branch())
.group(REGISTERED_USERS))
.update();
requestScopeOperations.setApiUser(user.id());
AuthException thrown =
assertThrows(AuthException.class, () -> move(r.getChangeId(), newBranch.branch()));
assertThat(thrown).hasMessageThat().isEqualTo("move not permitted");
}
@Test
public void moveChangeToBranchThatContainsCurrentCommit() throws Exception {
// Move change to a branch for which current PS revision is reachable from
// tip
// Create a change
PushOneCommit.Result r = createChange();
int changeNum = r.getChange().change().getChangeId();
// Create a branch with that same commit
BranchNameKey newBranch = BranchNameKey.create(r.getChange().change().getProject(), "moveTest");
BranchInput bi = new BranchInput();
bi.revision = r.getCommit().name();
gApi.projects().name(newBranch.project().get()).branch(newBranch.branch()).create(bi);
// Try to move the change to the branch with the same commit
ResourceConflictException thrown =
assertThrows(ResourceConflictException.class, () -> move(changeNum, newBranch.branch()));
assertThat(thrown)
.hasMessageThat()
.contains("Current patchset revision is reachable from tip of " + newBranch.branch());
}
@Test
public void moveChangeWithCurrentPatchSetLocked() throws Exception {
// Move change that is locked
PushOneCommit.Result r = createChange();
BranchNameKey newBranch = BranchNameKey.create(r.getChange().change().getProject(), "moveTest");
createBranch(newBranch);
LabelType patchSetLock = TestLabels.patchSetLock();
try (ProjectConfigUpdate u = updateProject(project)) {
u.getConfig().upsertLabelType(patchSetLock);
u.save();
}
projectOperations
.project(project)
.forUpdate()
.add(
allowLabel(patchSetLock.getName())
.ref("refs/heads/*")
.group(REGISTERED_USERS)
.range(0, 1))
.update();
revision(r).review(new ReviewInput().label("Patch-Set-Lock", 1));
ResourceConflictException thrown =
assertThrows(
ResourceConflictException.class, () -> move(r.getChangeId(), newBranch.branch()));
assertThat(thrown)
.hasMessageThat()
.contains(
String.format("The current patch set of change %s is locked", r.getChange().getId()));
}
@Test
public void moveChangeKeepAllVotesOnlyAllowedForAdmins() throws Exception {
// Keep all votes options is only permitted for admins.
BranchNameKey destinationBranch = BranchNameKey.create(project, "dest");
createBranch(destinationBranch);
BranchNameKey sourceBranch = BranchNameKey.create(project, "source");
createBranch(sourceBranch);
String changeId = createChangeInBranch(sourceBranch.branch()).getChangeId();
// Grant change permissions to the registered users.
projectOperations
.project(project)
.forUpdate()
.add(allow(Permission.PUSH).ref(destinationBranch.branch()).group(REGISTERED_USERS))
.update();
projectOperations
.project(project)
.forUpdate()
.add(allow(Permission.ABANDON).ref(sourceBranch.branch()).group(REGISTERED_USERS))
.update();
requestScopeOperations.setApiUser(user.id());
AuthException thrown =
assertThrows(
AuthException.class, () -> move(changeId, destinationBranch.shortName(), true));
assertThat(thrown).hasMessageThat().isEqualTo("move is not permitted with keepAllVotes option");
requestScopeOperations.setApiUser(admin.id());
move(changeId, destinationBranch.branch(), true);
assertThat(info(changeId).branch).isEqualTo(destinationBranch.shortName());
}
@Test
public void moveChangeKeepAllVotesNoLabelInDestination() throws Exception {
BranchNameKey destinationBranch = BranchNameKey.create(project, "dest");
createBranch(destinationBranch);
BranchNameKey sourceBranch = BranchNameKey.create(project, "source");
createBranch(sourceBranch);
String testLabelA = "Label-A";
// The label has the range [-1; 1]
configLabel(testLabelA, LabelFunction.NO_BLOCK, ImmutableList.of(sourceBranch.branch()));
// Registered users have permissions for the entire range [-1; 1] on all branches.
projectOperations
.project(project)
.forUpdate()
.add(allowLabel(testLabelA).ref("refs/heads/*").group(REGISTERED_USERS).range(-1, +1))
.update();
String changeId = createChangeInBranch(sourceBranch.branch()).getChangeId();
requestScopeOperations.setApiUser(user.id());
ReviewInput userReviewInput = new ReviewInput();
userReviewInput.label(testLabelA, 1);
gApi.changes().id(changeId).current().review(userReviewInput);
assertLabelVote(user, changeId, testLabelA, (short) 1);
requestScopeOperations.setApiUser(admin.id());
assertThat(localCtx.getContext().getUser().getAccountId()).isEqualTo(admin.id());
// Move the change to the destination branch.
assertThat(info(changeId).branch).isEqualTo(sourceBranch.shortName());
move(changeId, destinationBranch.shortName(), true);
assertThat(info(changeId).branch).isEqualTo(destinationBranch.shortName());
// Label is missing in the destination branch.
assertThat(gApi.changes().id(changeId).current().reviewer(user.email()).votes()).isEmpty();
// Move the change back to the source, the label is kept.
move(changeId, sourceBranch.shortName(), true);
assertThat(info(changeId).branch).isEqualTo(sourceBranch.shortName());
assertLabelVote(user, changeId, testLabelA, (short) 1);
}
@Test
public void moveChangeKeepAllVotesOutOfUserPermissionRange() throws Exception {
BranchNameKey destinationBranch = BranchNameKey.create(project, "dest");
createBranch(destinationBranch);
BranchNameKey sourceBranch = BranchNameKey.create(project, "source");
createBranch(sourceBranch);
String testLabelA = "Label-A";
// The label has the range [-2; 2]
configLabel(
project,
testLabelA,
LabelFunction.NO_BLOCK,
value(2, "Passes"),
value(1, "Mostly ok"),
value(0, "No score"),
value(-1, "Needs work"),
value(-2, "Failed"));
// Registered users have [-2; 2] permissions on the source.
projectOperations
.project(project)
.forUpdate()
.add(
allowLabel(testLabelA).ref(sourceBranch.branch()).group(REGISTERED_USERS).range(-2, +2))
.update();
// Registered users have [-1; 1] permissions on the destination.
projectOperations
.project(project)
.forUpdate()
.add(
allowLabel(testLabelA)
.ref(destinationBranch.branch())
.group(REGISTERED_USERS)
.range(-1, +1))
.update();
String changeId = createChangeInBranch(sourceBranch.branch()).getChangeId();
requestScopeOperations.setApiUser(user.id());
// Vote within the range of the source branch.
ReviewInput userReviewInput = new ReviewInput();
userReviewInput.label(testLabelA, 2);
gApi.changes().id(changeId).current().review(userReviewInput);
assertLabelVote(user, changeId, testLabelA, (short) 2);
requestScopeOperations.setApiUser(admin.id());
assertThat(localCtx.getContext().getUser().getAccountId()).isEqualTo(admin.id());
// Move the change to the destination branch.
assertThat(info(changeId).branch).isEqualTo(sourceBranch.shortName());
move(changeId, destinationBranch.branch(), true);
// User does not have label permissions for the same vote on the destination branch.
requestScopeOperations.setApiUser(user.id());
AuthException thrown =
assertThrows(
AuthException.class,
() -> gApi.changes().id(changeId).current().review(userReviewInput));
assertThat(thrown)
.hasMessageThat()
.isEqualTo(String.format("Applying label \"%s\": 2 is restricted", testLabelA));
// Label is kept even though the user's permission range is different from the source.
// Since we do not squash users votes based on the destination branch access label
// configuration, this is working as intended.
// It's the same behavior as when a project owner reduces user's permission range on label.
// Administrators should take this into account.
assertThat(info(changeId).branch).isEqualTo(destinationBranch.shortName());
assertLabelVote(user, changeId, testLabelA, (short) 2);
requestScopeOperations.setApiUser(admin.id());
// Move the change back to the source, the label is kept.
move(changeId, sourceBranch.shortName(), true);
assertThat(info(changeId).branch).isEqualTo(sourceBranch.shortName());
assertLabelVote(user, changeId, testLabelA, (short) 2);
}
@Test
public void moveKeepAllVotesCanMoveAllInRange() throws Exception {
BranchNameKey destinationBranch = BranchNameKey.create(project, "dest");
createBranch(destinationBranch);
BranchNameKey sourceBranch = BranchNameKey.create(project, "source");
createBranch(sourceBranch);
// The non-block label has the range [-2; 2]
String testLabelA = "Label-A";
configLabel(
project,
testLabelA,
LabelFunction.NO_BLOCK,
value(2, "Passes"),
value(1, "Mostly ok"),
value(0, "No score"),
value(-1, "Needs work"),
value(-2, "Failed"));
// Registered users have [-2; 2] permissions on all branches.
projectOperations
.project(project)
.forUpdate()
.add(allowLabel(testLabelA).ref("refs/heads/*").group(REGISTERED_USERS).range(-2, +2))
.update();
String changeId = createChangeInBranch(sourceBranch.branch()).getChangeId();
for (int vote = -2; vote <= 2; vote++) {
TestAccount testUser = accountCreator.create("TestUser" + vote);
requestScopeOperations.setApiUser(testUser.id());
ReviewInput userReviewInput = new ReviewInput();
userReviewInput.label(testLabelA, vote);
gApi.changes().id(changeId).current().review(userReviewInput);
}
assertThat(
gApi.changes().id(changeId).current().votes().get(testLabelA).stream()
.map(approvalInfo -> approvalInfo.value)
.collect(ImmutableList.toImmutableList()))
.containsExactly(-2, -1, 1, 2);
requestScopeOperations.setApiUser(admin.id());
// Move the change to the destination branch.
assertThat(info(changeId).branch).isEqualTo(sourceBranch.shortName());
move(changeId, destinationBranch.shortName(), true);
assertThat(info(changeId).branch).isEqualTo(destinationBranch.shortName());
// All votes are kept
assertThat(
gApi.changes().id(changeId).current().votes().get(testLabelA).stream()
.map(approvalInfo -> approvalInfo.value)
.collect(ImmutableList.toImmutableList()))
.containsExactly(-2, -1, 1, 2);
// Move the change back to the source, the label is kept.
move(changeId, sourceBranch.shortName(), true);
assertThat(info(changeId).branch).isEqualTo(sourceBranch.shortName());
assertThat(
gApi.changes().id(changeId).current().votes().get(testLabelA).stream()
.map(approvalInfo -> approvalInfo.value)
.collect(ImmutableList.toImmutableList()))
.containsExactly(-2, -1, 1, 2);
}
@Test
public void moveChangeOnlyKeepVetoVotes() throws Exception {
// A vote for a label will be kept after moving if the label's function is *WithBlock and the
// vote holds the minimum value.
createBranch(BranchNameKey.create(project, "foo"));
String codeReviewLabel = LabelId.CODE_REVIEW; // 'Code-Review' uses 'MaxWithBlock' function.
String testLabelA = "Label-A";
String testLabelB = "Label-B";
String testLabelC = "Label-C";
configLabel(testLabelA, LabelFunction.ANY_WITH_BLOCK);
configLabel(testLabelB, LabelFunction.MAX_NO_BLOCK);
configLabel(testLabelC, LabelFunction.NO_BLOCK);
projectOperations
.project(project)
.forUpdate()
.add(allowLabel(testLabelA).ref("refs/heads/*").group(REGISTERED_USERS).range(-1, +1))
.add(allowLabel(testLabelB).ref("refs/heads/*").group(REGISTERED_USERS).range(-1, +1))
.add(allowLabel(testLabelC).ref("refs/heads/*").group(REGISTERED_USERS).range(-1, +1))
.update();
String changeId = createChange().getChangeId();
gApi.changes().id(changeId).current().review(ReviewInput.reject());
amendChange(changeId);
ReviewInput input = new ReviewInput();
input.label(testLabelA, -1);
input.label(testLabelB, -1);
input.label(testLabelC, -1);
gApi.changes().id(changeId).current().review(input);
assertThat(gApi.changes().id(changeId).current().reviewer(admin.email()).votes().keySet())
.containsExactly(codeReviewLabel, testLabelA, testLabelB, testLabelC);
assertThat(gApi.changes().id(changeId).current().reviewer(admin.email()).votes().values())
.containsExactly((short) -2, (short) -1, (short) -1, (short) -1);
// Move the change to the 'foo' branch.
assertThat(gApi.changes().id(changeId).get().branch).isEqualTo("master");
move(changeId, "foo");
assertThat(gApi.changes().id(changeId).get().branch).isEqualTo("foo");
// 'Code-Review -2' and 'Label-A -1' will be kept.
assertThat(gApi.changes().id(changeId).current().reviewer(admin.email()).votes().values())
.containsExactly((short) -2, (short) -1, (short) 0, (short) 0);
// Move the change back to 'master'.
move(changeId, "master");
assertThat(gApi.changes().id(changeId).get().branch).isEqualTo("master");
assertThat(gApi.changes().id(changeId).current().reviewer(admin.email()).votes().values())
.containsExactly((short) -2, (short) -1, (short) 0, (short) 0);
}
@Test
public void moveToBranchThatDoesNotHaveCustomLabel() throws Exception {
createBranch(BranchNameKey.create(project, "foo"));
String testLabelA = "Label-A";
configLabel(testLabelA, LabelFunction.MAX_WITH_BLOCK, Arrays.asList("refs/heads/master"));
projectOperations
.project(project)
.forUpdate()
.add(allowLabel(testLabelA).ref("refs/heads/master").group(REGISTERED_USERS).range(-1, +1))
.update();
String changeId = createChange().getChangeId();
ReviewInput input = new ReviewInput();
input.label(testLabelA, -1);
gApi.changes().id(changeId).current().review(input);
assertThat(gApi.changes().id(changeId).current().reviewer(admin.email()).votes().keySet())
.containsExactly(testLabelA);
assertThat(gApi.changes().id(changeId).current().reviewer(admin.email()).votes().values())
.containsExactly((short) -1);
move(changeId, "foo");
// "foo" branch does not have the custom label
assertThat(gApi.changes().id(changeId).current().reviewer(admin.email()).votes().keySet())
.isEmpty();
// Move back to master and confirm that the custom label score is still there
move(changeId, "master");
assertThat(gApi.changes().id(changeId).current().reviewer(admin.email()).votes().keySet())
.containsExactly(testLabelA);
assertThat(gApi.changes().id(changeId).current().reviewer(admin.email()).votes().values())
.containsExactly((short) -1);
}
@Test
public void moveNoDestinationBranchSpecified() throws Exception {
PushOneCommit.Result r = createChange();
BadRequestException thrown =
assertThrows(BadRequestException.class, () -> move(r.getChangeId(), null));
assertThat(thrown).hasMessageThat().contains("destination branch is required");
}
@Test
@GerritConfig(name = "change.move", value = "false")
public void moveCanBeDisabledByConfig() throws Exception {
PushOneCommit.Result r = createChange();
MethodNotAllowedException thrown =
assertThrows(MethodNotAllowedException.class, () -> move(r.getChangeId(), null));
assertThat(thrown).hasMessageThat().contains("move changes endpoint is disabled");
}
private void move(int changeNum, String destination) throws RestApiException {
gApi.changes().id(changeNum).move(destination);
}
private void move(String changeId, String destination) throws RestApiException {
gApi.changes().id(changeId).move(destination);
}
private void move(String changeId, String destination, String message) throws RestApiException {
MoveInput in = new MoveInput();
in.destinationBranch = destination;
in.message = message;
gApi.changes().id(changeId).move(in);
}
private void move(String changeId, String destination, boolean keepAllVotes)
throws RestApiException {
MoveInput in = new MoveInput();
in.destinationBranch = destination;
in.keepAllVotes = keepAllVotes;
gApi.changes().id(changeId).move(in);
}
private PushOneCommit.Result createChange(String branch, String changeId) throws Exception {
PushOneCommit push = pushFactory.create(admin.newIdent(), testRepo, changeId);
PushOneCommit.Result result = push.to("refs/for/" + branch);
result.assertOkStatus();
return result;
}
private PushOneCommit.Result createChangeInBranch(String branch) throws Exception {
return createChange("refs/for/" + branch);
}
private void assertLabelVote(TestAccount user, String changeId, String label, short vote)
throws Exception {
assertThat(gApi.changes().id(changeId).current().reviewer(user.email()).votes())
.containsEntry(label, vote);
}
}