[LD-161] Add coverage for components module and update excluded dirs.

.github/workflows/run-test.yml

@@ -181,15 +181,15 @@ jobs:
         with:
           name: coverage-report
           path: |
-            app/build/reports/jacoco/codeCoverage
+            */build/reports/jacoco/codeCoverage
 
       - name: Add coverage to PR
         if: ${{ github.event_name != 'schedule' }}
         id: jacoco
         uses: madrapps/[email protected]
         with:
           paths: |
-            app/build/reports/jacoco/codeCoverage/codeCoverage.xml
+            */build/reports/jacoco/codeCoverage/codeCoverage.xml
           token: ${{ secrets.GITHUB_TOKEN }}
 
   test-license-headers:

app/build.gradle

@@ -4,9 +4,10 @@ plugins {
     alias(libs.plugins.kotlin.android)
     alias(libs.plugins.ktlint.gradle)
     alias(libs.plugins.kotlin.serialization)
-    id 'jacoco'
 }
 
+apply from: '../module-jacoco.gradle'
+
 def fileName = "../app/google-services.json"
 
 if (project.file(fileName).exists()) {
@@ -102,43 +103,6 @@ android {
     }
 }
 
-tasks.register('codeCoverage', JacocoReport) {
-
-    reports {
-        html.required.set(true)
-        xml.required.set(true)
-    }
-
-    classDirectories.setFrom(
-            fileTree(project.buildDir) {
-                include("**/tmp/kotlin-classes/debug/**")
-                exclude(
-                        '**/R.class',
-                        '**/R$*.class',
-                        '**/BuildConfig.*',
-                        "**/*Application.*",
-                        '**/Manifest*.*',
-                        '**/*Test*.*',
-                        '**/android/**/*.*',
-                        '**/androidx/**/*.*',
-                        '**/airbnb/**/*.*',
-                        "**/di/**/*.*",
-                        '**/*Dagger*.*',
-                        "**/*Screen*"
-                )
-            }
-    )
-
-    sourceDirectories.setFrom(
-            fileTree(dir: project.projectDir, includes: ["src/main/java/**", "src/main/kotlin/**"])
-    )
-
-    executionData.setFrom(
-            fileTree(dir: "${buildDir}/test-results/instrumentation_results", includes: ['*.ec']),
-            fileTree(dir: "${buildDir}/outputs/unit_test_code_coverage", includes: ["**/*.exec", "**/*.ec"])
-    )
-}
-
 dependencies {
     api project(':components')
 

build.gradle

@@ -7,3 +7,23 @@ plugins {
     alias(libs.plugins.kotlin.kapt) apply false
     alias(libs.plugins.google.services) apply false
 }
+
+subprojects {
+    plugins.withId("app.cash.paparazzi") {
+        // Defer until afterEvaluate so that testImplementation is created by Android plugin.
+        afterEvaluate {
+            dependencies.constraints {
+                add("testImplementation", "com.google.guava:guava") {
+                    attributes {
+                        attribute(
+                                TargetJvmEnvironment.TARGET_JVM_ENVIRONMENT_ATTRIBUTE,
+                                objects.named(TargetJvmEnvironment, TargetJvmEnvironment.STANDARD_JVM)
+                        )
+                    }
+                    because("LayoutLib and sdk-common depend on Guava's -jre published variant." +
+                            "See https://github.com/cashapp/paparazzi/issues/906.")
+                }
+            }
+        }
+    }
+}

components/build.gradle

@@ -6,6 +6,8 @@ plugins {
     alias(libs.plugins.kotlin.kapt)
 }
 
+apply from: '../module-jacoco.gradle'
+
 java {
     toolchain {
         languageVersion = JavaLanguageVersion.of(17)
@@ -37,6 +39,13 @@ android {
             arg("skipPrivatePreviews", "true")
         }
     }
+
+    buildTypes {
+        debug {
+            enableUnitTestCoverage true
+            testCoverageEnabled true
+        }
+    }
 }
 
 dependencies {

gradle/libs.versions.toml

@@ -42,7 +42,7 @@ androidLibrary = "8.1.2"
 kotlin = "1.9.0"
 kotlinSerialization = "1.9.0"
 ktlintGradle = "11.6.1"
-paparazzi = "1.3.1"
+paparazzi = "1.3.2"
 totp = "1.7.1"
 uiTestJunit4 = "1.5.4"
 

module-jacoco.gradle

@@ -0,0 +1,35 @@
+apply plugin: 'jacoco'
+
+tasks.register('codeCoverage', JacocoReport) {
+
+    reports {
+        html.required.set(true)
+        xml.required.set(true)
+    }
+
+    classDirectories.setFrom(
+            fileTree(project.buildDir) {
+                include("**/tmp/kotlin-classes/debug/**")
+                exclude(
+                        '**/R.class',
+                        '**/R$*.class',
+                        '**/BuildConfig.*',
+                        '**/Manifest*.*',
+                        '**/airbnb/**/*.*',
+                        "**/*Screen*",
+                        "**/Showkase*",
+                        "**/theme/Theme*",
+                        "**/theme/Type*",
+                )
+            }
+    )
+
+    sourceDirectories.setFrom(
+            fileTree(dir: project.projectDir, includes: ["src/main/java/**", "src/main/kotlin/**"])
+    )
+
+    executionData.setFrom(
+            fileTree(dir: "${buildDir}/test-results/instrumentation_results", includes: ['*.ec']),
+            fileTree(dir: "${buildDir}/outputs/unit_test_code_coverage", includes: ["**/*.exec", "**/*.ec"])
+    )
+}