From 159a6969c2ed87db4e89223987408fed91211cf1 Mon Sep 17 00:00:00 2001
From: Shinokuni <shinokuni@openserver.fr>
Date: Fri, 20 Sep 2024 16:40:20 +0200
Subject: [PATCH] Migrate Jacoco task to Kotlin DSL

---
 .github/workflows/android.yml |  4 +-
 api/build.gradle.kts          |  4 ++
 build.gradle.kts              | 88 +++++++++++++++++++----------------
 db/build.gradle.kts           |  5 ++
 4 files changed, 58 insertions(+), 43 deletions(-)

diff --git a/.github/workflows/android.yml b/.github/workflows/android.yml
index f824a80f..ebc871c9 100644
--- a/.github/workflows/android.yml
+++ b/.github/workflows/android.yml
@@ -32,6 +32,6 @@ jobs:
           script: ./gradlew clean build connectedCheck
       - uses: codecov/codecov-action@v2.1.0
         with:
-          files: ./build/reports/jacoco/jacocoFullReport.xml
-          fail_ci_if_error: false
+          files: ./build/reports/jacoco/jacocoFullReport/jacocoFullReport.xml
+          fail_ci_if_error: true
           verbose: true
\ No newline at end of file
diff --git a/api/build.gradle.kts b/api/build.gradle.kts
index 40e54751..e514d723 100644
--- a/api/build.gradle.kts
+++ b/api/build.gradle.kts
@@ -7,6 +7,10 @@ android {
     namespace = "com.readrops.api"
 
     buildTypes {
+        debug {
+            enableUnitTestCoverage = true
+        }
+
         create("beta") {
             initWith(getByName("release"))
 
diff --git a/build.gradle.kts b/build.gradle.kts
index 7f8b0bea..73428be3 100644
--- a/build.gradle.kts
+++ b/build.gradle.kts
@@ -1,4 +1,3 @@
-
 import com.android.build.gradle.AppPlugin
 import com.android.build.gradle.BaseExtension
 import com.android.build.gradle.LibraryExtension
@@ -24,11 +23,10 @@ buildscript {
 plugins {
     alias(libs.plugins.ksp) apply false
     alias(libs.plugins.compose.compiler) apply false
+    jacoco
 }
 
 allprojects {
-    //apply("jacoco")
-
     repositories {
         google()
         mavenCentral()
@@ -87,54 +85,62 @@ tasks.register<Delete>("clean") {
     delete(rootProject.layout.buildDirectory)
 }
 
-/*jacoco {
+jacoco {
     toolVersion = "0.8.7"
 }
 
 
-tasks.register("jacocoFullReport", JacocoReport) {
+tasks.register<JacocoReport>("jacocoFullReport") {
     group = "Reporting"
     description = "Generate Jacoco coverage reports for the debug build"
 
     reports {
-        html {
-            enabled true
-            destination file("build/reports/jacoco/html")
-        }
-        xml {
-            enabled true
-            destination file("build/reports/jacoco/jacocoFullReport.xml")
-        }
+        xml.required.set(true)
+        html.required.set(true)
     }
 
-    //dependsOn ":app:testDebugUnitTest"
-    dependsOn ":api:testDebugUnitTest"
-    dependsOn ":db:testDebugUnitTest"
-    //dependsOn ":app:connectedAndroidTest"
-    dependsOn ":db:connectedAndroidTest"*/
+    dependsOn(":app:testDebugUnitTest")
+    dependsOn(":api:testDebugUnitTest")
+    dependsOn(":db:testDebugUnitTest")
+    dependsOn(":app:connectedAndroidTest")
+    dependsOn(":db:connectedAndroidTest")
 
-//final fileFilter = ["**/R.class", "**/R\$*.class", "**/BuildConfig.*", "**/Manifest*.*", "android/**/*.*"]
+    val excludeFilter = listOf(
+        "**/R.class",
+        "**/R\$*.class",
+        "**/BuildConfig.*",
+        "**/Manifest*.*",
+        "android/**/*.*"
+    )
 
-/*    classDirectories.setFrom files([
-            //fileTree(dir: "$project.rootDir/app/build/intermediates/javac/debug", excludes: fileFilter),
-            //fileTree(dir: "$project.rootDir/app/build/tmp/kotlin-classes/debug", excludes: fileFilter),
-            fileTree(dir: "$project.rootDir/api/build/intermediates/javac/debug", excludes: fileFilter),
-            fileTree(dir: "$project.rootDir/api/build/tmp/kotlin-classes/debug", excludes: fileFilter),
-            fileTree(dir: "$project.rootDir/db/build/tmp/kotlin-classes/debug", excludes: fileFilter),
-    ])
-    def coverageSourceDirs = [
-            "$project.rootDir/app/src/main/java",
-            "$project.rootDir/api/src/main/java",
-            "$project.rootDir/db/src/main/java",
-    ]
+    classDirectories.setFrom(
+        files(
+            fileTree("${project.rootDir}/app/build/intermediates/javac/debug") { exclude(excludeFilter) },
+            fileTree("${project.rootDir}/app/build/tmp/kotlin-classes/debug") { exclude(excludeFilter) }
+        ),
+        fileTree("${project.rootDir}/api/build/tmp/kotlin-classes/debug")  { exclude(excludeFilter) },
+        fileTree("${project.rootDir}/db/build/tmp/kotlin-classes/debug")  { exclude(excludeFilter) },
+    )
 
-    additionalSourceDirs.setFrom files(coverageSourceDirs)
-    sourceDirectories.setFrom files(coverageSourceDirs)
-    executionData.setFrom fileTree(dir: project.rootDir, includes: [
-            "app/jacoco.exec",
-            "db/jacoco.exec",
-            "api/jacoco.exec",
-            "app/build/outputs/code_coverage/debugAndroidTest/connected/*-coverage.ec",
-            "db/build/outputs/code_coverage/debugAndroidTest/connected/*-coverage.ec"
-    ])
-}*/
+    val coverageSourceDirs = listOf(
+        "${project.rootDir}/app/src/main/java",
+        "${project.rootDir}/api/src/main/java",
+        "${project.rootDir}/db/src/main/java",
+    )
+
+    additionalSourceDirs.setFrom(files(coverageSourceDirs))
+    sourceDirectories.setFrom(files(coverageSourceDirs))
+
+    executionData.setFrom(
+        fileTree(project.rootDir) {
+            include(
+                listOf(
+                    "api/build/outputs/unit_test_code_coverage/**/*.exec",
+                    "db/build/outputs/unit_test_code_coverage/**/*.exec",
+                    "app/build/outputs/code_coverage/debugAndroidTest/connected/**/*.ec",
+                    "db/build/outputs/code_coverage/debugAndroidTest/connected/**/*.ec"
+                )
+            )
+        }
+    )
+}
diff --git a/db/build.gradle.kts b/db/build.gradle.kts
index 29315466..7065a505 100644
--- a/db/build.gradle.kts
+++ b/db/build.gradle.kts
@@ -8,6 +8,11 @@ android {
     namespace = "com.readrops.db"
 
     buildTypes {
+        debug {
+            enableUnitTestCoverage = true
+            enableAndroidTestCoverage = true
+        }
+
         create("beta") {
             initWith(getByName("release"))