Kover exclude rules in Sonarqube scan

[johnryM]

I'm using Kover in my Android Project to generate the xml report that's sent to my Sonarqube server.

I would like to exclude all Kotlin classes that are inside a "di" folder but my Kover exclude rules don't seem to work.

My package structure is

My merged report gradle config is

// in root build.gradle

tasks.koverMergedXmlReport {
    enabled = true
    xmlReportFile.set(layout.buildDirectory.file("kover/xml/result.xml"))
    excludes = [
        "*.*Module.*",
        "*.di.*Module*",
        "*.di.*"
    ]
}

sonarqube {
    properties {
        property "sonar.sourceEncoding", "UTF-8"
        property "sonar.coverage.jacoco.xmlReportPaths", "$buildDir/kcover/xml/result.xml"


    }
}

project.tasks["sonarqube"].dependsOn "koverMergedXmlReport"

When I run my sonarqube scan. The module files in the various DI folders are visible in the coverage report

Defining "sonar.exclusions property" in my build gradle file does solve this issue

   property("sonar.exclusions","**/di/*")

Is this the correct way of excluding files in the final sonarqube report or is possible to perform the same exclusions by just configuring the Kover merged reports?

[shanshin]

Hi,
at the moment, full integration with Sonarqube is not implemented, so for the correct display of reports, you need to configure it yourself.

[johnryM]

Thanks @shanshin for clarifying. Will there be plans to support full integration with analysis tools like Sonarqube?

[shanshin]

There are no short-term plans for integration with Sonarqube, but it will be added as soon as the plugin and the IntelliJ Coverage Engine are implemented to a stable state.

[sarn0ld]

@johnryM with the property you mentioned above you will afaik exclude the files from sonar completely (coverage + static analysis). To just exclude them from coverage you need to use this property:

property "sonar.coverage.exclusions", "**/di/*"

Edit: We need to do the same when using Jacoco reports, so I guess it's just the intended sonarqube behavior that you need to define the exclusions twice.

[johnryM]

@sarn0ld In the end we went with your solution. I didn't know it was the same process with Jacoco as well. Thanks for letting me know!

[shanshin]

Duplicates #268