Improve Gradle configuration

build.gradle

@@ -21,7 +21,7 @@ scmVersion {
 allprojects {
     // Apply common project setup
     apply from: "${project.rootDir}/project.gradle"
-    version = scmVersion.version
+    version = rootProject.scmVersion.version
 }
 
 subprojects {

buildSrc/src/main/groovy/TestSummaryListener.groovy

@@ -0,0 +1,32 @@
+import org.gradle.api.tasks.testing.TestDescriptor
+import org.gradle.api.tasks.testing.TestListener
+import org.gradle.api.tasks.testing.TestResult
+
+class TestSummaryListener implements TestListener, Serializable {
+
+    @Override
+    void beforeSuite(TestDescriptor suite) {}
+
+    @Override
+    void afterSuite(TestDescriptor desc, TestResult result) {
+        if (desc.parent == null) {
+            def output = "Results: ${result.resultType} (${result.testCount} tests, ${result.successfulTestCount} passed, ${result.failedTestCount} failed, ${result.skippedTestCount} skipped)"
+            def startItem = '|  '
+            def endItem = '  |'
+            def repeatLength = startItem.length() + output.length() + endItem.length()
+
+            println '\n' +
+                    ('-' * repeatLength) + '\n' +
+                    startItem + output + endItem + '\n' +
+                    ('-' * repeatLength)
+        }
+    }
+
+    @Override
+    void beforeTest(TestDescriptor desc) {}
+
+    @Override
+    void afterTest(TestDescriptor desc, TestResult result) {
+        println "${desc.className} > ${desc.name} took: ${(result.endTime - result.startTime)}ms"
+    }
+}

gradle.properties

@@ -1,3 +1,4 @@
+org.gradle.configuration-cache=true
 org.gradle.jvmargs=-Xmx3g
 org.gradle.parallel=false
 
@@ -15,5 +16,8 @@ projectName = openremote-extensions
 
 openremoteVersion = 1.27.0-SNAPSHOT
 
+axionReleasePluginVersion = 1.21.2
+gradleNexusPublishPluginVersion = 2.0.0
+ideaExtPluginVersion = 1.2
 testLoggerVersion = 4.0.0
 

project.gradle

@@ -174,20 +174,9 @@ plugins.withType(JavaPlugin).whenPluginAdded {
             }
             info.events = debug.events
             info.exceptionFormat = debug.exceptionFormat
-
-            afterTest { desc, result ->
-                logger.quiet "${desc.className} > ${desc.name} took: ${(result.endTime - result.startTime)}ms"
-            }
-
-            afterSuite { desc, result ->
-                if (!desc.parent) { // will match the outermost suite
-                    def output = "Results: ${result.resultType} (${result.testCount} tests, ${result.successfulTestCount} passed, ${result.failedTestCount} failed, ${result.skippedTestCount} skipped)"
-                    def startItem = '|  ', endItem = '  |'
-                    def repeatLength = startItem.length() + output.length() + endItem.length()
-                    println('\n' + ('-' * repeatLength) + '\n' + startItem + output + endItem + '\n' + ('-' * repeatLength))
-                }
-            }
         }
+
+        addTestListener(new TestSummaryListener())
     }
 }
 

settings.gradle

@@ -1,15 +1,32 @@
 plugins {
-    id 'io.github.gradle-nexus.publish-plugin' version '2.0.0' apply false
-    id "org.jetbrains.gradle.plugin.idea-ext" version "1.3" apply false
-    id 'pl.allegro.tech.build.axion-release' version '1.21.1' apply false
+    id 'io.github.gradle-nexus.publish-plugin' version "$gradleNexusPublishPluginVersion" apply false
+    id 'org.jetbrains.gradle.plugin.idea-ext' version "$ideaExtPluginVersion" apply false
+    id 'pl.allegro.tech.build.axion-release' version "$axionReleasePluginVersion" apply false
 }
 
 rootProject.name = "$projectName"
 
 // Include sub-projects dynamically, every directory with a build.gradle (and no .buildignore)
-fileTree(dir: rootDir, include: "**/build.gradle", excludes: ["**/node_modules/**", "**/generic_app/**"])
-        .filter { it.parent != rootDir }
-        .filter { !file("${it.parent}/.buildignore").exists() }
+fileTree(
+        dir: rootDir,
+        include: "**/build.gradle",
+        excludes: [
+                ".git/**",
+                "**/.git/**",
+                ".gradle/**",
+                "**/.gradle/**",
+                "**/build/**",
+                "**/node_modules/**",
+                "**/generic_app/**",
+                "**/dist/**",
+                "**/lib/**"
+        ]
+)
+        .filter { it.parentFile != rootDir }
+        .filter { !new File(it.parentFile, ".buildignore").exists() }
         .each {
-            include it.parent.replace(rootDir.canonicalPath, "").replace("\\", ":").replace("/", ":")
+            include it.parentFile.canonicalPath
+                    .replace(rootDir.canonicalPath, "")
+                    .replace("\\", ":")
+                    .replace("/", ":")
         }