Add detection of scala version via 'implementation' and 'compileOnly' configurations

Author: eyalroth

src/functionalTest/java/org.scoverage/DetectScalaLibraryTest.java

@@ -0,0 +1,43 @@
+package org.scoverage;
+
+import org.junit.Assert;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.Parameterized;
+
+import java.util.Arrays;
+import java.util.Collection;
+
+@RunWith(Parameterized.class)
+public class DetectScalaLibraryTest extends ScoverageFunctionalTest {
+
+    private static final String SCALA_VERSION = "0.0";
+    private static final String SCALA_LIBRARY_PARAMETER = "-PdetectedScalaLibraryVersion=" + SCALA_VERSION + ".0";
+    private static final String EXPECTED_OUTPUT = "Using scoverage scalac plugin version '" + SCALA_VERSION;
+
+    @Parameterized.Parameter(0)
+    public String projectDir;
+
+    @Parameterized.Parameters(name = "{index}: Project {0} ")
+    public static Collection<Object[]> data() {
+        Object[][] data = new Object[][]{{"/compile"}, {"/compileOnly"}, {"/implementation"}, {"/dependency-management"}};
+        return Arrays.asList(data);
+    }
+
+    public DetectScalaLibraryTest() {
+        super(null);
+    }
+
+    @Test
+    public void test() {
+        setProjectName("detect-scala-library" + projectDir);
+
+        // build supposed to fail since repositories are not configured
+        AssertableBuildResult result = runAndFail("clean", SCALA_LIBRARY_PARAMETER, "--info");
+
+        // all we want to know is that the plugin detected our configured library version
+        String output = result.getResult().getOutput();
+        Assert.assertTrue(output.contains(EXPECTED_OUTPUT));
+    }
+
+}

src/functionalTest/java/org.scoverage/ScoverageFunctionalTest.java

@@ -23,18 +23,12 @@
 
 public abstract class ScoverageFunctionalTest {
 
-    private final String projectName;
-    private final GradleRunner runner;
+    private String projectName;
+    private GradleRunner runner;
     private final XmlParser parser;
 
     protected ScoverageFunctionalTest(String projectName) {
-
-        this.projectName = projectName;
-        this.runner = GradleRunner.create()
-                .withProjectDir(projectDir())
-                .withPluginClasspath()
-                .forwardOutput();
-
+        setProjectName(projectName);
         try {
             this.parser = new XmlParser();
             parser.setFeature("http://apache.org/xml/features/disallow-doctype-decl", false);
@@ -44,6 +38,16 @@ protected ScoverageFunctionalTest(String projectName) {
         }
     }
 
+    protected void setProjectName(String projectName) {
+        if (projectName != null) {
+            this.projectName = projectName;
+            this.runner = GradleRunner.create()
+                    .withProjectDir(projectDir())
+                    .withPluginClasspath()
+                    .forwardOutput();
+        }
+    }
+
     protected File projectDir() {
 
         return new File("src/functionalTest/resources/projects/" + projectName);

src/functionalTest/resources/projects/detect-scala-library/compile/build.gradle

@@ -0,0 +1,9 @@
+plugins {
+    id 'org.scoverage'
+}
+
+description = 'defines scala library using the "compile" configuration'
+
+dependencies {
+    compile group: 'org.scala-lang', name: 'scala-library', version: "${detectedScalaLibraryVersion}"
+}

src/functionalTest/resources/projects/detect-scala-library/compile/settings.gradle

@@ -0,0 +1,9 @@
+plugins {
+    id 'org.scoverage'
+}
+
+description = 'defines scala library using the "compileOnly" configuration'
+
+dependencies {
+    compileOnly group: 'org.scala-lang', name: 'scala-library', version: "${detectedScalaLibraryVersion}"
+}

src/functionalTest/resources/projects/detect-scala-library/compileOnly/build.gradle

@@ -0,0 +1,16 @@
+plugins {
+    id 'io.spring.dependency-management' version "1.0.4.RELEASE"
+    id 'org.scoverage'
+}
+
+description = 'defines scala library using the "implementation" configuration and the dependency-management plugin'
+
+dependencyManagement {
+    dependencies {
+        dependency group: 'org.scala-lang', name: 'scala-library', version: "${detectedScalaLibraryVersion}"
+    }
+}
+
+dependencies {
+    implementation group: 'org.scala-lang', name: 'scala-library'
+}

src/functionalTest/resources/projects/detect-scala-library/compileOnly/settings.gradle

@@ -0,0 +1,9 @@
+plugins {
+    id 'org.scoverage'
+}
+
+description = 'defines scala library using the "implementation" configuration'
+
+dependencies {
+    implementation group: 'org.scala-lang', name: 'scala-library', version: "${detectedScalaLibraryVersion}"
+}

src/functionalTest/resources/projects/detect-scala-library/dependency-management/build.gradle

@@ -63,27 +63,8 @@ class ScoveragePlugin implements Plugin<PluginAware> {
             }
 
             project.afterEvaluate {
+                def scalaVersion = resolveScalaVersion(project)
                 def scoverageVersion = project.extensions.scoverage.scoverageVersion.get()
-                def scalaVersion = null
-
-                def scalaLibrary = project.configurations.compile.dependencies.find {
-                    it.group == "org.scala-lang" && it.name == "scala-library"
-                }
-
-                if (scalaLibrary != null) {
-                    scalaVersion = scalaLibrary.version
-                }
-
-                if (scalaVersion == null && project.pluginManager.hasPlugin("io.spring.dependency-management")) {
-                    scalaVersion = project.dependencyManagement.compile.managedVersions["org.scala-lang:scala-library"]
-                }
-
-                if (scalaVersion == null) {
-                    scalaVersion = project.extensions.scoverage.scoverageScalaVersion.get()
-                } else {
-                    scalaVersion = scalaVersion.substring(0, scalaVersion.lastIndexOf("."))
-                }
-
                 def fullScoverageVersion = "$scalaVersion:$scoverageVersion"
 
                 project.logger.info("Using scoverage scalac plugin version '$fullScoverageVersion'")
@@ -319,6 +300,39 @@ class ScoveragePlugin implements Plugin<PluginAware> {
         }
     }
 
+    private String resolveScalaVersion(Project project) {
+        def scalaVersion = null
+
+        def configurations = [
+                project.configurations.compile,
+                project.configurations.compileOnly,
+                project.configurations.implementation
+        ]
+        def dependencies = configurations.collectMany { it.dependencies }
+
+        def scalaLibrary = dependencies.find {
+            it.group == "org.scala-lang" && it.name == "scala-library"
+        }
+
+        if (scalaLibrary != null) {
+            scalaVersion = scalaLibrary.version
+        }
+
+        if (scalaVersion == null && project.pluginManager.hasPlugin("io.spring.dependency-management")) {
+            scalaVersion = project.dependencyManagement.compile.managedVersions["org.scala-lang:scala-library"]
+        }
+
+        if (scalaVersion == null) {
+            project.logger.info("No scala library detected. Using property 'scoverageScalaVersion'")
+            scalaVersion = project.extensions.scoverage.scoverageScalaVersion.get()
+        } else {
+            project.logger.info("Detected scala library in compilation classpath")
+            scalaVersion = scalaVersion.substring(0, scalaVersion.lastIndexOf("."))
+        }
+
+        return scalaVersion
+    }
+
     private Set<? extends Task> recursiveDependenciesOf(Task task) {
         if (!taskDependencies.containsKey(task)) {
             def directDependencies = task.getTaskDependencies().getDependencies(task)