feat: react to manifest file creation via webhook

- Added location for logs - Added manifest file path in payload
leanix · Nov 4, 2024 · 1cd5b16 · 1cd5b16
1 parent 59ffd77
commit 1cd5b16
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 4 deletions.
diff --git a/src/main/kotlin/net/leanix/githubagent/services/WebhookEventService.kt b/src/main/kotlin/net/leanix/githubagent/services/WebhookEventService.kt
@@ -85,7 +85,7 @@ class WebhookEventService(
         }
 
         removedManifestFiles.forEach { filePath ->
-            handleRemovedManifestFile(repositoryFullName)
+            handleRemovedManifestFile(repositoryFullName, filePath)
         }
     }
 
@@ -133,15 +133,20 @@ class WebhookEventService(
         )
     }
 
-    private fun handleRemovedManifestFile(repositoryFullName: String) {
-        logger.info("Manifest file ${ManifestFileAction.REMOVED} from repository $repositoryFullName")
+    private fun handleRemovedManifestFile(repositoryFullName: String, manifestFilePath: String) {
+        val location = if ('/' in manifestFilePath) {
+            "directory '/${manifestFilePath.substringBeforeLast('/')}'"
+        } else {
+            "root folder"
+        }
+        logger.info("Manifest file ${ManifestFileAction.REMOVED} from repository $repositoryFullName under $location")
         webSocketService.sendMessage(
             "/events/manifestFile",
             ManifestFileUpdateDto(
                 repositoryFullName,
                 ManifestFileAction.REMOVED,
                 null,
-                null
+                manifestFilePath
             )
         )
     }

diff --git a/src/test/kotlin/net/leanix/githubagent/services/WebhookEventServiceTest.kt b/src/test/kotlin/net/leanix/githubagent/services/WebhookEventServiceTest.kt
@@ -150,6 +150,39 @@ class WebhookEventServiceTest {
         verify { webSocketService.sendMessage(any(), any<ManifestFileUpdateDto>()) }
     }
 
+    @Test
+    fun `should handle manifest file removal in subdirectory`() {
+        val payload = """{
+            "repository": {
+                "name": "repo",
+                "full_name": "owner/repo",
+                "owner": {"name": "owner"},
+                "default_branch": "main"
+            },
+            "head_commit": {
+                "added": [],
+                "modified": [],
+                "removed": ["a/b/c/$MANIFEST_FILE_NAME", "a/b/c/some_other_file.yaml"]
+            },
+            "installation": {"id": 1},
+            "ref": "refs/heads/main"
+        }"""
+
+        webhookEventService.consumeWebhookEvent("PUSH", payload)
+
+        verify(exactly = 1) {
+            webSocketService.sendMessage(
+                "/events/manifestFile",
+                ManifestFileUpdateDto(
+                    "owner/repo",
+                    ManifestFileAction.REMOVED,
+                    null,
+                    "a/b/c/$MANIFEST_FILE_NAME"
+                )
+            )
+        }
+    }
+
     @Test
     fun `should handle manifest file in subdirectory`() {
         val payload = """{