cleanup: remove unused CodeWhispererExplorerActionManager methods & funcs (aws#4423)

Author: Will-ShaoHua

plugins/toolkit/jetbrains-core/src/migration/software/aws/toolkits/jetbrains/services/codewhisperer/explorer/CodeWhispererExplorerActionManager.kt

@@ -23,19 +23,13 @@ import software.aws.toolkits.jetbrains.services.codewhisperer.explorer.CodeWhisp
 import software.aws.toolkits.jetbrains.services.codewhisperer.util.CodeWhispererConstants
 import software.aws.toolkits.jetbrains.services.codewhisperer.util.CodeWhispererUtil.getConnectionStartUrl
 import software.aws.toolkits.telemetry.AwsTelemetry
-import java.time.LocalDateTime
 
 // TODO: refactor this class, now it's managing action and state
 @State(name = "codewhispererStates", storages = [Storage("aws.xml")])
 class CodeWhispererExplorerActionManager : PersistentStateComponent<CodeWhispererExploreActionState> {
     private val actionState = CodeWhispererExploreActionState()
     private val suspendedConnections = mutableSetOf<String>()
 
-    fun isSuspended(project: Project): Boolean {
-        val startUrl = getCodeWhispererConnectionStartUrl(project)
-        return suspendedConnections.contains(startUrl)
-    }
-
     fun setSuspended(project: Project) {
         val startUrl = getCodeWhispererConnectionStartUrl(project)
         if (!suspendedConnections.add(startUrl)) {
@@ -70,42 +64,12 @@ class CodeWhispererExplorerActionManager : PersistentStateComponent<CodeWhispere
         actionState.value[CodeWhispererExploreStateType.HasShownNewOnboardingPage] = hasShownNewOnboardingPage
     }
 
-    fun setAccountlessNotificationWarnTimestamp() {
-        actionState.accountlessWarnTimestamp = LocalDateTime.now().format(CodeWhispererConstants.TIMESTAMP_FORMATTER)
-    }
-
-    fun setAccountlessNotificationErrorTimestamp() {
-        actionState.accountlessErrorTimestamp = LocalDateTime.now().format(CodeWhispererConstants.TIMESTAMP_FORMATTER)
-    }
-
-    fun getAccountlessWarnNotificationTimestamp(): String? = actionState.accountlessWarnTimestamp
-
-    fun getAccountlessErrorNotificationTimestamp(): String? = actionState.accountlessErrorTimestamp
-
-    fun getDoNotShowAgainWarn(): Boolean = actionState.value.getOrDefault(CodeWhispererExploreStateType.DoNotShowAgainWarn, false)
-
-    fun setDoNotShowAgainWarn(doNotShowAgain: Boolean) {
-        actionState.value[CodeWhispererExploreStateType.DoNotShowAgainWarn] = doNotShowAgain
-    }
-
-    fun getDoNotShowAgainError(): Boolean = actionState.value.getOrDefault(CodeWhispererExploreStateType.DoNotShowAgainError, false)
-
-    fun setDoNotShowAgainError(doNotShowAgain: Boolean) {
-        actionState.value[CodeWhispererExploreStateType.DoNotShowAgainError] = doNotShowAgain
-    }
-
     fun getConnectionExpiredDoNotShowAgain(): Boolean = actionState.value.getOrDefault(CodeWhispererExploreStateType.ConnectionExpiredDoNotShowAgain, false)
 
     fun setConnectionExpiredDoNotShowAgain(doNotShowAgain: Boolean) {
         actionState.value[CodeWhispererExploreStateType.ConnectionExpiredDoNotShowAgain] = doNotShowAgain
     }
 
-    fun getAccountlessNullified(): Boolean = actionState.value.getOrDefault(CodeWhispererExploreStateType.AccountlessNullified, false)
-
-    fun setAccountlessNullified(accountlessNullified: Boolean) {
-        actionState.value[CodeWhispererExploreStateType.AccountlessNullified] = accountlessNullified
-    }
-
     fun setAutoSuggestion(project: Project, isAutoEnabled: Boolean) {
         setAutoEnabled(isAutoEnabled)
         val autoSuggestionState = if (isAutoEnabled) CodeWhispererConstants.AutoSuggestion.ACTIVATED else CodeWhispererConstants.AutoSuggestion.DEACTIVATED
@@ -160,13 +124,6 @@ class CodeWhispererExplorerActionManager : PersistentStateComponent<CodeWhispere
         } ?: CodeWhispererLoginType.Logout
     }
 
-    fun nullifyAccountlessCredentialIfNeeded() {
-        if (actionState.token != null) {
-            setAccountlessNullified(true)
-            actionState.token = null
-        }
-    }
-
     override fun getState(): CodeWhispererExploreActionState = CodeWhispererExploreActionState().apply {
         value.putAll(actionState.value)
         token = actionState.token

plugins/toolkit/jetbrains-core/src/software/aws/toolkits/jetbrains/services/codewhisperer/explorer/CodeWhispererExplorerActionManager.kt

@@ -53,13 +53,3 @@ fun isCodeWhispererEnabled(project: Project) = with(CodeWhispererExplorerActionM
 fun isUserBuilderId(project: Project) = with(CodeWhispererExplorerActionManager.getInstance()) {
     checkActiveCodeWhispererConnectionType(project) == CodeWhispererLoginType.Sono
 }
-
-/**
- * Note: please use this util with extra caution, it will return "false" for a "logout" scenario,
- *  the reasoning is we need handling specifically for a "Expired" condition thus excluding logout from here
- *  If callers rather need a predicate "isInvalidConnection", please use the combination of the two (!isCodeWhispererEnabled() || isCodeWhispererExpired())
- */
-@Deprecated("remove it, use isQExpired")
-fun isCodeWhispererExpired(project: Project) = with(CodeWhispererExplorerActionManager.getInstance()) {
-    checkActiveCodeWhispererConnectionType(project) == CodeWhispererLoginType.Expired
-}

plugins/toolkit/jetbrains-core/tst/software/aws/toolkits/jetbrains/services/codewhisperer/CodeWhispererExplorerActionManagerTest.kt

@@ -34,7 +34,6 @@ import software.aws.toolkits.jetbrains.core.credentials.sso.bearer.InteractiveBe
 import software.aws.toolkits.jetbrains.services.codewhisperer.credentials.CodeWhispererLoginType
 import software.aws.toolkits.jetbrains.services.codewhisperer.explorer.CodeWhispererExplorerActionManager
 import software.aws.toolkits.jetbrains.services.codewhisperer.explorer.isCodeWhispererEnabled
-import software.aws.toolkits.jetbrains.services.codewhisperer.explorer.isCodeWhispererExpired
 import java.nio.file.Files
 import java.nio.file.Path
 import java.nio.file.Paths
@@ -97,7 +96,6 @@ class CodeWhispererExplorerActionManagerTest {
         val actual = mockManager.checkActiveCodeWhispererConnectionType(project)
         assertThat(actual).isEqualTo(CodeWhispererLoginType.Logout)
         assertThat(isCodeWhispererEnabled(project)).isFalse
-        assertThat(isCodeWhispererExpired(project)).isFalse
     }
 
     /**
@@ -174,6 +172,7 @@ class CodeWhispererExplorerActionManagerTest {
         assertThat(ConnectionPinningManager.getInstance().isFeaturePinned(CodeWhispererConnection.getInstance())).isFalse
     }
 
+    @SuppressWarnings("UnusedParameter")
     private fun assertConnectionState(
         startUrl: String,
         refreshToken: String?,
@@ -213,6 +212,5 @@ class CodeWhispererExplorerActionManagerTest {
         assertThat(myTokenProvider.state()).isEqualTo(expectedState)
         assertThat(CodeWhispererExplorerActionManager.getInstance().checkActiveCodeWhispererConnectionType(project)).isEqualTo(expectedLoginType)
         assertThat(isCodeWhispererEnabled(project)).isEqualTo(expectedIsCwEnabled)
-        assertThat(isCodeWhispererExpired(project)).isEqualTo(expectedIsCwExpired)
     }
 }