Skip to content

Refresh receiptCard.withQuantity snapshot#5841

Closed
OEvgeny wants to merge 1 commit into
mainfrom
fix/receipt-card-quantity-snapshot
Closed

Refresh receiptCard.withQuantity snapshot#5841
OEvgeny wants to merge 1 commit into
mainfrom
fix/receipt-card-quantity-snapshot

Conversation

@OEvgeny

@OEvgeny OEvgeny commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Fixes #

Changelog Entry

Not required for this PR because it refreshes a test snapshot only.

Description

Refreshes the stored snapshot for __tests__/html2/richCard/receiptCard.withQuantity.html, which is currently failing deterministically on main in CI.

Design

I reproduced the failure locally on a fresh branch from main with the repo's native Docker and Jest workflow. The renderer code path in ReceiptCardContent.tsx did not show a recent logic change, and the failing test passed immediately after updating only the snapshot and rerunning without -u, so this points to a stale baseline snapshot rather than a new code regression.

Specific Changes

  • Updated __tests__/html2/richCard/receiptCard.withQuantity.html.snap-1.png to match current rendering on main.

  • Re-ran the focused HTML test after the snapshot refresh to confirm it now passes.

  • I have added tests and executed them locally

  • I have updated CHANGELOG.md

  • I have updated documentation

Review Checklist

This section is for contributors to review your work.

  • Accessibility reviewed (tab order, content readability, alt text, color contrast)
  • Browser and platform compatibilities reviewed
  • CSS styles reviewed (minimal rules, no z-index)
  • Documents reviewed (docs, samples, live demo)
  • Internationalization reviewed (strings, unit formatting)
  • package.json and package-lock.json reviewed
  • Security reviewed (no data URIs, check for nonce leak)
  • Tests reviewed (coverage, legitimacy)

Copilot AI review requested due to automatic review settings June 9, 2026 17:44

Copilot AI 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.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@OEvgeny OEvgeny enabled auto-merge (squash) June 9, 2026 17:50
@OEvgeny OEvgeny closed this Jun 9, 2026
auto-merge was automatically disabled June 9, 2026 18:25

Pull request was closed

@compulim compulim mentioned this pull request Jun 9, 2026
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants