Remove unused functions and variables (#2103)

Author: abrooksv

core/src/software/aws/toolkits/core/telemetry/MetricEvent.kt

@@ -108,8 +108,6 @@ class DefaultMetricEvent internal constructor(
         const val METADATA_NA = "n/a"
         const val METADATA_NOT_SET = "not-set"
         const val METADATA_INVALID = "invalid"
-
-        private val LOG = getLogger<DefaultDatum>()
     }
 
     class DefaultDatum(

jetbrains-core/src/software/aws/toolkits/jetbrains/services/clouddebug/execution/Steps.kt

@@ -73,7 +73,7 @@ abstract class Step {
  * [Step] that creates multiple child steps and runs them in parallel waiting on the result.
  */
 abstract class ParallelStep : Step() {
-    private inner class ChildStep(val future: CompletableFuture<*>, val step: Step)
+    private inner class ChildStep(val future: CompletableFuture<*>)
 
     private val listOfChildTasks = mutableListOf<ChildStep>()
 
@@ -88,7 +88,7 @@ abstract class ParallelStep : Step() {
     ) {
         buildChildSteps(context).forEach {
             val stepFuture = CompletableFuture<Unit>()
-            listOfChildTasks.add(ChildStep(stepFuture, it))
+            listOfChildTasks.add(ChildStep(stepFuture))
 
             ApplicationManager.getApplication().executeOnPooledThread {
                 try {

jetbrains-core/src/software/aws/toolkits/jetbrains/services/ecs/execution/EcsServiceCloudDebuggingOptions.kt

@@ -4,7 +4,6 @@
 package software.aws.toolkits.jetbrains.services.ecs.execution
 
 import com.intellij.util.xmlb.annotations.Tag
-import software.amazon.awssdk.services.ecs.model.LaunchType
 import software.aws.toolkits.core.credentials.ToolkitCredentialsProvider
 import software.aws.toolkits.core.region.AwsRegion
 import software.aws.toolkits.jetbrains.services.clouddebug.CloudDebuggingPlatform
@@ -64,7 +63,3 @@ data class ImmutablePortMapping(
     val localPort: Int,
     val remotePort: Int
 )
-
-enum class EcsLaunchType(val sdkType: LaunchType) {
-    EC2(LaunchType.EC2), FARGATE(LaunchType.FARGATE);
-}

jetbrains-core/src/software/aws/toolkits/jetbrains/services/lambda/execution/remote/RemoteLambdaRunSettingsEditor.kt

@@ -7,7 +7,6 @@ import com.intellij.openapi.options.SettingsEditor
 import com.intellij.openapi.project.Project
 import software.aws.toolkits.core.credentials.ToolkitCredentialsProvider
 import software.aws.toolkits.core.region.AwsRegion
-import software.aws.toolkits.core.utils.getLogger
 import software.aws.toolkits.core.utils.tryOrNull
 import software.aws.toolkits.jetbrains.core.credentials.CredentialManager
 import software.aws.toolkits.jetbrains.core.map
@@ -58,8 +57,4 @@ class RemoteLambdaRunSettingsEditor(project: Project) : SettingsEditor<RemoteLam
             configuration.useInputText(view.lambdaInput.inputText)
         }
     }
-
-    private companion object {
-        val LOG = getLogger<RemoteLambdaRunSettingsEditor>()
-    }
 }

jetbrains-core/src/software/aws/toolkits/jetbrains/services/s3/editor/S3TreeNode.kt

@@ -71,10 +71,6 @@ class S3TreeDirectoryNode(private val bucket: S3VirtualBucket, parent: S3TreeDir
         return (folders + s3Objects).sortedBy { it.key } + continuation
     }
 
-    fun removeChild(node: S3TreeNode) {
-        cachedList = cachedList.filter { it != node }
-    }
-
     fun removeAllChildren() {
         cachedList = listOf()
     }

jetbrains-core/src/software/aws/toolkits/jetbrains/services/schemas/search/SchemaSearchResults.kt

@@ -13,6 +13,4 @@ data class SchemaSearchResultWithRegistry(
     val registry: String
 )
 
-data class SchemaSearchResultVersion(val version: String)
-
 data class SchemaSearchError(val registryName: String, val errorMessage: String)

jetbrains-core/tst/software/aws/toolkits/jetbrains/core/credentials/MockAwsConnectionManager.kt

@@ -44,11 +44,6 @@ class MockAwsConnectionManager(project: Project) : AwsConnectionManager(project)
         waitUntilConnectionStateIsStable()
     }
 
-    fun nullifyRegionAndWait() {
-        changeConnectionSettings(selectedCredentialIdentifier, null)
-        waitUntilConnectionStateIsStable()
-    }
-
     fun setConnectionState(state: ConnectionState) {
         connectionState = state
     }

jetbrains-core/tst/software/aws/toolkits/jetbrains/services/cloudwatch/logs/DeleteGroupActionTest.kt

@@ -16,7 +16,6 @@ import software.amazon.awssdk.services.cloudwatchlogs.CloudWatchLogsClient
 import software.amazon.awssdk.services.cloudwatchlogs.model.DeleteLogGroupRequest
 import software.amazon.awssdk.services.cloudwatchlogs.model.DescribeLogStreamsRequest
 import software.amazon.awssdk.services.cloudwatchlogs.model.DescribeLogStreamsResponse
-import software.amazon.awssdk.services.cloudwatchlogs.model.LogGroup
 import software.amazon.awssdk.services.cloudwatchlogs.model.LogStream
 import software.aws.toolkits.core.utils.test.retryableAssert
 import software.aws.toolkits.jetbrains.core.MockClientManagerRule
@@ -34,7 +33,6 @@ class DeleteGroupActionTest {
     @Rule
     val mockClientManager = MockClientManagerRule(projectRule)
 
-    private val group = LogGroup.builder().logGroupName("name").build()
     private val stream = LogStream.builder().logStreamName("eman").build()
     private lateinit var cloudwatchmock: CloudWatchLogsClient
 

jetbrains-core/tst/software/aws/toolkits/jetbrains/services/lambda/sam/SamVersionCacheTest.kt

@@ -3,8 +3,6 @@
 
 package software.aws.toolkits.jetbrains.services.lambda.sam
 
-import org.jetbrains.concurrency.Promise
-import org.jetbrains.concurrency.all
 import org.junit.After
 import org.junit.Assert.assertEquals
 import org.junit.Rule
@@ -88,9 +86,4 @@ class SamVersionCacheTest {
         val samPath = SamCommonTestUtils.makeATestSam(message).toString()
         SamVersionCache.evaluateBlocking(samPath)
     }
-
-    private fun waitAll(promises: Collection<Promise<*>>) {
-        val all = promises.all(null, ignoreErrors = true)
-        all.blockingGet(3000)
-    }
 }

jetbrains-ultimate/it/software/aws/toolkits/jetbrains/services/clouddebug/nodejs/NodeJsDebugEndToEndTest.kt

@@ -7,7 +7,6 @@ import com.intellij.execution.executors.DefaultDebugExecutor
 import com.intellij.openapi.application.ApplicationInfo
 import com.intellij.openapi.util.registry.Registry
 import com.intellij.testFramework.runInEdtAndWait
-import com.intellij.xdebugger.XDebuggerUtil
 import org.assertj.core.api.Assertions.assertThat
 import org.junit.After
 import org.junit.Assume.assumeTrue
@@ -123,15 +122,5 @@ class NodeJsDebugEndToEndTest : CloudDebugTestCase("CloudDebugTestECSClusterTask
         return psiFile.virtualFile.path
     }
 
-    private fun addBreakpoint(lineNumber: Int) {
-        runInEdtAndWait {
-            XDebuggerUtil.getInstance().toggleLineBreakpoint(
-                projectRule.project,
-                projectRule.fixture.file.virtualFile,
-                lineNumber
-            )
-        }
-    }
-
     override fun getProject() = projectRule.project
 }