Skip to content

test(F0Card): cover imageAspectRatio with unit test and Snapshot story#3659

Draft
Copilot wants to merge 4 commits intomainfrom
copilot/sub-pr-3652-another-one
Draft

test(F0Card): cover imageAspectRatio with unit test and Snapshot story#3659
Copilot wants to merge 4 commits intomainfrom
copilot/sub-pr-3652-another-one

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 12, 2026

The imageAspectRatio="16/9" prop introduced in the previous PR had no unit test coverage and was missing from the Chromatic Snapshot story.

Changes

  • CardInternal.tsx: Added data-testid="card-image-container" to the image wrapper div to make it queryable in tests.
  • Card.test.tsx: Added test asserting that imageAspectRatio="16/9" applies aspect-video and that none of the fixed-height classes (h-24h-64) are present.
  • Card.stories.tsx: Added WithAspectRatio16x9 to the Snapshot story for visual regression coverage.

📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

Co-authored-by: ekaterina-spiridonova <261384366+ekaterina-spiridonova@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot March 12, 2026 14:21
Copilot AI changed the title [WIP] Add image aspect ratio prop to F0Card test(F0Card): cover imageAspectRatio with unit test and Snapshot story Mar 12, 2026
Base automatically changed from feature/f0-image-16-9 to main March 12, 2026 15:14
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