commit b318c23f6ee52bde8aecfeb105adb47f7c891d76
author Marija Savtchouk <mariasavtchouk@google.com> Tue Aug 03 11:06:43 2021 +0100
committer Marija Savtchouk <mariasavtchouk@google.com> Tue Aug 03 11:06:43 2021 +0100
tree 96d319b388b0820b0f195fb608b0799f57837297
parent 0e9423a965b7dea672fd1cd1f253d919893224ec

Fix change message syntax

Change-Id: I6dd9ad6dc56aa05444d8dd945270fd21a70b5e97

java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnAddReviewer.java

diff --git a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnAddReviewer.java b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnAddReviewer.java
index b82e3c3..58ad10e 100644
--- a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnAddReviewer.java
+++ b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnAddReviewer.java
@@ -177,7 +177,7 @@
       StringBuilder message = new StringBuilder();
       message.append(
           String.format(
-              "%s who was added as reviewer owns the following files:\n",
+              "%s, who was added as reviewer owns the following files:\n",
               ChangeMessagesUtil.getAccountTemplate(reviewerAccountId)));
 
       if (ownedPaths.size() <= limit) {

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

diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnAddReviewerIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnAddReviewerIT.java
index 5d60df9..b0890c1 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnAddReviewerIT.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnAddReviewerIT.java
@@ -89,7 +89,7 @@
     assertThat(Iterables.getLast(messages).message)
         .isEqualTo(
             String.format(
-                "%s who was added as reviewer owns the following files:\n* %s\n",
+                "%s, who was added as reviewer owns the following files:\n* %s\n",
                 ChangeMessagesUtil.getAccountTemplate(user.id()), path));
   }
 
@@ -125,7 +125,7 @@
     assertThat(Iterables.getLast(messages).message)
         .isEqualTo(
             String.format(
-                "%s who was added as reviewer owns the following files:\n* %s\n* %s\n",
+                "%s, who was added as reviewer owns the following files:\n* %s\n* %s\n",
                 ChangeMessagesUtil.getAccountTemplate(user.id()), path1, path2));
   }
 
@@ -188,7 +188,7 @@
     assertThat(Iterables.getLast(messages).message)
         .isEqualTo(
             String.format(
-                "%s who was added as reviewer owns the following files:\n"
+                "%s, who was added as reviewer owns the following files:\n"
                     + "* %s\n"
                     + "* %s\n"
                     + "* %s\n"
@@ -234,7 +234,7 @@
     assertThat(Iterables.getLast(messages).message)
         .isEqualTo(
             String.format(
-                "%s who was added as reviewer owns the following files:\n"
+                "%s, who was added as reviewer owns the following files:\n"
                     + "* %s\n"
                     + "* %s\n"
                     + "* %s\n"
@@ -321,7 +321,7 @@
     assertThat(Iterables.getLast(messages).message)
         .isEqualTo(
             String.format(
-                "%s who was added as reviewer owns the following files:\n* %s\n",
+                "%s, who was added as reviewer owns the following files:\n* %s\n",
                 ChangeMessagesUtil.getAccountTemplate(user.id()), path));
   }
 
@@ -357,7 +357,7 @@
     assertThat(Iterables.getLast(messages).message)
         .isEqualTo(
             String.format(
-                "%s who was added as reviewer owns the following files:\n* %s\n",
+                "%s, who was added as reviewer owns the following files:\n* %s\n",
                 ChangeMessagesUtil.getAccountTemplate(user.id()), path));
   }
 }