#1262 handle only created comments from Github

self-xdsd · Oct 30, 2021 · cfe1fb2 · cfe1fb2
1 parent 9d2dd94
commit cfe1fb2
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 24 deletions.
diff --git a/self-api/src/main/java/com/selfxdsd/api/Event.java b/self-api/src/main/java/com/selfxdsd/api/Event.java
@@ -106,7 +106,7 @@ private Type(){}
         /**
          * Event for a comment.
          */
-        public static final String ISSUE_COMMENT = "issue_comment";
+        public static final String ISSUE_COMMENT = "comment_created";
 
         /**
          * Event for renaming a repo.

diff --git a/self-core-impl/src/main/java/com/selfxdsd/core/projects/GithubWebhookEvent.java b/self-core-impl/src/main/java/com/selfxdsd/core/projects/GithubWebhookEvent.java
@@ -34,6 +34,7 @@
  * @author Mihai Andronache (amihaiemil@gmail.com)
  * @version $Id$
  * @since 0.0.61
+ * @checkstyle CyclomaticComplexity (500 lines)
  */
 final class GithubWebhookEvent implements Event {
 
@@ -87,6 +88,13 @@ public String type() {
             } else {
                 resolved = type;
             }
+        } else if ("issue_comment".equalsIgnoreCase(type)) {
+            final String act = event.getString("action");
+            if("created".equalsIgnoreCase(act)) {
+                resolved = Type.ISSUE_COMMENT;
+            } else {
+                resolved = type;
+            }
         } else if("repository".equalsIgnoreCase(type)){
             final String act = event.getString("action");
             if("renamed".equalsIgnoreCase(act)) {

diff --git a/self-core-impl/src/test/java/com/selfxdsd/core/projects/GithubWebhookEventTestCase.java b/self-core-impl/src/test/java/com/selfxdsd/core/projects/GithubWebhookEventTestCase.java
@@ -253,29 +253,6 @@ public void returnsReopenedPullRequestType() {
         );
     }
 
-    /**
-     * GithubWebhookEvent can return label event type when
-     * an issue label is added, edited or deleted.
-     */
-    @Test
-    public void returnsLabelType(){
-        final Project project = Mockito.mock(Project.class);
-        final Event event = new GithubWebhookEvent(
-            project, "label",
-            Json.createObjectBuilder()
-                .add("action", "created")
-                .add("label", Json.createObjectBuilder()
-                    .add("name", "60min")
-                    .build())
-                .build()
-                .toString()
-        );
-        MatcherAssert.assertThat(
-            event.type(),
-            Matchers.equalTo(Event.Type.LABEL)
-        );
-    }
-
     /**
      * GithubWebhookEvent can return its Issue.
      */
@@ -457,4 +434,20 @@ public void returnsNullCommitForNonPush() {
             Matchers.nullValue()
         );
     }
+
+    /**
+     * GithubWebhookEvent can return the ISSUE_COMMENT type for a comment
+     * created action.
+     */
+    @Test
+    public void returnsIssueCommentType() {
+        final Project project = Mockito.mock(Project.class);
+        final Event event = new GithubWebhookEvent(
+            project, "issue_comment", "{\"action\":\"created\"}"
+        );
+        MatcherAssert.assertThat(
+            event.type(),
+            Matchers.equalTo(Event.Type.ISSUE_COMMENT)
+        );
+    }
 }