Fixed creation of Kover Agent arguments file

Previously parent directory hasn't been creating for an arguments file Fixes #721
Kotlin · Jan 6, 2025 · d9b5f4d · d9b5f4d
1 parent d7e35a7
commit d9b5f4d
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 3 deletions.
diff --git a/...tionalTest/kotlin/kotlinx/kover/gradle/plugin/test/functional/cases/DefaultConfigTests.kt b/...tionalTest/kotlin/kotlinx/kover/gradle/plugin/test/functional/cases/DefaultConfigTests.kt
@@ -19,4 +19,15 @@ internal class DefaultConfigTests {
             checkDefaultReports()
         }
     }
+
+    @GeneratedTest()
+    fun BuildConfigurator.testCleanCheck() {
+        addProjectWithKover {
+            sourcesFrom("simple")
+        }
+
+        run("clean", "check") {
+            checkDefaultBinReport()
+        }
+    }
 }
diff --git a/...ugin/src/main/kotlin/kotlinx/kover/gradle/aggregation/project/KoverProjectGradlePlugin.kt b/...ugin/src/main/kotlin/kotlinx/kover/gradle/aggregation/project/KoverProjectGradlePlugin.kt
@@ -26,6 +26,7 @@ import org.gradle.api.tasks.compile.JavaCompile
 import org.gradle.api.tasks.testing.Test
 import org.gradle.kotlin.dsl.create
 import org.gradle.kotlin.dsl.named
+import org.gradle.kotlin.dsl.register
 import org.gradle.kotlin.dsl.withType
 import java.io.File
 
@@ -63,7 +64,7 @@ internal class KoverProjectGradlePlugin : Plugin<Project> {
         val artifactFile = layout.buildDirectory.file("kover/kover.artifact")
 
         // we create task immediately because of mustRunAfter
-        val generateArtifactTask = tasks.create<ArtifactGenerationTask>("koverGenerateArtifact")
+        val generateArtifactTask = tasks.register<ArtifactGenerationTask>("koverGenerateArtifact").get()
         generateArtifactTask.outputFile.set(artifactFile)
 
         // add tests

diff --git a/...in/kotlinx/kover/gradle/aggregation/project/instrumentation/JvmTestTaskInstrumentation.kt b/...in/kotlinx/kover/gradle/aggregation/project/instrumentation/JvmTestTaskInstrumentation.kt
@@ -126,7 +126,7 @@ private fun buildKoverJvmAgentArgs(
 }
 
 private fun File.writeAgentArgs(binReportFile: File, includedClasses: Set<String>, excludedClasses: Set<String>) {
-    binReportFile.parentFile.mkdirs()
+    parentFile.mkdirs()
     val binReportPath = binReportFile.canonicalPath
 
     printWriter().use { pw ->

diff --git a/...gin/src/main/kotlin/kotlinx/kover/gradle/plugin/tools/kover/KoverOnlineInstrumentation.kt b/...gin/src/main/kotlin/kotlinx/kover/gradle/plugin/tools/kover/KoverOnlineInstrumentation.kt
@@ -20,7 +20,7 @@ internal fun buildJvmAgentArgs(
 }
 
 private fun File.writeAgentArgs(binReportFile: File, excludedClasses: Set<String>, includedClasses: Set<String>) {
-    binReportFile.parentFile.mkdirs()
+    parentFile.mkdirs()
     val binReportPath = binReportFile.canonicalPath
 
     printWriter().use { pw ->