Switch from Roborazzi to Paparazzi for snapshot testing#767
Open
runningcode wants to merge 18 commits intomainfrom
Open
Switch from Roborazzi to Paparazzi for snapshot testing#767runningcode wants to merge 18 commits intomainfrom
runningcode wants to merge 18 commits intomainfrom
Conversation
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
runningcode
force-pushed
the
no/switch-roborazzi-to-paparazzi
branch
from
5305a6a to
18adf8b
Compare
🛰️ Build DistributionBuild available for installation
🛸 Powered by Emerge Tools |
Sentry Build Distribution
|
📸 Snapshot TestBase build not foundNo build was found for the base commit a79c91d. This is required to generate a snapshot diff for your pull request. It's possible that you created a branch off the base commit before all of the CI steps have finished processing, e.g. the one that uploads a build to our system. If that's the case, no problem! Just wait and this will eventually resolve. 🛸 Powered by Emerge Tools |
runningcode
commented
Mar 19, 2026
| uses: actions/setup-java@v5 | ||
| with: | ||
| java-version: "17" | ||
| java-version: "21" |
Contributor
Author
There was a problem hiding this comment.
paparazzi 2.0.0-alpha4 requires jdk 21.
runningcode
commented
Mar 19, 2026
| distributionBase=GRADLE_USER_HOME | ||
| distributionPath=wrapper/dists | ||
| distributionUrl=https\://services.gradle.org/distributions/gradle-8.14.4-bin.zip | ||
| distributionUrl=https\://services.gradle.org/distributions/gradle-9.1.0-bin.zip |
Contributor
Author
There was a problem hiding this comment.
paparazzi requires Gradle 9.1+
runningcode
force-pushed
the
no/switch-roborazzi-to-paparazzi
branch
from
7057c21 to
57e8120
Compare
runningcode
force-pushed
the
no/switch-roborazzi-to-paparazzi
branch
from
02772aa to
ef8328a
Compare
runningcode
commented
Apr 2, 2026
| getDefaultProguardFile("proguard-android-optimize.txt"), "proguard-rules.pro" | ||
| ) | ||
| signingConfig = signingConfigs.getByName("debug") | ||
| enableUnitTestCoverage = false |
Contributor
Author
There was a problem hiding this comment.
This is just here in order to test a bug where users could disable unit tests for specific variants and the auto-wiring would fail trying to find a non-existent task.
runningcode
commented
Apr 2, 2026
| snapshotsPath.set( | ||
| project.extensions.getByType<RoborazziExtension>().outputDir | ||
| ) | ||
| snapshots { |
Contributor
Author
There was a problem hiding this comment.
Here's our API! Everything else is wired up automatically when the paparazzi plugin is detected.
Replace Roborazzi with Paparazzi for snapshot generation and wire the sentryUploadSnapshotsRelease task to consume Paparazzi's output directory. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Instead of hardcoding the snapshot directory path, use the output directory registered by Paparazzi on testDebugUnitTest. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Instead of fragile name matching on "snapshots", use the registered property name "paparazzi.snapshots.output.dir" to find the output dir. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The Gradle public API doesn't expose task output properties by name. The withPropertyName API is only for up-to-date checking, not lookup. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Upgrade SAGP from 6.3.0-alpha.01 to 6.4.0-alpha.1 - Switch Paparazzi plugin to use version catalog alias - Remove Roborazzi dependencies and plugin references - Add SENTRY_DONT_IGNORE_DEBUG env var to CI snapshot job - Conditionally include debug in ignoredVariants based on env var - Remove manual afterEvaluate snapshot path wiring - Add mavenLocal() to plugin repositories Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ebug Remove manual snapshotsPath wiring and let SAGP resolve it automatically. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Configure the sentryUploadSnapshotsDebug task to resolve the snapshots path from the testDebugUnitTest task outputs, and upgrade SAGP to 6.4.0-alpha.3. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace the manual afterEvaluate task wiring with the new SAGP snapshots DSL block. Also remove the SENTRY_DONT_IGNORE_DEBUG workaround and disable unit test coverage for release builds. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
runningcode
force-pushed
the
no/switch-roborazzi-to-paparazzi
branch
from
c516a51 to
e016669
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replace Roborazzi plugin with Paparazzi for snapshot generation.
This PR also updates to Gradle 9.1 because paparazzi requires it.
🤖 Generated with Claude Code