add unit test for gerrit patchset create

Author: gy2006

core/src/main/java/com/flowci/core/githook/converter/GerritConverter.java

@@ -76,7 +76,7 @@ private static class Event {
 
     private static class PatchSetCreateEvent implements GitTriggerable {
 
-        public PathSet pathSet;
+        public PatchSet patchSet;
 
         public Change change;
 
@@ -94,20 +94,20 @@ public GitTrigger toTrigger() {
             t.setChangeNumber(change.number);
             t.setChangeUrl(change.url);
 
-            t.setPatchNumber(pathSet.number);
-            t.setPatchUrl(change.url + "/" + pathSet.number);
-            t.setRevision(pathSet.revision);
-            t.setRef(pathSet.ref);
-            t.setCreatedOn(pathSet.createdOn);
-            t.setAuthor(pathSet.author.toGitUser());
-            t.setSizeInsertions(pathSet.sizeInsertions);
-            t.setSizeDeletions(pathSet.sizeDeletions);
+            t.setPatchNumber(patchSet.number);
+            t.setPatchUrl(change.url + "/" + patchSet.number);
+            t.setRevision(patchSet.revision);
+            t.setRef(patchSet.ref);
+            t.setCreatedOn(patchSet.createdOn);
+            t.setAuthor(patchSet.author.toGitUser());
+            t.setSizeInsertions(patchSet.sizeInsertions);
+            t.setSizeDeletions(patchSet.sizeDeletions);
 
             return t;
         }
     }
 
-    private static class PathSet {
+    private static class PatchSet {
 
         public Integer number;
 

core/src/main/java/com/flowci/core/githook/converter/TriggerConverter.java

@@ -46,6 +46,7 @@ public Optional<GitTrigger> convert(String event, InputStream body) {
             GitTrigger trigger = mapping.get(event).apply(body);
             return Optional.ofNullable(trigger);
         } catch (Exception e) {
+            log.warn(e.getMessage());
             return Optional.empty();
         }
     }

core/src/test/java/com/flowci/core/test/githook/GerritConverterTest.java

@@ -0,0 +1,44 @@
+package com.flowci.core.test.githook;
+
+import com.flowci.core.githook.converter.GerritConverter;
+import com.flowci.core.githook.domain.GitPatchSetTrigger;
+import com.flowci.core.githook.domain.GitTrigger;
+import com.flowci.core.test.SpringScenario;
+import org.junit.Assert;
+import org.junit.Test;
+import org.springframework.beans.factory.annotation.Autowired;
+
+import java.io.InputStream;
+import java.util.Optional;
+
+public class GerritConverterTest extends SpringScenario {
+
+    @Autowired
+    private GerritConverter gerritConverter;
+
+    @Test
+    public void should_parse_patchset_create_event() {
+        InputStream stream = load("gerrit/patchset_created.json");
+
+        Optional<GitTrigger> optional = gerritConverter.convert(GerritConverter.AllEvent, stream);
+        Assert.assertTrue(optional.isPresent());
+        Assert.assertTrue(optional.get() instanceof GitPatchSetTrigger);
+
+        GitPatchSetTrigger t = (GitPatchSetTrigger) optional.get();
+        Assert.assertEquals("task 1 try", t.getSubject());
+        Assert.assertEquals("task 1 try\n\nChange-Id: I508fde11a59c36f9ab8b086d13d41b5d9c597db6\n", t.getMessage());
+        Assert.assertEquals("gerrit_test", t.getProject());
+        Assert.assertEquals("master", t.getBranch());
+        Assert.assertEquals("I508fde11a59c36f9ab8b086d13d41b5d9c597db6", t.getChangeId());
+        Assert.assertEquals(1, t.getChangeNumber().intValue());
+        Assert.assertEquals("http://192.168.31.173:8088/c/gerrit_test/+/1", t.getChangeUrl());
+
+        Assert.assertEquals(2, t.getPatchNumber().intValue());
+        Assert.assertEquals("http://192.168.31.173:8088/c/gerrit_test/+/1/2", t.getPatchUrl());
+        Assert.assertEquals("34ebc1c63ef2173b3704663c32bcd14471b68a9b", t.getRevision());
+        Assert.assertEquals("refs/changes/01/1/2", t.getRef());
+        Assert.assertEquals("1642965660", t.getCreatedOn());
+        Assert.assertEquals(3, t.getSizeInsertions().intValue());
+        Assert.assertEquals(0, t.getSizeDeletions().intValue());
+    }
+}