detekt

Author: samgst-amazon

plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/workspace/WorkspaceServiceHandler.kt

@@ -98,7 +98,7 @@ class WorkspaceServiceHandler(
             val validFiles = events.mapNotNull { event ->
                 when (event) {
                     is VFileDeleteEvent -> {
-                        val file = event.file?.takeIf { shouldHandleFile(it) } ?: return@mapNotNull null
+                        val file = event.file.takeIf { shouldHandleFile(it) } ?: return@mapNotNull null
                         toUriString(file)
                     }
                     is VFileMoveEvent -> {
@@ -159,10 +159,12 @@ class WorkspaceServiceHandler(
                     is VFileCopyEvent -> {
                         event.newParent.findChild(event.newChildName)?.let { newFile ->
                             toUriString(newFile)?.let { uri ->
-                                listOf(FileEvent().apply {
-                                    this.uri = uri
-                                    type = FileChangeType.Created
-                                })
+                                listOf(
+                                    FileEvent().apply {
+                                        this.uri = uri
+                                        type = FileChangeType.Created
+                                    }
+                                )
                             }
                         } ?: emptyList()
                     }
@@ -185,14 +187,16 @@ class WorkspaceServiceHandler(
                     else -> {
                         event.file?.let { file ->
                             toUriString(file)?.let { uri ->
-                                listOf(FileEvent().apply {
-                                    this.uri = uri
-                                    type = when (event) {
-                                        is VFileCreateEvent -> FileChangeType.Created
-                                        is VFileDeleteEvent -> FileChangeType.Deleted
-                                        else -> FileChangeType.Changed
+                                listOf(
+                                    FileEvent().apply {
+                                        this.uri = uri
+                                        type = when (event) {
+                                            is VFileCreateEvent -> FileChangeType.Created
+                                            is VFileDeleteEvent -> FileChangeType.Deleted
+                                            else -> FileChangeType.Changed
+                                        }
                                     }
-                                })
+                                )
                             }
                         } ?: emptyList()
                     }

plugins/amazonq/shared/jetbrains-community/tst/software/aws/toolkits/jetbrains/services/amazonq/lsp/workspace/WorkspaceServiceHandlerTest.kt

@@ -265,7 +265,6 @@ class WorkspaceServiceHandlerTest {
         assertEquals(normalizeFileUri(dirUri.toString()), paramsSlot.captured.files[0].uri)
     }
 
-
     @Test
     fun `test didDeleteFiles handles both delete and move events in same batch`() = runTest {
         val deleteUri = URI("file:///test/deleteFile")
@@ -628,7 +627,12 @@ class WorkspaceServiceHandlerTest {
         }
     }
 
-    private fun createMockVFileEvent(uri: URI, type: FileChangeType = FileChangeType.Changed, isDirectory: Boolean = false, extension: String = "py"): VFileEvent {
+    private fun createMockVFileEvent(
+        uri: URI,
+        type: FileChangeType = FileChangeType.Changed,
+        isDirectory: Boolean = false,
+        extension: String = "py",
+    ): VFileEvent {
         val virtualFile = createMockVirtualFile(uri, "test.$extension", isDirectory)
         return when (type) {
             FileChangeType.Deleted -> mockk<VFileDeleteEvent>()
@@ -681,8 +685,6 @@ class WorkspaceServiceHandlerTest {
         }
     }
 
-
-
     // for windows unit tests
     private fun normalizeFileUri(uri: String): String {
         if (!System.getProperty("os.name").lowercase().contains("windows")) {