Implemented the use of the full path to the project instead of the name

Fixes #151

Author: shanshin

README.md

@@ -332,7 +332,7 @@ kover {
     intellijEngineVersion.set("1.0.656")    // change version of IntelliJ agent and reporter
     jacocoEngineVersion.set("0.8.7")        // change version of JaCoCo agent and reporter
     generateReportOnCheck = true            // false to do not execute `koverMergedReport` task before `check` task
-    disabledProjects = setOf()              // setOf("project-name") to disable coverage for project with name `project-name`
+    disabledProjects = setOf()              // setOf(":project-name") to disable coverage for project with path `:project-name` (`:` for the root project)
     instrumentAndroidPackage = false        // true to instrument packages `android.*` and `com.android.*`
     runAllTestsForProjectTask = false       // true to run all tests in all projects if `koverHtmlReport`, `koverXmlReport`, `koverReport`, `koverVerify` or `check` tasks executed on some project
 }
@@ -349,7 +349,7 @@ kover {
     intellijEngineVersion.set('1.0.656')    // change version of IntelliJ agent and reporter
     jacocoEngineVersion.set('0.8.7')        // change version of JaCoCo agent and reporter
     generateReportOnCheck = true            // false to do not execute `koverMergedReport` task before `check` task
-    disabledProjects = []                   // ["project-name"] to disable coverage for project with name `project-name`
+    disabledProjects = []                   // [":project-name"] to disable coverage for project with path `:project-name` (`:` for the root project)
     instrumentAndroidPackage = false        // true to instrument packages `android.*` and `com.android.*`
     runAllTestsForProjectTask = false       // true to run all tests in all projects if `koverHtmlReport`, `koverXmlReport`, `koverReport`, `koverVerify` or `check` tasks executed on some project
 }

gradle.properties

@@ -1,4 +1,4 @@
-version=0.5.0
+version=0.5.1-SNAPSHOT
 group=org.jetbrains.kotlinx
 
 kotlin.code.style=official

src/functionalTest/kotlin/kotlinx/kover/test/functional/cases/MultiProjectTests.kt

