[Gradle, JS] dependsOn for test tasks

libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinPlugin.kt

@@ -157,7 +157,11 @@ internal abstract class KotlinSourceSetProcessor<T : AbstractKotlinCompile<*>>(
     protected fun applyStandardTaskConfiguration(taskConfiguration: AbstractKotlinCompileConfig<*>) {
         taskConfiguration.configureTask {
             it.description = taskDescription
-            it.destinationDirectory.convention(defaultKotlinDestinationDir)
+            if (it is Kotlin2JsCompile) {
+                it.defaultDestinationDirectory.convention(defaultKotlinDestinationDir)
+            } else {
+                it.destinationDirectory.convention(defaultKotlinDestinationDir)
+            }
             it.libraries.from({ kotlinCompilation.compileDependencyFiles })
         }
     }

libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/targets/js/ir/KotlinJsIrSubTarget.kt

@@ -121,17 +121,15 @@ abstract class KotlinJsIrSubTarget(
                 KotlinJsBinaryMode.DEVELOPMENT
             ).single()
 
-            testJs.inputFileProperty.set(
+            testJs.dependsOn(binary.linkSyncTask)
-                project.layout.file(
+            testJs.inputFileProperty.fileProvider(
-                    binary.linkSyncTask.flatMap { copyTask ->
+                binary.linkSyncTask.flatMap { linkSyncTask ->
-                        binary.linkTask
+                    binary.linkTask.flatMap { linkTask ->
-                            .flatMap { linkTask -> linkTask.compilerOptions.moduleName }
+                        linkTask.outputFileProperty.map {
-                            .map { outputName ->
+                            linkSyncTask.destinationDir.resolve(it.name)
-                                copyTask.destinationDir
+                        }
-                                    .resolve(outputName + compilation.platformType.fileExtension)
-                            }
                     }
-                )
+                }
             )
 
             configureTestDependencies(testJs)

libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/tasks/Tasks.kt

@@ -977,6 +977,10 @@ abstract class Kotlin2JsCompile @Inject constructor(
         }
     }
 
+    // Workaround to be able to use default value and change it later based on external input
+    @get:Internal
+    internal abstract val defaultDestinationDirectory: DirectoryProperty
+
     @Deprecated("Use destinationDirectory and moduleName instead")
     @get:Internal
     abstract val outputFileProperty: Property<File>

libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/tasks/configuration/Kotlin2JsCompileConfig.kt

@@ -50,6 +50,24 @@ internal open class BaseKotlin2JsCompileConfig<TASK : Kotlin2JsCompile>(
                 }
             )
 
+            task.destinationDirectory
+                .convention(
+                    project.objects.directoryProperty().fileProvider(
+                        task.defaultDestinationDirectory.map {
+                            val freeArgs = task.enhancedFreeCompilerArgs.get()
+                            if (task.compilerOptions.outputFile.orNull != null) {
+                                if (freeArgs.contains(PRODUCE_UNZIPPED_KLIB)) {
+                                    File(task.compilerOptions.outputFile.get())
+                                } else {
+                                    File(task.compilerOptions.outputFile.get()).parentFile
+                                }
+                            } else {
+                                it.asFile
+                            }
+                        }
+                    )
+                )
+
             task.libraryCache.set(libraryCacheService).also { task.libraryCache.disallowChanges() }
         }
     }