Show snackbar copy take a screenshot test

.github/dependabot.yml

@@ -12,12 +12,11 @@ updates:
  registries: "*"
  labels: [ "version update" ]
  groups:
- kotlin-ksp-compose:
+ kotlin-ksp:
  patterns:
  - "org.jetbrains.kotlin:*"
  - "org.jetbrains.kotlin.jvm"
  - "com.google.devtools.ksp"
- - "androidx.compose.compiler:compiler"
  open-pull-requests-limit: 10
 registries:
  maven-google:

.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

.gitignore

@@ -43,3 +43,6 @@ _sandbox
 
 # Android Studio captures folder
 captures/
+
+# Kotlin
+.kotlin

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,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
+androidx.annotation:annotation:1.8.0
 androidx.appcompat:appcompat-resources:1.6.1
 androidx.arch.core:core-common:2.2.0
 androidx.arch.core:core-runtime:2.2.0
@@ -12,61 +12,72 @@ 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:material3-android:1.2.1
-androidx.compose.material3:material3:1.2.1
+androidx.compose.animation:animation-android:1.7.0-beta01
+androidx.compose.animation:animation-core-android:1.7.0-beta01
+androidx.compose.animation:animation-core:1.7.0-beta01
+androidx.compose.animation:animation:1.7.0-beta01
+androidx.compose.foundation:foundation-android:1.7.0-beta01
+androidx.compose.foundation:foundation-layout-android:1.7.0-beta01
+androidx.compose.foundation:foundation-layout:1.7.0-beta01
+androidx.compose.foundation:foundation:1.7.0-beta01
+androidx.compose.material3.adaptive:adaptive-android:1.0.0-beta01
+androidx.compose.material3.adaptive:adaptive:1.0.0-beta01
+androidx.compose.material3:material3-adaptive-navigation-suite-android:1.3.0-beta01
+androidx.compose.material3:material3-adaptive-navigation-suite:1.3.0-beta01
+androidx.compose.material3:material3-android:1.3.0-beta01
+androidx.compose.material3:material3:1.3.0-beta01
 androidx.compose.material:material-icons-core-android:1.6.3
 androidx.compose.material:material-icons-core:1.6.3
 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: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.material:material-ripple-android:1.7.0-beta01
+androidx.compose.material:material-ripple:1.7.0-beta01
+androidx.compose.runtime:runtime-android:1.7.0-beta01
+androidx.compose.runtime:runtime-saveable-android:1.7.0-beta01
+androidx.compose.runtime:runtime-saveable:1.7.0-beta01
+androidx.compose.runtime:runtime:1.7.0-beta01
+androidx.compose.ui:ui-android:1.7.0-beta01
+androidx.compose.ui:ui-geometry-android:1.7.0-beta01
+androidx.compose.ui:ui-geometry:1.7.0-beta01
+androidx.compose.ui:ui-graphics-android:1.7.0-beta01
+androidx.compose.ui:ui-graphics:1.7.0-beta01
+androidx.compose.ui:ui-text-android:1.7.0-beta01
+androidx.compose.ui:ui-text:1.7.0-beta01
+androidx.compose.ui:ui-tooling-preview-android:1.7.0-beta01
+androidx.compose.ui:ui-tooling-preview:1.7.0-beta01
+androidx.compose.ui:ui-unit-android:1.7.0-beta01
+androidx.compose.ui:ui-unit:1.7.0-beta01
+androidx.compose.ui:ui-util-android:1.7.0-beta01
+androidx.compose.ui:ui-util:1.7.0-beta01
+androidx.compose.ui:ui:1.7.0-beta01
 androidx.compose:compose-bom:2024.02.02
 androidx.concurrent:concurrent-futures:1.1.0
-androidx.core:core-ktx:1.12.0
-androidx.core:core:1.12.0
+androidx.core:core-ktx:1.13.1
+androidx.core:core:1.13.1
 androidx.customview:customview-poolingcontainer:1.0.0
 androidx.customview:customview:1.0.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.1
 androidx.interpolator:interpolator: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-ktx:2.7.0