@@ -124,7 +124,7 @@ internal class MultiProjectTests : BaseGradleScriptTest() {
             .subproject(subprojectName) {
                 sources("multiproject-common")
             }
-            .configKover { disabledProjects += subprojectName }
+            .configKover { disabledProjects += ":$subprojectName" }
             .build()
             .run("build", "koverReport") {
                 checkDefaultBinaryReport()

src/main/kotlin/kotlinx/kover/KoverPlugin.kt

@@ -55,7 +55,7 @@ class KoverPlugin : Plugin<Project> {
 
     private fun Project.applyToProject(providers: BuildProviders, agents: Map<CoverageEngine, CoverageAgent>) {
         val projectProviders =
-            providers.projects[name] ?: throw GradleException("Kover: Providers for project '$name' was not found")
+            providers.projects[path] ?: throw GradleException("Kover: Providers for project '$name' ('$path') was not found")
 
         val xmlReportTask = createKoverProjectTask(
             XML_REPORT_TASK_NAME,
@@ -165,10 +165,10 @@ class KoverPlugin : Plugin<Project> {
 
         task.group = VERIFICATION_GROUP
 
-        providers.projects.forEach { (projectName, m) ->
-            task.binaryReportFiles.put(projectName, NestedFiles(task.project.objects, m.reports))
-            task.srcDirs.put(projectName, NestedFiles(task.project.objects, m.sources))
-            task.outputDirs.put(projectName, NestedFiles(task.project.objects, m.output))
+        providers.projects.forEach { (projectPath, m) ->
+            task.binaryReportFiles.put(projectPath, NestedFiles(task.project.objects, m.reports))
+            task.srcDirs.put(projectPath, NestedFiles(task.project.objects, m.sources))
+            task.outputDirs.put(projectPath, NestedFiles(task.project.objects, m.output))
         }
 
         task.coverageEngine.set(providers.engine)
@@ -199,8 +199,8 @@ class KoverPlugin : Plugin<Project> {
                 task.mustRunAfter(xmlReportTask)
                 task.mustRunAfter(htmlReportTask)
 
-                task.xmlFiles[proj.name] = xmlReportTask.xmlReportFile
-                task.htmlDirs[proj.name] = htmlReportTask.htmlReportDir
+                task.xmlFiles[proj.path] = xmlReportTask.xmlReportFile
+                task.htmlDirs[proj.path] = htmlReportTask.htmlReportDir
             }
         }
     }
@@ -279,7 +279,7 @@ class KoverPlugin : Plugin<Project> {
         }
         val targetErrorProvider = project.layout.buildDirectory.file("kover/errors/$name.log").map { it.asFile }
 
-        doFirst(BinaryReportCleanupAction(project.name, providers.koverExtension, taskExtension))
+        doFirst(BinaryReportCleanupAction(project.path, providers.koverExtension, taskExtension))
         doLast(MoveIntellijErrorLogAction(sourceErrorProvider, targetErrorProvider))
     }
 }
@@ -289,7 +289,7 @@ class KoverPlugin : Plugin<Project> {
   For this reason, before starting the tests, it is necessary to clear the file from the results of previous runs.
 */
 private class BinaryReportCleanupAction(
-    private val projectName: String,
+    private val projectPath: String,
     private val koverExtensionProvider: Provider<KoverExtension>,
     private val taskExtension: KoverTaskExtension
 ) : Action<Task> {
@@ -302,7 +302,7 @@ private class BinaryReportCleanupAction(
 
         if (!taskExtension.isDisabled
             && !koverExtension.isDisabled
-            && !koverExtension.disabledProjects.contains(projectName)
+            && !koverExtension.disabledProjects.contains(projectPath)
             && koverExtension.coverageEngine.get() == CoverageEngine.INTELLIJ
         ) {
             // IntelliJ engine expected empty file for parallel test execution.
@@ -333,7 +333,7 @@ private class CoverageArgumentProvider(
     @get:Input val excludeAndroidPackage: Provider<Boolean>
 ) : CommandLineArgumentProvider, Named {
 
-    private val projectName: String = task.project.name
+    private val projectPath: String = task.project.path
 
     @Internal
     override fun getName(): String {
@@ -345,7 +345,7 @@ private class CoverageArgumentProvider(
 
         if (taskExtension.isDisabled
             || koverExtensionValue.isDisabled
-            || koverExtensionValue.disabledProjects.contains(projectName)
+            || koverExtensionValue.disabledProjects.contains(projectPath)
         ) {
             return mutableListOf()
         }

src/main/kotlin/kotlinx/kover/Providers.kt

@@ -15,7 +15,7 @@ internal fun Project.createProviders(agents: Map<CoverageEngine, CoverageAgent>)
     val projects: MutableMap<String, ProjectProviders> = mutableMapOf()
 
     allprojects {
-        projects[it.name] = ProjectProviders(
+        projects[it.path] = ProjectProviders(
             it.provider { it.files(if (runAllTests()) allBinaryReports() else it.binaryReports(this)) },
             it.provider { if (runAllTests()) allTestTasks() else it.testTasks(this) },
             it.provider { it.collectDirs(this).first },
@@ -108,7 +108,7 @@ private fun Project.collectDirs(root: Project): Pair<FileCollection, FileCollect
 
 private fun Project.isDisabled(root: Project): Boolean {
     val koverExtension = root.extensions.getByType(KoverExtension::class.java)
-    return koverExtension.isDisabled || koverExtension.disabledProjects.contains(name)
+    return koverExtension.isDisabled || koverExtension.disabledProjects.contains(path)
 }
 
 private fun Project.runAllTests(): Boolean {

src/main/kotlin/kotlinx/kover/engines/commons/Reports.kt

@@ -30,3 +30,5 @@ internal fun String.wildcardsToRegex(): String {
 
     return builder.toString()
 }
+
+internal val ONE_HUNDRED = 100.toBigDecimal()

src/main/kotlin/kotlinx/kover/engines/intellij/IntellijReports.kt

@@ -189,9 +189,9 @@ private fun Task.checkRule(counters: Map<VerificationValueType, Int>, rule: Veri
 
     val ruleName = if (rule.name != null) "'${rule.name}' " else ""
     return if (boundsViolations.size > 1) {
-        "Rule ${ruleName}violated for '${project.name}':" + boundsViolations.joinToString("\n  ", "\n  ")
+        "Rule ${ruleName}violated for '${project.path}':" + boundsViolations.joinToString("\n  ", "\n  ")
     } else if (boundsViolations.size == 1) {
-        "Rule ${ruleName}violated for '${project.name}': ${boundsViolations[0]}"
+        "Rule ${ruleName}violated for '${project.path}': ${boundsViolations[0]}"
     } else {
         null
     }

src/main/kotlin/kotlinx/kover/engines/jacoco/JacocoReports.kt

@@ -6,6 +6,8 @@ package kotlinx.kover.engines.jacoco
 
 import groovy.lang.*
 import kotlinx.kover.api.*
+import kotlinx.kover.engines.commons.*
+import kotlinx.kover.engines.commons.ONE_HUNDRED
 import kotlinx.kover.engines.commons.Report
 import org.gradle.api.*
 import org.gradle.api.file.*
@@ -40,7 +42,7 @@ private fun Task.callJacocoAntReportTask(
         invokeWithBody("executiondata") {
             project.files(report.files).addToAntBuilder(this, "resources")
         }
-        invokeWithBody("structure", mapOf("name" to project.name)) {
+        invokeWithBody("structure", mapOf("name" to project.path)) {
             invokeWithBody("sourcefiles") {
                 project.files(sources).addToAntBuilder(this, "resources")
             }
@@ -93,8 +95,8 @@ internal fun Task.jacocoVerification(
                             }
                             VerificationValueType.COVERED_LINES_PERCENTAGE -> {
                                 limitArgs["value"] = "COVEREDRATIO"
-                                min = min?.divide(BigDecimal(100))
-                                max = max?.divide(BigDecimal(100))
+                                min = min?.divide(ONE_HUNDRED)
+                                max = max?.divide(ONE_HUNDRED)
                             }
                         }
 

src/main/kotlin/kotlinx/kover/tasks/KoverCollectingTask.kt

@@ -24,20 +24,26 @@ open class KoverCollectingTask : DefaultTask() {
             it.into(outputDir)
             xmlFiles.forEach { (p, f) ->
                 it.from(f) { c ->
-                    c.rename { "$p.xml" }
+                    c.rename { "${p.pathAsFilename()}.xml" }
                 }
             }
         }
 
 
         htmlDirs.forEach { (p, d) ->
+            val name = p.pathAsFilename()
+
             // delete directory for HTML reports so that the old reports do not overlap with the new ones
-            project.delete(outputDir.dir("html/$p"))
+            project.delete(outputDir.dir("html/$name"))
 
             project.copy {
                 it.from(d)
-                it.into(outputDir.dir("html/$p"))
+                it.into(outputDir.dir("html/$name"))
             }
         }
     }
+
+    private fun String.pathAsFilename(): String {
+        return if (this == ":") "_root_" else replace(':', '_')
+    }
 }

src/main/kotlin/kotlinx/kover/tasks/KoverHtmlReport.kt

@@ -44,7 +44,7 @@ open class KoverMergedHtmlReportTask : KoverMergedTask() {
 
 @CacheableTask
 open class KoverHtmlReportTask : KoverProjectTask() {
-    private val projectName = project.name
+    private val projectPath = project.path
 
     /**
      * Specifies directory path of generated HTML report.
@@ -72,6 +72,6 @@ open class KoverHtmlReportTask : KoverProjectTask() {
                 classpath.get(),
             )
         }
-        logger.lifecycle("Kover: HTML report for '$projectName' file://${htmlDirFile.canonicalPath}/index.html")
+        logger.lifecycle("Kover: HTML report for '$projectPath' file://${htmlDirFile.canonicalPath}/index.html")
     }
 }