Added unit test cases

Author: LokeshDogga13

plugins/amazonq/chat/jetbrains-community/tst/software/aws/toolkits/jetbrains/services/amazonq/lsp/artifacts/ManifestFetcherTest.kt

@@ -5,6 +5,7 @@ package software.aws.toolkits.jetbrains.services.amazonq.lsp.artifacts
 
 import com.intellij.openapi.util.text.StringUtil
 import com.intellij.util.io.DigestUtil
+import org.assertj.core.util.VisibleForTesting
 import software.aws.toolkits.core.utils.error
 import software.aws.toolkits.core.utils.exists
 import software.aws.toolkits.core.utils.getLogger
@@ -38,7 +39,8 @@ class ManifestFetcher {
         return fetchManifestFromRemote()
     }
 
-    private fun fetchManifestFromRemote(): ManifestManager.Manifest? {
+    @VisibleForTesting
+    internal fun fetchManifestFromRemote(): ManifestManager.Manifest? {
         val manifest: ManifestManager.Manifest?
         try {
             val manifestString = getTextFromUrl(lspManifestUrl)
@@ -64,7 +66,8 @@ class ManifestFetcher {
         }
     }
 
-    private fun fetchManifestFromLocal(): ManifestManager.Manifest? {
+    @VisibleForTesting
+    internal fun fetchManifestFromLocal(): ManifestManager.Manifest? {
         val localETag = getManifestETagFromLocal()
         val remoteETag = getManifestETagFromUrl()
         // If local and remote have same ETag, we can re-use the manifest file from local to fetch artifacts.

plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/artifacts/ManifestFetcher.kt

@@ -0,0 +1,100 @@
+// Copyright 2025 Amazon.com, Inc. or its affiliates. All Rights Reserved.
+// SPDX-License-Identifier: Apache-2.0
+
+package software.aws.toolkits.jetbrains.services.amazonq.lsp.artifacts
+
+import io.mockk.every
+import io.mockk.mockkStatic
+import org.assertj.core.api.Assertions.assertThat
+import org.jetbrains.annotations.TestOnly
+import org.junit.jupiter.api.BeforeEach
+import org.junit.jupiter.api.Test
+import org.mockito.kotlin.atLeastOnce
+import org.mockito.kotlin.never
+import org.mockito.kotlin.reset
+import org.mockito.kotlin.spy
+import org.mockito.kotlin.verify
+import org.mockito.kotlin.whenever
+import software.aws.toolkits.jetbrains.core.getTextFromUrl
+import software.aws.toolkits.jetbrains.services.amazonq.project.manifest.ManifestManager
+
+@TestOnly
+class ManifestFetcherTest {
+
+    private lateinit var manifestFetcher: ManifestFetcher
+    private lateinit var manifest: ManifestManager.Manifest
+    private lateinit var manifestManager: ManifestManager
+
+    @BeforeEach
+    fun setup() {
+        manifestFetcher = spy(ManifestFetcher())
+        manifestManager = spy(ManifestManager())
+        manifest = ManifestManager.Manifest()
+    }
+
+    @Test
+    fun `should return null when both local and remote manifests are null`() {
+        whenever(manifestFetcher.fetchManifestFromLocal()).thenReturn(null)
+        whenever(manifestFetcher.fetchManifestFromRemote()).thenReturn(null)
+
+        assertThat(manifestFetcher.fetch()).isNull()
+    }
+
+    @Test
+    fun `should return valid result from local should not execute remote method`() {
+        reset(manifestFetcher)
+        whenever(manifestFetcher.fetchManifestFromLocal()).thenReturn(manifest)
+
+        assertThat(manifestFetcher.fetch()).isNotNull().isEqualTo(manifest)
+        verify(manifestFetcher, atLeastOnce()).fetchManifestFromLocal()
+        verify(manifestFetcher, never()).fetchManifestFromRemote()
+    }
+
+    @Test
+    fun `should return valid result from remote`() {
+        whenever(manifestFetcher.fetchManifestFromLocal()).thenReturn(null)
+        whenever(manifestFetcher.fetchManifestFromRemote()).thenReturn(manifest)
+
+        assertThat(manifestFetcher.fetch()).isNotNull().isEqualTo(manifest)
+        verify(manifestFetcher, atLeastOnce()).fetchManifestFromLocal()
+        verify(manifestFetcher, atLeastOnce()).fetchManifestFromRemote()
+    }
+
+    @Test
+    fun `fetchManifestFromRemote should return null due to invalid manifestString`() {
+        mockkStatic("software.aws.toolkits.jetbrains.core.HttpUtilsKt")
+        every { getTextFromUrl(any()) } returns "ManifestContent"
+
+        whenever(manifestManager.readManifestFile("")).thenReturn(null)
+
+        assertThat(manifestFetcher.fetchManifestFromRemote()).isNull()
+    }
+
+    @Test
+    fun `fetchManifestFromRemote should return manifest and update manifest`() {
+        val validManifest = ManifestManager.Manifest(manifestSchemaVersion = "1.0")
+        mockkStatic("software.aws.toolkits.jetbrains.core.HttpUtilsKt")
+
+        every { getTextFromUrl(any()) } returns "{ \"manifestSchemaVersion\": \"1.0\" }"
+
+        val result = manifestFetcher.fetchManifestFromRemote()
+        assertThat(result).isNotNull().isEqualTo(validManifest)
+    }
+
+    @Test
+    fun `fetchManifestFromRemote should return null if manifest is deprecated`() {
+        mockkStatic("software.aws.toolkits.jetbrains.core.HttpUtilsKt")
+        every { getTextFromUrl(any()) } returns "ManifestContent"
+
+        val deprecatedManifest = ManifestManager.Manifest(isManifestDeprecated = true)
+
+        whenever(manifestManager.readManifestFile("")).thenReturn(deprecatedManifest)
+
+        assertThat(manifestFetcher.fetchManifestFromRemote()).isNull()
+    }
+
+    @Test
+    fun `fetchManifestFromLocal should return null`() {
+        assertThat(manifestFetcher.fetchManifestFromLocal()).isNull()
+    }
+}