commit f4b51bb1e020930a5edfa1ea39cba40afcd613b2
author Hikaru Yoshimura <h1k4ruy@gmail.com> Fri Nov 04 12:43:37 2016 +0900
committer Hikaru Yoshimura <h1k4ruy@gmail.com> Fri Nov 04 12:45:22 2016 +0900
tree dbdd8a7fca0ad5daf6c016bd56aeb2cb09e64b42
parent 4f3ad6d289e529d4c044f5fd56c6ffb6cceeee9b

add the owner name to the comment added message

Change-Id: If4d232fb6305b8fdea3d1f902c9328e691cee8f0

src/main/java/com/cisco/gerrit/plugins/slack/message/CommentAddedMessageGenerator.java

diff --git a/src/main/java/com/cisco/gerrit/plugins/slack/message/CommentAddedMessageGenerator.java b/src/main/java/com/cisco/gerrit/plugins/slack/message/CommentAddedMessageGenerator.java
index 5379a10..90f9ff1 100644
--- a/src/main/java/com/cisco/gerrit/plugins/slack/message/CommentAddedMessageGenerator.java
+++ b/src/main/java/com/cisco/gerrit/plugins/slack/message/CommentAddedMessageGenerator.java
@@ -81,7 +81,9 @@
             text = new StringBuilder();
 
             text.append(escape(event.author.get().name));
-            text.append(" commented\\n>>>");
+            text.append(" commented to ");
+            text.append(escape(event.change.get().owner.name));
+            text.append("\\n>>>");
             text.append(escape(event.change.get().project));
             text.append(" (");
             text.append(escape(event.change.get().branch));

src/test/java/com/cisco/gerrit/plugins/slack/message/CommentAddedMessageGeneratorTest.java

diff --git a/src/test/java/com/cisco/gerrit/plugins/slack/message/CommentAddedMessageGeneratorTest.java b/src/test/java/com/cisco/gerrit/plugins/slack/message/CommentAddedMessageGeneratorTest.java
index dce6fc7..c1cf569 100644
--- a/src/test/java/com/cisco/gerrit/plugins/slack/message/CommentAddedMessageGeneratorTest.java
+++ b/src/test/java/com/cisco/gerrit/plugins/slack/message/CommentAddedMessageGeneratorTest.java
@@ -61,6 +61,7 @@
     private CommentAddedEvent mockEvent = mock(CommentAddedEvent.class);
     private AccountAttribute mockAccount = mock(AccountAttribute.class);
     private ChangeAttribute mockChange = mock(ChangeAttribute.class);
+    private AccountAttribute mockOwner = mock(AccountAttribute.class);
 
     private ProjectConfig config;
 
@@ -157,6 +158,8 @@
         mockChange.project = "testproject";
         mockChange.branch = "master";
         mockChange.url = "https://change/";
+        mockChange.owner = mockOwner;
+        mockOwner.name = "Owner";
 
         mockAccount.name = "Unit Tester";
 
@@ -166,7 +169,7 @@
                 mockEvent, config);
 
         String expectedResult;
-        expectedResult = "{\"text\": \"Unit Tester commented\\n>>>" +
+        expectedResult = "{\"text\": \"Unit Tester commented to Owner\\n>>>" +
                 "testproject (master): This is the first line\n" +
                 "And the second line. (https://change/)\"," +
                 "\"channel\": \"#testchannel\",\"username\": \"testuser\"}\n";
@@ -192,6 +195,8 @@
         mockChange.project = "testproject";
         mockChange.branch = "master";
         mockChange.url = "https://change/";
+        mockChange.owner = mockOwner;
+        mockOwner.name = "Owner";
 
         mockAccount.name = "Unit Tester";
 
@@ -201,7 +206,7 @@
                 mockEvent, config);
 
         String expectedResult;
-        expectedResult = "{\"text\": \"Unit Tester commented\\n>>>" +
+        expectedResult = "{\"text\": \"Unit Tester commented to Owner\\n>>>" +
                 "testproject (master): " + mockEvent.comment.substring(0, 197) + "... (https://change/)\"," +
                 "\"channel\": \"#testchannel\",\"username\": \"testuser\"}\n";