-androidx.lifecycle:lifecycle-runtime: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
+androidx.lifecycle:lifecycle-common-jvm:2.8.0
+androidx.lifecycle:lifecycle-common:2.8.0
+androidx.lifecycle:lifecycle-livedata-core-ktx:2.8.0
+androidx.lifecycle:lifecycle-livedata-core:2.8.0
+androidx.lifecycle:lifecycle-livedata:2.8.0
+androidx.lifecycle:lifecycle-process:2.8.0
+androidx.lifecycle:lifecycle-runtime-android:2.8.0
+androidx.lifecycle:lifecycle-runtime-compose-android:2.8.0
+androidx.lifecycle:lifecycle-runtime-compose:2.8.0
+androidx.lifecycle:lifecycle-runtime-ktx-android:2.8.0
+androidx.lifecycle:lifecycle-runtime-ktx:2.8.0
+androidx.lifecycle:lifecycle-runtime:2.8.0
+androidx.lifecycle:lifecycle-viewmodel-android:2.8.0
+androidx.lifecycle:lifecycle-viewmodel-ktx:2.8.0
+androidx.lifecycle:lifecycle-viewmodel-savedstate:2.8.0
+androidx.lifecycle:lifecycle-viewmodel:2.8.0
 androidx.loader:loader:1.0.0
 androidx.metrics:metrics-performance:1.0.0-alpha04
 androidx.profileinstaller:profileinstaller:1.3.1
@@ -79,12 +90,16 @@ androidx.vectordrawable:vectordrawable-animated:1.1.0
 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-beta02
+androidx.window:window-core:1.3.0-beta02
+androidx.window:window:1.3.0-beta02
 com.google.accompanist:accompanist-drawablepainter:0.32.0
 com.google.code.findbugs:jsr305:3.0.2
-com.google.dagger:dagger-lint-aar:2.51
-com.google.dagger:dagger:2.51
-com.google.dagger:hilt-android:2.51
-com.google.dagger:hilt-core:2.51
+com.google.dagger:dagger-lint-aar:2.51.1
+com.google.dagger:dagger:2.51.1
+com.google.dagger:hilt-android:2.51.1
+com.google.dagger:hilt-core:2.51.1
 com.google.guava:listenablefuture:1.0
 com.squareup.okhttp3:okhttp:4.12.0
 com.squareup.okio:okio-jvm:3.8.0
@@ -94,10 +109,10 @@ io.coil-kt:coil-compose-base:2.6.0
 io.coil-kt:coil-compose:2.6.0
 io.coil-kt:coil:2.6.0
 javax.inject:javax.inject:1
-org.jetbrains.kotlin:kotlin-stdlib-common:1.9.22
+org.jetbrains.kotlin:kotlin-stdlib-common:2.0.0
 org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.9.0
 org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.9.0
-org.jetbrains.kotlin:kotlin-stdlib:1.9.22
+org.jetbrains.kotlin:kotlin-stdlib:2.0.0
 org.jetbrains.kotlinx:kotlinx-coroutines-android:1.7.3
 org.jetbrains.kotlinx:kotlinx-coroutines-bom:1.7.3
 org.jetbrains.kotlinx:kotlinx-coroutines-core-jvm:1.7.3

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)
 
@@ -113,9 +113,10 @@ dependencies {
 
  testImplementation(projects.core.dataTest)
  testImplementation(projects.core.testing)
- testImplementation(libs.accompanist.testharness)
+ testImplementation(projects.sync.syncTest)
+ testImplementation(libs.androidx.compose.ui.test)
+ testImplementation(libs.androidx.work.testing)
  testImplementation(libs.hilt.android.testing)
- testImplementation(libs.work.testing)
 
  testDemoImplementation(libs.robolectric)
  testDemoImplementation(libs.roborazzi)
@@ -126,7 +127,7 @@ dependencies {
  androidTestImplementation(projects.core.datastoreTest)
  androidTestImplementation(libs.androidx.test.espresso.core)
  androidTestImplementation(libs.androidx.navigation.testing)
- androidTestImplementation(libs.accompanist.testharness)
+ androidTestImplementation(libs.androidx.compose.ui.test)
  androidTestImplementation(libs.hilt.android.testing)
 
  baselineProfile(projects.benchmarks)