diff options
author | Patrick Hiesel <hiesel@google.com> | 2023-08-22 08:46:04 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2023-08-22 08:46:04 +0000 |
commit | aba53b0d951f8e8d9ffded6d0c0d6e0966acbce6 (patch) | |
tree | f0c79ecaf5f464464db9163a14630628c4e68da4 | |
parent | 62ac7c8a86ec5649577523971b875c393e3f8773 (diff) | |
parent | adc003103d9416f2efef9e03bac4146d86516c00 (diff) |
Merge "Fix change message when change is created as WiP revert"
3 files changed, 8 insertions, 2 deletions
diff --git a/java/com/google/gerrit/server/restapi/change/SetReadyForReview.java b/java/com/google/gerrit/server/restapi/change/SetReadyForReview.java index 7d3fe983f8..97b08f0b7d 100644 --- a/java/com/google/gerrit/server/restapi/change/SetReadyForReview.java +++ b/java/com/google/gerrit/server/restapi/change/SetReadyForReview.java @@ -77,7 +77,7 @@ public class SetReadyForReview bu.addOp(rsrc.getChange().getId(), opFactory.create(false, input)); if (change.getRevertOf() != null) { commitUtil.addChangeRevertedNotificationOps( - bu, change.getRevertOf(), change.getId(), change.getKey().get()); + bu, change.getRevertOf(), change.getId(), change.getKey().get().substring(1)); } bu.execute(); return Response.ok(); diff --git a/javatests/com/google/gerrit/acceptance/api/change/ChangeIT.java b/javatests/com/google/gerrit/acceptance/api/change/ChangeIT.java index c9208430ef..3a835b78b4 100644 --- a/javatests/com/google/gerrit/acceptance/api/change/ChangeIT.java +++ b/javatests/com/google/gerrit/acceptance/api/change/ChangeIT.java @@ -448,7 +448,7 @@ public class ChangeIT extends AbstractDaemonTest { List<ChangeMessageInfo> sourceMessages = new ArrayList<>(gApi.changes().id(r.getChangeId()).get().messages); assertThat(sourceMessages).hasSize(4); - String expectedMessage = String.format("Created a revert of this change as I%s", changeId); + String expectedMessage = String.format("Created a revert of this change as %s", changeId); assertThat(sourceMessages.get(3).message).isEqualTo(expectedMessage); } diff --git a/javatests/com/google/gerrit/acceptance/api/change/RevertIT.java b/javatests/com/google/gerrit/acceptance/api/change/RevertIT.java index 4855ba4b0b..7c50e93819 100644 --- a/javatests/com/google/gerrit/acceptance/api/change/RevertIT.java +++ b/javatests/com/google/gerrit/acceptance/api/change/RevertIT.java @@ -225,6 +225,12 @@ public class RevertIT extends AbstractDaemonTest { List<ChangeMessageInfo> sourceMessages = new ArrayList<>(gApi.changes().id(r.getChangeId()).get().messages); assertThat(sourceMessages).hasSize(3); + // Publishing creates a revert message + gApi.changes().id(revertChange.changeId).setReadyForReview(); + sourceMessages = new ArrayList<>(gApi.changes().id(r.getChangeId()).get().messages); + assertThat(sourceMessages).hasSize(4); + assertThat(sourceMessages.get(3).message) + .isEqualTo("Created a revert of this change as " + revertChange.changeId); } @Test |