ci: add caching (#1183)

* ci: add caching

Add caching when building off master, and use the caches in the ci file.

* ci: minor step name change

* ci: publish unit test results to PR

* ci: fix unit test results pathing

* ci: remove junit from danger plugin

Remove as it has never been used.

.github/workflows/master.yml

@@ -5,7 +5,7 @@ on:
     branches: [ master ]
 
 jobs:
-  assemble:
+  build:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
@@ -14,37 +14,16 @@ jobs:
         with:
           distribution: 'adopt'
           java-version: 17
+      - name: setup gradle cache
+        uses: gradle/actions/setup-gradle@v3
+        with:
+          cache-read-only: false
       - name: run assemble
         run: ./gradlew assemble
-
-  test:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - name: set up JDK 17
-        uses: actions/setup-java@v4
-        with:
-          distribution: 'adopt'
-          java-version: 17
       - name: run kover
         run: ./gradlew koverMergedXmlReport
       - uses: codecov/codecov-action@v4
-
-  analyze:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - name: set up JDK 17
-        uses: actions/setup-java@v4
-        with:
-          distribution: 'adopt'
-          java-version: 17
-      - name: run lintDebug
+      - name: run lint
         run: ./gradlew :app:lint
       - name: run detektCheck
         run: ./gradlew detektCheck
-      - name: upload linting results
-        uses: actions/upload-artifact@v4
-        with:
-          name: lint
-          path: build/reports/

.github/workflows/pre-merge.yml

@@ -13,6 +13,8 @@ jobs:
         with:
           distribution: 'adopt'
           java-version: 17
+      - name: setup gradle cache
+        uses: gradle/actions/setup-gradle@v3
       - name: run assemble
         run: ./gradlew assemble
 
@@ -25,9 +27,18 @@ jobs:
         with:
           distribution: 'adopt'
           java-version: 17
+      - name: setup gradle cache
+        uses: gradle/actions/setup-gradle@v3
       - name: run kover
         run: ./gradlew koverMergedXmlReport
       - uses: codecov/codecov-action@v4
+      - name: publish results
+        uses: EnricoMi/publish-unit-test-result-action@v2
+        if: always()
+        with:
+          files: ./**/build/test-results/**/*.xml
+          action_fail: true
+          action_fail_on_inconclusive: true
 
   ktlint:
     runs-on: ubuntu-latest
@@ -38,6 +49,8 @@ jobs:
         with:
           distribution: 'adopt'
           java-version: 17
+      - name: setup gradle cache
+        uses: gradle/actions/setup-gradle@v3
       - name: run ktlintCheck
         run: ./gradlew ktlintCheck
 
@@ -52,6 +65,8 @@ jobs:
         with:
           distribution: 'adopt'
           java-version: 17
+      - name: setup gradle cache
+        uses: gradle/actions/setup-gradle@v3
       - name: run lint
         run: ./gradlew :app:lint
       - name: run detektCheck

Dangerfile

@@ -23,11 +23,3 @@ end
 # CheckstyleFormat
 checkstyle_format.base_path = Dir.pwd
 checkstyle_format.report 'build/reports/detekt/detekt.xml'
-
-# JUnit
-junit_tests_dir = "**/test-results/**/*.xml"
-Dir[junit_tests_dir].each do |file_name|
-  junit.parse file_name
-  junit.show_skipped_tests = true
-  junit.report
-end

build.gradle.kts

@@ -34,6 +34,13 @@ subprojects {
                     sourceCompatibility = JavaVersion.VERSION_17
                     targetCompatibility = JavaVersion.VERSION_17
                 }
+                testOptions {
+                    unitTests {
+                        all {
+                            it.ignoreFailures = true
+                        }
+                    }
+                }
             }
             configure<org.jetbrains.kotlin.gradle.dsl.KotlinAndroidProjectExtension> {
                 jvmToolchain(17)