From 1cd5b16ecc9f0d8d4419819b274e95925c6cda7d Mon Sep 17 00:00:00 2001
From: Kostas Petrakis <kostas.petrakis@leanix.net>
Date: Mon, 4 Nov 2024 15:10:32 +0100
Subject: [PATCH] feat: react to manifest file creation via webhook

- Added location for logs
- Added manifest file path in payload
---
 .../services/WebhookEventService.kt           | 13 +++++---
 .../services/WebhookEventServiceTest.kt       | 33 +++++++++++++++++++
 2 files changed, 42 insertions(+), 4 deletions(-)

diff --git a/src/main/kotlin/net/leanix/githubagent/services/WebhookEventService.kt b/src/main/kotlin/net/leanix/githubagent/services/WebhookEventService.kt
index 95bac9c..a608709 100644
--- 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
index 7de900c..153fa1a 100644
--- 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 = """{