Merge remote-tracking branch 'origin/main' into fix/1161-2

Author: SimonMarquis

.github/pull_request_template.md

@@ -1,17 +1,25 @@
-Thanks for submitting a pull request. Please include the following information.
+**DO NOT CREATE A PULL REQUEST WITHOUT READING THESE INSTRUCTIONS**
 
-**What I have done and why**
-Include a summary of what your pull request contains, and why you have made these changes.
+## Instructions
+Thanks for submitting a pull request. To accept your pull request we need you do a few things: 
+
+**If this is your first pull request**
+
+- [Sign the contributors license agreement](https://cla.developers.google.com/)
+
+**Ensure tests pass and code is formatted correctly**
 
-Fixes #<issue_number_goes_here>
+- Run local tests on the `DemoDebug` variant by running `./gradlew testDemoDebug`
+- Fix code formatting: `./gradlew --init-script gradle/init.gradle.kts spotlessApply`
 
-**Do tests pass?**
-- [ ] Run local tests on `DemoDebug` variant: `./gradlew testDemoDebug`
-- [ ] Check formatting: `./gradlew --init-script gradle/init.gradle.kts spotlessApply`
+**Add a description**
 
-**Is this your first pull request?**
-- [ ] [Sign the CLA](https://cla.developers.google.com/)
-- [ ] Run `./tools/setup.sh`
-- [ ] Import the code formatting style as explained in [the setup script](/tools/setup.sh#L40).
+We need to know what you've done and why you've done it. Include a summary of what your pull request contains, and why you have made these changes. Include links to any relevant issues which it fixes.
 
+[Here's an example](https://github.com/android/nowinandroid/pull/1257).
+
+**NOW DELETE THIS LINE AND EVERYTHING ABOVE IT**
+
+**What I have done and why**
 
+\<add your PR description here\> 

.github/workflows/Build.yaml

@@ -17,6 +17,7 @@ jobs:
 
     permissions:
       contents: write
+      pull-requests: write
 
     timeout-minutes: 60
 
@@ -25,7 +26,7 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Validate Gradle Wrapper
-        uses: gradle/wrapper-validation-action@v2
+        uses: gradle/wrapper-validation-action@v3
 
       - name: Copy CI gradle.properties
         run: mkdir -p ~/.gradle ; cp .github/ci-gradle.properties ~/.gradle/gradle.properties
@@ -100,12 +101,13 @@ jobs:
           commit_message: "🤖 Updates screenshots"
 
       # Run local tests after screenshot tests to avoid wrong UP-TO-DATE. TODO: Ignore screenshots.
-      - name: Run local tests
+      - name: Run local tests and create report
         if: always()
         run: ./gradlew testDemoDebug :lint:test
       # Replace task exclusions with `-Pandroidx.baselineprofile.skipgeneration` when
       # https://android-review.googlesource.com/c/platform/frameworks/support/+/2602790 landed in a
       # release build
+
       - name: Build all build type and flavor permutations
         run: ./gradlew :app:assemble :benchmarks:assemble
           -x pixel6Api33ProdNonMinifiedReleaseAndroidTest
@@ -119,13 +121,20 @@ jobs:
           name: APKs
           path: '**/build/outputs/apk/**/*.apk'
 
-      - name: Upload test results (XML)
+      - name: Upload JVM local results (XML)
         if: always()
         uses: actions/upload-artifact@v4
         with:
-          name: test-results
+          name: local-test-results
           path: '**/build/test-results/test*UnitTest/**.xml'
 
+      - name: Upload screenshot results (PNG)
+        if: always()
+        uses: actions/upload-artifact@v4
+        with:
+          name: screenshot-test-results
+          path: '**/build/outputs/roborazzi/*_compare.png'
+
       - name: Check lint
         run: ./gradlew :app:lintProdRelease :app-nia-catalog:lintRelease :lint:lint
 
@@ -180,10 +189,7 @@ jobs:
       - name: Setup Gradle
         uses: gradle/gradle-build-action@v3
 
-      - name: Build projects before running emulator
-        run: ./gradlew packageDemoDebug packageDemoDebugAndroidTest
-
-      - name: Run instrumentation tests
+      - name: Build projects and run instrumentation tests
         uses: reactivecircus/android-emulator-runner@v2
         with:
           api-level: ${{ matrix.api-level }}
@@ -193,9 +199,41 @@ jobs:
           heap-size: 600M
           script: ./gradlew connectedDemoDebugAndroidTest --daemon
 
+      - name: Run local tests (including Roborazzi) for the combined coverage report (only API 30)
+        if: matrix.api-level == 30
+        # There is no need to verify Roborazzi tests to generate coverage.
+        run: ./gradlew testDemoDebugUnitTest -Proborazzi.test.verify=false # Add Prod if we ever add JVM tests for prod
+
+      # Add `createProdDebugUnitTestCoverageReport` if we ever add JVM tests for prod
+      - name: Generate coverage reports for Debug variants (only API 30)
+        if: matrix.api-level == 30
+        run: ./gradlew createDemoDebugCombinedCoverageReport
+        
       - name: Upload test reports
         if: always()
         uses: actions/upload-artifact@v4
         with:
           name: test-reports-${{ matrix.api-level }}
           path: '**/build/reports/androidTests'
+
+      - name: Display local test coverage (only API 30)
+        if: matrix.api-level == 30
+        id: jacoco
+        uses: madrapps/[email protected]
+        with:
+          title: Combined test coverage report
+          min-coverage-overall: 40
+          min-coverage-changed-files: 60
+          paths: |
+            ${{ github.workspace }}/**/build/reports/jacoco/**/*Report.xml
+          token: ${{ secrets.GITHUB_TOKEN }}
+        
+      - name: Upload local coverage reports (XML + HTML) (only API 30)
+        if: matrix.api-level == 30
+        uses: actions/upload-artifact@v4
+        with:
+          name: coverage-reports
+          if-no-files-found: error
+          compression-level: 1
+          overwrite: false
+          path: '**/build/reports/jacoco/'

.github/workflows/Release.yml

@@ -22,7 +22,7 @@ jobs:
         uses: actions/checkout@v4   
 
       - name: Validate Gradle Wrapper
-        uses: gradle/wrapper-validation-action@v2
+        uses: gradle/wrapper-validation-action@v3
 
       - name: Copy CI gradle.properties
         run: mkdir -p ~/.gradle ; cp .github/ci-gradle.properties ~/.gradle/gradle.properties

app-nia-catalog/build.gradle.kts

@@ -59,7 +59,7 @@ android {
             // To publish on the Play store a private signing key is required, but to allow anyone
             // who clones the code to sign and run the release variant, use the debug signing key.
             // TODO: Abstract the signing configuration to a separate file to avoid hardcoding this.
-            signingConfig = signingConfigs.getByName("debug")
+            signingConfig = signingConfigs.named("debug").get()
         }
     }
 }

app-nia-catalog/dependencies/releaseRuntimeClasspath.txt

@@ -1,6 +1,6 @@
-androidx.activity:activity-compose:1.8.0
-androidx.activity:activity-ktx:1.8.0
-androidx.activity:activity:1.8.0
+androidx.activity:activity-compose:1.8.2
+androidx.activity:activity-ktx:1.8.2
+androidx.activity:activity:1.8.2
 androidx.annotation:annotation-experimental:1.4.0
 androidx.annotation:annotation-jvm:1.7.1
 androidx.annotation:annotation:1.7.1

app/build.gradle.kts

@@ -21,7 +21,6 @@ plugins {
     alias(libs.plugins.nowinandroid.android.application.flavors)
     alias(libs.plugins.nowinandroid.android.application.jacoco)
     alias(libs.plugins.nowinandroid.android.hilt)
-    id("jacoco")
     alias(libs.plugins.nowinandroid.android.application.firebase)
     id("com.google.android.gms.oss-licenses-plugin")
     alias(libs.plugins.baselineprofile)
@@ -53,7 +52,7 @@ android {
             // To publish on the Play store a private signing key is required, but to allow anyone
             // who clones the code to sign and run the release variant, use the debug signing key.
             // TODO: Abstract the signing configuration to a separate file to avoid hardcoding this.
-            signingConfig = signingConfigs.getByName("debug")
+            signingConfig = signingConfigs.named("debug").get()
             // Ensure Baseline Profile is fresh for release builds.
             baselineProfile.automaticGenerationDuringBuild = true
         }
@@ -101,6 +100,7 @@ dependencies {
     implementation(libs.androidx.navigation.compose)
     implementation(libs.androidx.profileinstaller)
     implementation(libs.androidx.tracing.ktx)
+    implementation(libs.androidx.window.core)
     implementation(libs.kotlinx.coroutines.guava)
     implementation(libs.coil.kt)
 
@@ -112,9 +112,8 @@ dependencies {
     kspTest(libs.hilt.compiler)
 
     testImplementation(projects.core.dataTest)
-    testImplementation(libs.accompanist.testharness)
     testImplementation(libs.hilt.android.testing)
-    testImplementation(libs.work.testing)
+    testImplementation(projects.sync.syncTest)
 
     testDemoImplementation(libs.robolectric)
     testDemoImplementation(libs.roborazzi)
@@ -126,8 +125,7 @@ dependencies {
     androidTestImplementation(projects.core.datastoreTest)
     androidTestImplementation(libs.androidx.test.espresso.core)
     androidTestImplementation(libs.androidx.navigation.testing)
-    androidTestImplementation(libs.accompanist.testharness)
-    androidTestImplementation(libs.bundles.androidx.compose.ui.test)
+    androidTestImplementation(libs.androidx.compose.ui.test)
     androidTestImplementation(libs.hilt.android.testing)
 
     baselineProfile(projects.benchmarks)

app/dependencies/prodReleaseRuntimeClasspath.txt

@@ -1,9 +1,9 @@
-androidx.activity:activity-compose:1.8.0
-androidx.activity:activity-ktx:1.8.0
-androidx.activity:activity:1.8.0
+androidx.activity:activity-compose:1.8.2
+androidx.activity:activity-ktx:1.8.2
+androidx.activity:activity:1.8.2
 androidx.annotation:annotation-experimental:1.4.0
-androidx.annotation:annotation-jvm:1.7.1
-androidx.annotation:annotation:1.7.1
+androidx.annotation:annotation-jvm:1.8.0-beta01
+androidx.annotation:annotation:1.8.0-beta01
 androidx.appcompat:appcompat-resources:1.6.1
 androidx.appcompat:appcompat:1.6.1
 androidx.arch.core:core-common:2.2.0
@@ -13,20 +13,20 @@ androidx.browser:browser:1.8.0
 androidx.collection:collection-jvm:1.4.0
 androidx.collection:collection-ktx:1.4.0
 androidx.collection:collection:1.4.0
-androidx.compose.animation:animation-android:1.6.3
-androidx.compose.animation:animation-core-android:1.6.3
-androidx.compose.animation:animation-core:1.6.3
-androidx.compose.animation:animation:1.6.3
-androidx.compose.foundation:foundation-android:1.6.3
-androidx.compose.foundation:foundation-layout-android:1.6.3
-androidx.compose.foundation:foundation-layout:1.6.3
-androidx.compose.foundation:foundation:1.6.3
-androidx.compose.material3.adaptive:adaptive-android:1.0.0-alpha08
-androidx.compose.material3.adaptive:adaptive-layout-android:1.0.0-alpha08
-androidx.compose.material3.adaptive:adaptive-layout:1.0.0-alpha08
-androidx.compose.material3.adaptive:adaptive-navigation-android:1.0.0-alpha08
-androidx.compose.material3.adaptive:adaptive-navigation:1.0.0-alpha08
-androidx.compose.material3.adaptive:adaptive:1.0.0-alpha08
+androidx.compose.animation:animation-android:1.7.0-alpha06
+androidx.compose.animation:animation-core-android:1.7.0-alpha06
+androidx.compose.animation:animation-core:1.7.0-alpha06
+androidx.compose.animation:animation:1.7.0-alpha06
+androidx.compose.foundation:foundation-android:1.7.0-alpha06
+androidx.compose.foundation:foundation-layout-android:1.7.0-alpha06
+androidx.compose.foundation:foundation-layout:1.7.0-alpha06
+androidx.compose.foundation:foundation:1.7.0-alpha06
+androidx.compose.material3.adaptive:adaptive-android:1.0.0-alpha10
+androidx.compose.material3.adaptive:adaptive-layout-android:1.0.0-alpha10
+androidx.compose.material3.adaptive:adaptive-layout:1.0.0-alpha10
+androidx.compose.material3.adaptive:adaptive-navigation-android:1.0.0-alpha10
+androidx.compose.material3.adaptive:adaptive-navigation:1.0.0-alpha10
+androidx.compose.material3.adaptive:adaptive:1.0.0-alpha10
 androidx.compose.material3:material3-android:1.2.1
 androidx.compose.material3:material3-window-size-class-android:1.2.1
 androidx.compose.material3:material3-window-size-class:1.2.1
@@ -37,25 +37,25 @@ androidx.compose.material:material-icons-extended-android:1.6.3
 androidx.compose.material:material-icons-extended:1.6.3
 androidx.compose.material:material-ripple-android:1.6.3
 androidx.compose.material:material-ripple:1.6.3
-androidx.compose.runtime:runtime-android:1.6.3
-androidx.compose.runtime:runtime-saveable-android:1.6.3
-androidx.compose.runtime:runtime-saveable:1.6.3
+androidx.compose.runtime:runtime-android:1.7.0-alpha06
+androidx.compose.runtime:runtime-saveable-android:1.7.0-alpha06
+androidx.compose.runtime:runtime-saveable:1.7.0-alpha06
 androidx.compose.runtime:runtime-tracing:1.0.0-beta01
-androidx.compose.runtime:runtime:1.6.3
-androidx.compose.ui:ui-android:1.6.3
-androidx.compose.ui:ui-geometry-android:1.6.3
-androidx.compose.ui:ui-geometry:1.6.3
-androidx.compose.ui:ui-graphics-android:1.6.3
-androidx.compose.ui:ui-graphics:1.6.3
-androidx.compose.ui:ui-text-android:1.6.3
-androidx.compose.ui:ui-text:1.6.3
-androidx.compose.ui:ui-tooling-preview-android:1.6.3
-androidx.compose.ui:ui-tooling-preview:1.6.3
-androidx.compose.ui:ui-unit-android:1.6.3
-androidx.compose.ui:ui-unit:1.6.3
-androidx.compose.ui:ui-util-android:1.6.3
-androidx.compose.ui:ui-util:1.6.3
-androidx.compose.ui:ui:1.6.3
+androidx.compose.runtime:runtime:1.7.0-alpha06
+androidx.compose.ui:ui-android:1.7.0-alpha06
+androidx.compose.ui:ui-geometry-android:1.7.0-alpha06
+androidx.compose.ui:ui-geometry:1.7.0-alpha06
+androidx.compose.ui:ui-graphics-android:1.7.0-alpha06
+androidx.compose.ui:ui-graphics:1.7.0-alpha06
+androidx.compose.ui:ui-text-android:1.7.0-alpha06
+androidx.compose.ui:ui-text:1.7.0-alpha06
+androidx.compose.ui:ui-tooling-preview-android:1.7.0-alpha06
+androidx.compose.ui:ui-tooling-preview:1.7.0-alpha06
+androidx.compose.ui:ui-unit-android:1.7.0-alpha06
+androidx.compose.ui:ui-unit:1.7.0-alpha06
+androidx.compose.ui:ui-util-android:1.7.0-alpha06
+androidx.compose.ui:ui-util:1.7.0-alpha06
+androidx.compose.ui:ui:1.7.0-alpha06
 androidx.compose:compose-bom:2024.02.02
 androidx.concurrent:concurrent-futures:1.1.0
 androidx.core:core-ktx:1.12.0
@@ -74,34 +74,40 @@ androidx.emoji2:emoji2-views-helper:1.3.0
 androidx.emoji2:emoji2:1.3.0
 androidx.exifinterface:exifinterface:1.3.7
 androidx.fragment:fragment:1.5.1
+androidx.graphics:graphics-path:1.0.0-beta02
 androidx.hilt:hilt-common:1.1.0
 androidx.hilt:hilt-navigation-compose:1.2.0
 androidx.hilt:hilt-navigation:1.2.0
 androidx.hilt:hilt-work:1.1.0
 androidx.interpolator:interpolator:1.0.0
 androidx.legacy:legacy-support-core-utils:1.0.0
-androidx.lifecycle:lifecycle-common-java8:2.7.0
-androidx.lifecycle:lifecycle-common:2.7.0
-androidx.lifecycle:lifecycle-livedata-core-ktx:2.7.0
-androidx.lifecycle:lifecycle-livedata-core:2.7.0
-androidx.lifecycle:lifecycle-livedata:2.7.0
-androidx.lifecycle:lifecycle-process:2.7.0
-androidx.lifecycle:lifecycle-runtime-compose:2.7.0
-androidx.lifecycle:lifecycle-runtime-ktx:2.7.0
-androidx.lifecycle:lifecycle-runtime:2.7.0
-androidx.lifecycle:lifecycle-service:2.7.0
-androidx.lifecycle:lifecycle-viewmodel-compose:2.7.0
-androidx.lifecycle:lifecycle-viewmodel-ktx:2.7.0
-androidx.lifecycle:lifecycle-viewmodel-savedstate:2.7.0
-androidx.lifecycle:lifecycle-viewmodel:2.7.0
+androidx.lifecycle:lifecycle-common-java8:2.8.0-alpha04
+androidx.lifecycle:lifecycle-common-jvm:2.8.0-alpha04
+androidx.lifecycle:lifecycle-common:2.8.0-alpha04
+androidx.lifecycle:lifecycle-livedata-core-ktx:2.8.0-alpha04
+androidx.lifecycle:lifecycle-livedata-core:2.8.0-alpha04
+androidx.lifecycle:lifecycle-livedata:2.8.0-alpha04
+androidx.lifecycle:lifecycle-process:2.8.0-alpha04
+androidx.lifecycle:lifecycle-runtime-android:2.8.0-alpha04
+androidx.lifecycle:lifecycle-runtime-compose:2.8.0-alpha04
+androidx.lifecycle:lifecycle-runtime-ktx-android:2.8.0-alpha04
+androidx.lifecycle:lifecycle-runtime-ktx:2.8.0-alpha04
+androidx.lifecycle:lifecycle-runtime:2.8.0-alpha04
+androidx.lifecycle:lifecycle-service:2.8.0-alpha04
+androidx.lifecycle:lifecycle-viewmodel-android:2.8.0-alpha04
+androidx.lifecycle:lifecycle-viewmodel-compose-android:2.8.0-alpha04
+androidx.lifecycle:lifecycle-viewmodel-compose:2.8.0-alpha04
+androidx.lifecycle:lifecycle-viewmodel-ktx:2.8.0-alpha04
+androidx.lifecycle:lifecycle-viewmodel-savedstate:2.8.0-alpha04
+androidx.lifecycle:lifecycle-viewmodel:2.8.0-alpha04
 androidx.loader:loader:1.0.0
 androidx.localbroadcastmanager:localbroadcastmanager:1.0.0
 androidx.metrics:metrics-performance:1.0.0-alpha04
-androidx.navigation:navigation-common-ktx:2.7.4
-androidx.navigation:navigation-common:2.7.4
-androidx.navigation:navigation-compose:2.7.4
-androidx.navigation:navigation-runtime-ktx:2.7.4
-androidx.navigation:navigation-runtime:2.7.4
+androidx.navigation:navigation-common-ktx:2.8.0-alpha06
+androidx.navigation:navigation-common:2.8.0-alpha06
+androidx.navigation:navigation-compose:2.8.0-alpha06
+androidx.navigation:navigation-runtime-ktx:2.8.0-alpha06
+androidx.navigation:navigation-runtime:2.8.0-alpha06
 androidx.print:print:1.0.0
 androidx.privacysandbox.ads:ads-adservices-java:1.0.0-beta05
 androidx.privacysandbox.ads:ads-adservices:1.0.0-beta05
@@ -123,9 +129,9 @@ androidx.vectordrawable:vectordrawable:1.1.0
 androidx.versionedparcelable:versionedparcelable:1.1.1
 androidx.viewpager:viewpager:1.0.0
 androidx.window.extensions.core:core:1.0.0
-androidx.window:window-core-android:1.3.0-alpha02
-androidx.window:window-core:1.3.0-alpha02
-androidx.window:window:1.3.0-alpha02
+androidx.window:window-core-android:1.3.0-alpha03
+androidx.window:window-core:1.3.0-alpha03
+androidx.window:window:1.3.0-alpha03
 androidx.work:work-runtime-ktx:2.9.0
 androidx.work:work-runtime:2.9.0
 com.caverock:androidsvg-aar:1.4
@@ -182,8 +188,8 @@ com.google.guava:failureaccess:1.0.1
 com.google.guava:guava:31.1-android
 com.google.guava:listenablefuture:9999.0-empty-to-avoid-conflict-with-guava
 com.google.j2objc:j2objc-annotations:1.3
-com.google.protobuf:protobuf-javalite:3.25.2
-com.google.protobuf:protobuf-kotlin-lite:3.25.2
+com.google.protobuf:protobuf-javalite:4.26.0
+com.google.protobuf:protobuf-kotlin-lite:4.26.0
 com.jakewharton.retrofit:retrofit2-kotlinx-serialization-converter:1.0.0
 com.squareup.okhttp3:logging-interceptor:4.12.0
 com.squareup.okhttp3:okhttp:4.12.0