Skip to content

Add Google Pay golden request tests - #985

Merged
Max Harrison (maxharrison) merged 1 commit into
max/gpay-omit-billing-address-parameters-v2from
max/gpay-golden-requests
Aug 24, 2026
Merged

Add Google Pay golden request tests#985
Max Harrison (maxharrison) merged 1 commit into
max/gpay-omit-billing-address-parameters-v2from
max/gpay-golden-requests

Conversation

@maxharrison

@maxharrison Max Harrison (maxharrison) commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

The web Google Pay request had no whole-request test, so defaults and configured fields could change without a focused failure. googlePayGoldenRequest.test.ts now compares billing-disabled, billing-enabled, and custom requests with JSON fixtures.

The fixture type derives from buildPaymentRequest, so the projection stays tied to the production return type. A separate test covers web-only merchantId, merchantOrigin, and callbackIntents fields.

The fixtures match the Android SDK copies byte-for-byte, but no CI check enforces that yet.

Stacked on #987.

@maxharrison
Max Harrison (maxharrison) requested a review from a team as a code owner August 20, 2026 15:53
@changeset-bot

changeset-bot Bot commented Aug 20, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: ee22e91

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Comment thread packages/ui-components/test/googlePayGoldenRequest.test.ts

@ana-maksimovskikh ana-maksimovskikh 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.

Thanks Max Harrison (@maxharrison) - the change looks good 🙌

The web Google Pay request had no structural fixture tests, so a default
or a configured field could drift from the Android SDK without a focused
failure.

googlePayGoldenRequest.test.ts compares the full request projection with
billing-disabled, billing-enabled, and custom fixtures.

The fixtures are excluded from Prettier so they stay byte-identical to
the Android copies in evervault/evervault-pay.
@maxharrison
Max Harrison (maxharrison) added this pull request to the merge queue Aug 24, 2026
Merged via the queue into master with commit 03e46ea Aug 24, 2026
15 checks passed
@maxharrison
Max Harrison (maxharrison) deleted the max/gpay-golden-requests branch August 24, 2026 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants