CID-3590: Accept manifest files with case ignored

Author: mohamedlajmileanix

src/main/kotlin/net/leanix/githubagent/services/GitHubScanningService.kt

@@ -156,7 +156,7 @@ class GitHubScanningService(
                     "repo:${installation.account.login}/$repositoryName filename:$MANIFEST_FILE_NAME"
             )
         }
-        manifestFiles.items.filter { it.name == MANIFEST_FILE_NAME }
+        manifestFiles.items.filter { it.name.lowercase() == MANIFEST_FILE_NAME }
     }
     private fun fetchManifestContents(
         installation: Installation,

src/main/kotlin/net/leanix/githubagent/services/WebhookEventService.kt

@@ -110,9 +110,9 @@ class WebhookEventService(
         repositoryName: String,
         installationToken: String
     ) {
-        val addedManifestFiles = headCommit.added.filter { isLeanixManifestFile(it) }
-        val modifiedManifestFiles = headCommit.modified.filter { isLeanixManifestFile(it) }
-        val removedManifestFiles = headCommit.removed.filter { isLeanixManifestFile(it) }
+        val addedManifestFiles = headCommit.added.filter { isLeanixManifestFile(it.lowercase()) }
+        val modifiedManifestFiles = headCommit.modified.filter { isLeanixManifestFile(it.lowercase()) }
+        val removedManifestFiles = headCommit.removed.filter { isLeanixManifestFile(it.lowercase()) }
 
         addedManifestFiles.forEach { filePath ->
             handleAddedOrModifiedManifestFile(

src/main/kotlin/net/leanix/githubagent/shared/Constants.kt

@@ -12,7 +12,7 @@ val SUPPORTED_EVENT_TYPES = listOf(
     "INSTALLATION",
 )
 
-val fileNameMatchRegex = Regex("/?$MANIFEST_FILE_NAME\$")
+val fileNameMatchRegex = Regex("/?$MANIFEST_FILE_NAME\$", RegexOption.IGNORE_CASE)
 
 const val GITHUB_APP_LABEL = "GitHub App"
 const val INSTALLATION_LABEL = "Installation"

src/test/kotlin/net/leanix/githubagent/services/GitHubScanningServiceTest.kt

@@ -357,6 +357,78 @@ class GitHubScanningServiceTest {
         verify(exactly = 1) { syncLogService.sendInfoLog("Found 2 manifest files in repository TestRepo.") }
     }
 
+    @Test
+    fun `scanGitHubResources should accept manifest files with case ignored`() {
+        // given
+        every { cachingService.get("runId") } returns runId
+        every { gitHubGraphQLService.getRepositories(any(), any()) } returns PagedRepositories(
+            repositories = listOf(
+                RepositoryDto(
+                    id = "repo1",
+                    name = "TestRepo",
+                    organizationName = "testOrg",
+                    description = "A test repository",
+                    url = "https://github.com/testRepo",
+                    defaultBranch = "main",
+                    archived = false,
+                    visibility = RepositoryVisibility.PUBLIC,
+                    updatedAt = "2024-01-01T00:00:00Z",
+                    languages = listOf("Kotlin", "Java"),
+                    topics = listOf("test", "example"),
+                )
+            ),
+            hasNextPage = false,
+            cursor = null
+        )
+        every { gitHubClient.searchManifestFiles(any(), any()) } returns GitHubSearchResponse(
+            1,
+            listOf(
+                ItemResponse(
+                    name = "leanIX.yaml",
+                    path = "leanIX.yaml",
+                    repository = RepositoryItemResponse(
+                        name = "TestRepo",
+                        fullName = "testOrg/TestRepo"
+                    ),
+                    url = "http://url"
+                ),
+                ItemResponse(
+                    name = "lEAnIX.yaml",
+                    path = "a/lEAnIX.yaml",
+                    repository = RepositoryItemResponse(
+                        name = "TestRepo",
+                        fullName = "testOrg/TestRepo"
+                    ),
+                    url = "http://url"
+                ),
+                ItemResponse(
+                    name = MANIFEST_FILE_NAME,
+                    path = "b/$MANIFEST_FILE_NAME",
+                    repository = RepositoryItemResponse(
+                        name = "TestRepo",
+                        fullName = "testOrg/TestRepo"
+                    ),
+                    url = "http://url"
+                )
+            )
+        )
+        every {
+            gitHubGraphQLService.getManifestFileContent(any(), any(), "b/leanix.yaml", any())
+        } returns "content"
+        every {
+            gitHubGraphQLService.getManifestFileContent(any(), any(), "leanIX.yaml", any())
+        } returns "content"
+        every {
+            gitHubGraphQLService.getManifestFileContent(any(), any(), "a/lEAnIX.yaml", any())
+        } returns "content"
+
+        // when
+        gitHubScanningService.scanGitHubResources()
+
+        // then
+        verify(exactly = 1) { syncLogService.sendInfoLog("Found 3 manifest files in repository TestRepo.") }
+    }
+
     @Test
     fun `scanGitHubResources should skip organizations without correct permissions and events`() {
         every { cachingService.get("runId") } returns runId

src/test/kotlin/net/leanix/githubagent/services/WebhookEventServiceTest.kt

@@ -163,6 +163,63 @@ class WebhookEventServiceTest {
         }
     }
 
+    @Test
+    fun `should accept manifest files with case ignore`() {
+        val payload = """{
+            "repository": {
+                "name": "repo",
+                "full_name": "owner/repo",
+                "owner": {"name": "owner"},
+                "default_branch": "main"
+            },
+            "head_commit": {
+                "added": [],
+                "modified": ["a/$MANIFEST_FILE_NAME", "b/leanIX.yaml", "LEanIX.yaml"],
+                "removed": []
+            },
+            "installation": {"id": 1},
+            "ref": "refs/heads/main"
+        }"""
+
+        webhookEventService.consumeWebhookEvent("PUSH", payload)
+
+        verify(exactly = 1) {
+            webSocketService.sendMessage(
+                "/events/manifestFile",
+                ManifestFileUpdateDto(
+                    "owner/repo",
+                    ManifestFileAction.MODIFIED,
+                    "content",
+                    "tree/main/b"
+                )
+            )
+        }
+
+        verify(exactly = 1) {
+            webSocketService.sendMessage(
+                "/events/manifestFile",
+                ManifestFileUpdateDto(
+                    "owner/repo",
+                    ManifestFileAction.MODIFIED,
+                    "content",
+                    "tree/main/a"
+                )
+            )
+        }
+
+        verify(exactly = 1) {
+            webSocketService.sendMessage(
+                "/events/manifestFile",
+                ManifestFileUpdateDto(
+                    "owner/repo",
+                    ManifestFileAction.MODIFIED,
+                    "content",
+                    ""
+                )
+            )
+        }
+    }
+
     @Test
     fun `should send all events of type other than push to backend without processing`() {
         val payload = """{