commit acb71d801ac01db54598632391b954ec59926254
author Edwin Kempin <ekempin@google.com> Mon Jan 11 09:03:31 2021 +0100
committer Edwin Kempin <ekempin@google.com> Fri Jan 15 15:12:39 2021 +0100
tree 38d0570aa455cc4d586ad3160529d3c3c01d3525
parent a1226ac93620f3ded94d99adf94999f802a1f7d4

Test that change message is not extended when voting on old patch set

This slightly increases the test coverage for CodeOwnersOnPostReview
from 90.2% to 91.2%.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: Id3dceb604606778e7aea7a78aded021f8c76e18f

javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnPostReviewIT.java

diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnPostReviewIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnPostReviewIT.java
index 9160b32..c5afe3c 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnPostReviewIT.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnPostReviewIT.java
@@ -623,4 +623,26 @@
                     + "* %s\n",
                 admin.fullName(), path));
   }
+
+  @Test
+  public void changeMessageNotExtendedIfUsersPostsOnOldPatchSet() throws Exception {
+    codeOwnerConfigOperations
+        .newCodeOwnerConfig()
+        .project(project)
+        .branch("master")
+        .folderPath("/foo/")
+        .addCodeOwnerEmail(admin.email())
+        .create();
+
+    String changeId = createChange("Test Change", "foo/bar.baz", "file content").getChangeId();
+
+    // create a second patch set
+    amendChange(changeId);
+
+    // vote on the first patch set
+    gApi.changes().id(changeId).revision(1).review(ReviewInput.recommend());
+
+    Collection<ChangeMessageInfo> messages = gApi.changes().id(changeId).get().messages;
+    assertThat(Iterables.getLast(messages).message).isEqualTo("Patch Set 1: Code-Review+1");
+  }
 }