[Debugger, JS] Provide possibility to debug mocha tests

libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/testing/KotlinJsTest.kt

@@ -114,10 +114,11 @@ open class KotlinJsTest : KotlinTest(), RequiresNpmDependencies {
 
         val nodeJsArgs = mutableListOf<String>()
 
-        if (debug) {
+        return testFramework!!.createTestExecutionSpec(
-            nodeJsArgs.add("--inspect-brk")
+            task = this,
-        }
+            forkOptions = forkOptions,
-
+            nodeJsArgs = nodeJsArgs,
-        return testFramework!!.createTestExecutionSpec(this, forkOptions, nodeJsArgs)
+            debug = debug
+        )
     }
 }

libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/testing/KotlinJsTestFramework.kt

@@ -15,7 +15,8 @@ interface KotlinJsTestFramework : RequiresNpmDependencies {
     fun createTestExecutionSpec(
         task: KotlinJsTest,
         forkOptions: ProcessForkOptions,
-        nodeJsArgs: MutableList<String>
+        nodeJsArgs: MutableList<String>,
+        debug: Boolean
     ): TCServiceMessagesTestExecutionSpec
 
     override val nodeModulesRequired: Boolean

libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/testing/karma/KotlinKarma.kt

@@ -254,7 +254,8 @@ class KotlinKarma(override val compilation: KotlinJsCompilation) : KotlinJsTestF
     override fun createTestExecutionSpec(
         task: KotlinJsTest,
         forkOptions: ProcessForkOptions,
-        nodeJsArgs: MutableList<String>
+        nodeJsArgs: MutableList<String>,
+        debug: Boolean
     ): TCServiceMessagesTestExecutionSpec {
         if (config.browsers.isEmpty()) {
             error("No browsers configured for $task")

libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/testing/mocha/KotlinMocha.kt

@@ -40,7 +40,8 @@ class KotlinMocha(override val compilation: KotlinJsCompilation) : KotlinJsTestF
     override fun createTestExecutionSpec(
         task: KotlinJsTest,
         forkOptions: ProcessForkOptions,
-        nodeJsArgs: MutableList<String>
+        nodeJsArgs: MutableList<String>,
+        debug: Boolean
     ): TCServiceMessagesTestExecutionSpec {
         val clientSettings = TCServiceMessagesClientSettings(
             task.name,
@@ -59,20 +60,21 @@ class KotlinMocha(override val compilation: KotlinJsCompilation) : KotlinJsTestF
 
         createAdapterJs(task)
 
-        val nodeModules = listOf(
+        val mocha = npmProject.require("mocha/bin/mocha")
-            "mocha/bin/mocha",
+        val adapter = npmProject.require("./$ADAPTER_NODEJS")
-            "./$ADAPTER_NODEJS"
-        )
 
-        val args = nodeJsArgs +
+        val args = mutableListOf(mocha).apply {
-                nodeModules.map {
+            addAll(cliArg("--inspect-brk", debug))
-                    npmProject.require(it)
+            add(adapter)
-                } + cliArgs.toList() +
+            addAll(cliArgs.toList())
-                cliArg("--reporter", "kotlin-test-js-runner/mocha-kotlin-reporter.js") +
+            addAll(cliArg("--reporter", "kotlin-test-js-runner/mocha-kotlin-reporter.js"))
-                cliArg("--timeout", timeout) +
+            addAll(cliArg("--timeout", timeout))
+            addAll(
                 cliArg(
                     "-r", "kotlin-test-js-runner/kotlin-nodejs-source-map-support.js"
                 )
+            )
+        }
 
         return TCServiceMessagesTestExecutionSpec(
             forkOptions,
@@ -82,6 +84,10 @@ class KotlinMocha(override val compilation: KotlinJsCompilation) : KotlinJsTestF
         )
     }
 
+    private fun cliArg(cli: String, value: Boolean): List<String> {
+        return if (value) listOf(cli) else emptyList()
+    }
+
     private fun cliArg(cli: String, value: String?): List<String> {
         return value?.let { listOf(cli, it) } ?: emptyList()
     }