Fix detekt issues in TelemetryHelperTest and CodeInsightsSettingsFacadeTest

Author: aseemxs

plugins/amazonq/chat/jetbrains-community/tst/software/aws/toolkits/jetbrains/services/amazonq/TelemetryHelperTest.kt

@@ -15,7 +15,6 @@ import org.assertj.core.api.Assertions.assertThat
 import org.junit.jupiter.api.AfterEach
 import org.junit.jupiter.api.BeforeEach
 import org.junit.jupiter.api.Test
-import org.junit.jupiter.api.extension.RegisterExtension
 import org.mockito.kotlin.any
 import org.mockito.kotlin.argumentCaptor
 import org.mockito.kotlin.doReturn
@@ -72,7 +71,7 @@ import kotlin.test.assertNotNull
 
 class TelemetryHelperTest {
     private lateinit var myFixture: CodeInsightTestFixture
-    
+
     // sut
     private lateinit var sut: TelemetryHelper
 
@@ -161,14 +160,14 @@ class TelemetryHelperTest {
         val fixtureBuilder = factory.createLightFixtureBuilder("TelemetryHelperTest")
         myFixture = factory.createCodeInsightFixture(fixtureBuilder.fixture, LightTempDirTestFixtureImpl(true))
         myFixture.setUp()
-        
+
         // NOW manually initialize mocks - Application exists now
         mockClientManager.beforeEach(null)
         mockTelemetryService.beforeEach(null)
-        
+
         // Enable telemetry for tests
         software.aws.toolkits.jetbrains.settings.AwsSettings.getInstance().isTelemetryEnabled = true
-        
+
         // set up sut
         appInitContext = AmazonQAppInitContext(
             project = myFixture.project,
@@ -219,7 +218,7 @@ class TelemetryHelperTest {
         // Clean up mocks first
         mockTelemetryService.afterEach(null)
         mockClientManager.afterEach(null)
-        
+
         // Then tear down fixture
         myFixture.tearDown()
     }

plugins/amazonq/codewhisperer/jetbrains-community/tst/software/aws/toolkits/jetbrains/services/codewhisperer/util/CodeInsightsSettingsFacadeTest.kt

@@ -31,9 +31,7 @@ class CodeInsightsSettingsFacadeTest : HeavyPlatformTestCase() {
     }
 
     fun testDisableCodeInsightUntilShouldRevertWhenParentIsDisposed() {
-        val myFakePopup = object : Disposable {
-            override fun dispose() {}
-        }
+        val myFakePopup = Disposable {}
         Disposer.register(testRootDisposable, myFakePopup)
 
         // assume users' enable the following two codeinsight functionalities
@@ -78,14 +76,10 @@ class CodeInsightsSettingsFacadeTest : HeavyPlatformTestCase() {
     }
 
     fun testDisableCodeInsightUntilShouldAlwaysFlushPendingRevertsBeforeMakingNextChanges() {
-        val myFakePopup = object : Disposable {
-            override fun dispose() {}
-        }
+        val myFakePopup = Disposable {}
         Disposer.register(testRootDisposable, myFakePopup)
 
-        val myAnotherFakePopup = object : Disposable {
-            override fun dispose() {}
-        }
+        val myAnotherFakePopup = Disposable {}
         Disposer.register(testRootDisposable, myAnotherFakePopup)
 
         // assume users' enable the following two codeinsight functionalities