opendev/gerrit
Code Issues Proposed changes

Merge "Rename revert message on original change"

Browse Source
This commit is contained in:
David Pursehouse
2016-02-23 00:27:31 +00:00
committed by Gerrit Code Review
parent 56a55daf31 1616a6490e
commit 6e87fc2c84
2 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/change/ChangeIT.java
View File

@@ -190,8 +190,7 @@ public class ChangeIT extends AbstractDaemonTest {
gApi.changes().id(r.getChangeId()).get().messages);
assertThat(sourceMessages).hasSize(4);
String expectedMessage = String.format(
"Patch Set 1: Reverted\n\n" +
"This patchset was reverted in change: %s",
"Created a revert of this change as %s",
revertChange.changeId);
assertThat(sourceMessages.get(3).message).isEqualTo(expectedMessage);

5
gerrit-server/src/main/java/com/google/gerrit/server/change/Revert.java
View File

@@ -249,10 +249,7 @@ public class Revert implements RestModifyView<ChangeResource, RevertInput>,
ChangeUtil.messageUUID(db.get())),
ctx.getUser().getAccountId(), ctx.getWhen(), patchSetId);
StringBuilder msgBuf = new StringBuilder();
msgBuf.append("Patch Set ").append(patchSetId.get())
.append(": Reverted");
msgBuf.append("\n\n");
msgBuf.append("This patchset was reverted in change: ")
msgBuf.append("Created a revert of this change as ")
.append("I").append(computedChangeId.name());
changeMessage.setMessage(msgBuf.toString());
cmUtil.addChangeMessage(ctx.getDb(), ctx.getUpdate(patchSetId),