Skip to content

fix: reduce precision for flaky HeaderDirectHeroImage snapshot#3362

Merged
vegaro merged 1 commit into
mainfrom
cesar/fix-flaky-header-hero-preview
Apr 17, 2026
Merged

fix: reduce precision for flaky HeaderDirectHeroImage snapshot#3362
vegaro merged 1 commit into
mainfrom
cesar/fix-flaky-header-hero-preview

Conversation

@vegaro

@vegaro vegaro commented Apr 17, 2026

Copy link
Copy Markdown
Member

Motivation

The LoadedPaywallComponents_Preview_HeaderDirectHeroImage Emerge snapshot was flaky due to minor differences in one of the components. See the top right image in one of the failures https://www.emergetools.com/snapshot/8508c1a8-c118-47c6-96bf-8560bd509be2?tab=changed

Added @EmergeSnapshotConfig(precision = 0.99f) to the flaky preview to allow a 1% pixel difference tolerance, as we do in other previews.


Note

Low Risk
Low risk: only adjusts Emerge snapshot test configuration for a single Compose preview, with no runtime behavior changes.

Overview
Reduces flakiness in the LoadedPaywallComponents_Preview_HeaderDirectHeroImage Emerge snapshot by adding @EmergeSnapshotConfig(precision = 0.99f) (and its import) to tolerate small pixel diffs during snapshot comparisons.

Reviewed by Cursor Bugbot for commit 755270f. Bugbot is set up for automated code reviews on this repo. Configure here.

Add @EmergeSnapshotConfig(precision = 0.99f) to tolerate minor subpixel
rendering differences in the Android status bar icon when showSystemUi = true.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vegaro vegaro added pr:fix A bug fix pr:RevenueCatUI pr:other and removed pr:fix A bug fix labels Apr 17, 2026
@vegaro vegaro marked this pull request as ready for review April 17, 2026 16:01
@vegaro vegaro requested a review from a team as a code owner April 17, 2026 16:01

@fire-at-will fire-at-will left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@vegaro vegaro enabled auto-merge April 17, 2026 16:13
@vegaro vegaro added this pull request to the merge queue Apr 17, 2026
@codecov

codecov Bot commented Apr 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.36%. Comparing base (27bd0d2) to head (755270f).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3362   +/-   ##
=======================================
  Coverage   79.36%   79.36%           
=======================================
  Files         354      354           
  Lines       14259    14259           
  Branches     1951     1951           
=======================================
  Hits        11316    11316           
  Misses       2140     2140           
  Partials      803      803           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Merged via the queue into main with commit 64bafd5 Apr 17, 2026
37 checks passed
@vegaro vegaro deleted the cesar/fix-flaky-header-hero-preview branch April 17, 2026 16:37
matteinn pushed a commit to matteinn/purchases-android that referenced this pull request Apr 28, 2026
**This is an automatic release.**

## RevenueCat SDK
### 🐞 Bugfixes
* fix: move Google BillingClient connection off the main thread (RevenueCat#3369)
via Toni Rico (@tonidero)
* [EXTERNAL] fix(google): guard showInAppMessages against BillingClient
runtime crashes (RevenueCat#3367) by @matteinn (RevenueCat#3368) via Monika Mateska
(@MonikaMateska)

## RevenueCatUI SDK
### Paywallv2
#### 🐞 Bugfixes
* Add Workflows network layer (RevenueCat#3300) via Cesar de la Vega (@vegaro)

### 🔄 Other Changes
* Fix `revenuecat.useWorkflowsEndpoint` compiler flag (RevenueCat#3374) via Cesar
de la Vega (@vegaro)
* Create paywall from workflow response. Add `USE_WORKFLOWS_ENDPOINT`
BuildConfig (RevenueCat#3350) via Cesar de la Vega (@vegaro)
* Refactor: Remove unnecessary lint suppressions (RevenueCat#3373) via cursor[bot]
(@cursor[bot])
* Bump fastlane-plugin-revenuecat_internal from `a1eed48` to `b822f01`
(RevenueCat#3371) via dependabot[bot] (@dependabot[bot])
* Bump fastlane from 2.232.2 to 2.233.0 (RevenueCat#3370) via dependabot[bot]
(@dependabot[bot])
* Attempt to fix `AssertionError` "ms is denormalized" in
`QueryPurchasesUseCaseTest` (RevenueCat#3361) via Cesar de la Vega (@vegaro)
* Update baseline profiles (RevenueCat#3296) via Jaewoong Eum (@skydoves)
* fix: reduce precision for flaky HeaderDirectHeroImage snapshot (RevenueCat#3362)
via Cesar de la Vega (@vegaro)
* Fix test failures reported twice (RevenueCat#3360) via Cesar de la Vega
(@vegaro)
* refactor: extract `updateStateFromOffering` in `PaywallViewModel`
(RevenueCat#3359) via Cesar de la Vega (@vegaro)
* [Fix] Include parent tabs component_name in tab-control switch
interaction events (RevenueCat#3358) via Monika Mateska (@MonikaMateska)
* Refactor: Remove unnecessary lint suppressions (RevenueCat#3348) via cursor[bot]
(@cursor[bot])
* fix: always upload CI test results even when tests fail (RevenueCat#3357) via
Cesar de la Vega (@vegaro)
* refactor: extract `RevenueCatDialogScaffold` (RevenueCat#3355) via Cesar de la
Vega (@vegaro)
* Fix Slack notifications for nightly integration tests (RevenueCat#3354) via Toni
Rico (@tonidero)
* UI events for paywall component interactions (RevenueCat#3287) via Monika
Mateska (@MonikaMateska)
* Bump fastlane-plugin-revenuecat_internal from `20911d1` to `a1eed48`
(RevenueCat#3351) via dependabot[bot] (@dependabot[bot])

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Primarily a version bump and release automation updates (docs
deploy/redirect and changelog); no functional library code changes
beyond updating embedded version constants.
> 
> **Overview**
> Cuts the `10.2.1` release by updating version references across the
repo (Gradle `VERSION_NAME`, internal `frameworkVersion`, sample/test
app dependency pins, and `.version`).
> 
> Updates the docs release pipeline and website redirect to publish and
point at `10.2.1`, and refreshes `CHANGELOG.md`/`CHANGELOG.latest.md`
with the 10.2.1 release notes.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
a0a325b. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants