Scoverage Plugin Resolving dependencies at configuration time

[dthg]

I'm having some issues using the gradle-scoverage plugin alongside radle-consistent-versions. gradle-consistent-versions enforces that no dependencies are resolved at compile time( see).

I opened an issue with gradle-consistent-versions palantir/gradle-consistent-versions#494 , but ended up closing that and opening one here instead.

Is this something you are open to fixing? I'm happy to attempt writing a patch if that is the case (still new to gradle plugin development though).

[eyalroth]

The scoverage plugin is resolving the configurations in order to detect the Scala version (according to the version of the scala library dependency), which is required in order to know the name (not version) of the scalac scoverage dependencies (org.scoverage:scalac-scoverage-plugin_$scalaVersion, org.scoverage:scalac-scoverage-runtime_$scalaVersion).

This is a common issue with Scala dependencies, as each scala version requires a different artifact name; i.e, the scala version is not part of the dependency version. I wish Gradle would've add a more builtin and generic support, allowing to specify scala version in one place and make it available in the configuration.

Maybe there's a way to make this plugin compatible with the gradle-consistent-versions plugin, I'll have to look at this at a later time.

For now, you can try using Spring's dependency-management plugin, or use the native java-platform plugin to manage dependencies (available since Gradle 5 I believe).

[eyalroth]

Unfortunately, I believe this cannot be fixed.

The scoverage plugin must resolve two of the configuration-sets at configuration time:

1. compileClasspath - in order to resolve the scala version (as explained earlier).
2. scoverage - the plugin's own configuration, in order to determine the absolute path of the scalac-scoverage-plugin jar, which is required in order to configure the compilation task (-Xplugin:/path/to/plugin.jar).

I believe that the only way to make the two plugins compatible with each other is to remove the banning of dependency resolution at configuration time on palantir's plugin side (or at least provide an option to disable it).

I've added a commit with only a new functional test for the compatibility with the palantir plugin, if you wish to check it out.

[nyonson]

Would it be possible to add some options to the plugin extension to configure the scala version and path instead of relying on resolving the configurations?

[eyalroth]

@y0ns0n There's already an option to specify the scala version (scoverageScalaVersion). See the documentation.

Alas, the path of the plugin cannot be specified and it would be futile to add such an option, as the path is dynamic; the JAR file is resolved and downloaded by Gradle.

[nyonson]

I have seen some hacks to avoid resolving the configuration for downloaded jars like the following:

task downloadChromedriver(type: Copy, description: 'Downloads chromedriver from the repo', group: 'Resource') {
    // defer configuration resolution to execution time
    dependsOn configurations.chromedriver
    from {
        configurations.chromedriver.singleFile
    }
    into chromedriverDir
}

Could we use that approach here to copy the jar to a known spot?

[eyalroth]

@y0ns0n It doesn't seem to work I'm afraid :(

I tried this:

def pluginDependency = project.configurations[CONFIGURATION_NAME].incoming.dependencies.find {
    it.name.contains("scalac-scoverage-plugin")
}
def copyPlugin = project.tasks.create('copyScoveragePlugin',  org.gradle.api.tasks.Copy) {
    from pluginDependency
    into '/tmp/scalac-scoverage-plugin.jar'
}
compileTask.dependsOn(copyPlugin)
compileTask.configure {
    List<String> parameters = ['-Xplugin:' + '/tmp/scalac-scoverage-plugin.jar']
    // ...
}

But then I'm getting the exception on the copy task:

Execution failed for task ':copyScoveragePlugin'.
> Cannot convert the provided notation to a File or URI: DefaultExternalModuleDependency{group='org.scoverage', name='scalac-scoverage-plugin_2.12', version='1.4.1', configuration='default'}.
  The following types/formats are supported:
    - A String or CharSequence path, for example 'src/main/java' or '/usr/include'.
    - A String or CharSequence URI, for example 'file:/usr/include'.
    - A File instance.
    - A Path instance.
    - A Directory instance.
    - A RegularFile instance.
    - A URI or URL instance.
    - A TextResource instance.

I'm not exactly sure what is the 'chromedriver' configuration from your example, but it doesn't seem to be a dependency; rather, it looks like a custom object containing a reference to the file.

[nyonson]

Sorry about the lack of context in my example. chromedriver is a custom configuration which we toss a big chrome dependency on. We then grab that jar and use it in integration tests:

configurations {
    chromedriver
}

dependencies {
  chromedriver(group: 'com.google.chrome', name: 'chromedriver', version: '2.25', classifier: 'mac64', ext: 'bin')
}

I wonder if we could change the copyPlugin method to take a closure:

def copyPlugin = project.tasks.create('copyScoveragePlugin',  org.gradle.api.tasks.Copy) {
    dependsOn project.configurations[CONFIGURATION_NAME]
    from {
        project.configurations[CONFIGURATION_NAME].incoming.dependencies.find {
          it.name.contains("scalac-scoverage-plugin")
      }
    }
    into '/tmp/scalac-scoverage-plugin.jar'
}

After that it looks like you are just dealing with a type issue, but I think there must be a way to change it to a file similar to the way we are using the singleFile method.

[eyalroth]

@y0ns0n I'm pretty certain that the singleFile on a configuration resolves it, so that should fail as well.

I tried this:

project.configurations.create('scoveragePlugin') {
    visible = false
    transitive = false
}

project.dependencies {
    scoveragePlugin("org.scoverage:scalac-scoverage-plugin_2.12:1.4.1")
}

def copyPlugin = project.tasks.create('copyScoveragePlugin',  org.gradle.api.tasks.Copy) {
    dependsOn project.configurations.scoveragePlugin
    from project.configurations.scoveragePlugin.singleFile
    into '/tmp/scoverage/scalac-scoverage-plugin.jar'
}
compileTask.dependsOn(copyPlugin)

which yields "Not allowed to resolve configuration ':scoveragePlugin' at configuration time".

[nyonson]

singleFile does resolve it, but we defer it by putting it in a closure. Sofrom { project.configurations.scoveragePlugin.singleFile } instead of from project.configurations.scoveragePlugin.singleFile.

[eyalroth]

@y0ns0n Huh, that actually works, and it turns out there's an even simpler solution -- configuring the plugin parameter in doFirst.

I opened a PR :)