Skip to content

Conversation

@artembelik
Copy link
Contributor

No description provided.

@artembelik artembelik self-assigned this Jan 19, 2026
@github-actions
Copy link

github-actions bot commented Jan 19, 2026

Visit the preview URL for this PR (updated for commit fa85aee):

https://koobiq-next--prs-1296-7tt493mb.web.app

(expires Sun, 25 Jan 2026 10:56:28 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@github-actions
Copy link

🚨 E2E tests failed

Review the report for details.


💡 Comment /approve-snapshots to approve snapshot changes.

@artembelik artembelik changed the title chore(e2e): single spec for light/dark screnshots (#DS-4589) chore(e2e): single test for light & dark screenshots (#DS-4589) Jan 20, 2026
@artembelik
Copy link
Contributor Author

/approve-snapshots

@github-actions
Copy link

🔄 Updating snapshots.

@github-actions
Copy link

✅ Snapshots updated!

@github-actions
Copy link

🚨 E2E tests failed

Review the report for details.


💡 Comment /approve-snapshots to approve snapshot changes.

@artembelik
Copy link
Contributor Author

/approve-snapshots

@github-actions
Copy link

🔄 Updating snapshots.

@github-actions
Copy link

✅ Snapshots updated!

@artembelik artembelik marked this pull request as ready for review January 20, 2026 10:52
Copilot AI review requested due to automatic review settings January 20, 2026 10:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refactors E2E Playwright tests to consolidate light and dark theme screenshot tests into single test cases. Previously, separate test cases existed for light and dark themes. Now, a single test captures both theme screenshots sequentially with explicit filenames ('01-light.png', '01-dark.png', etc.). Additionally, a new utility function e2eWhenStable is introduced to ensure elements are stable before taking screenshots.

Changes:

  • Combined separate light/dark theme tests into single tests that capture both screenshots
  • Added explicit screenshot filenames following a numbered naming convention
  • Introduced e2eWhenStable utility function for element stability checking
  • Renamed tests to use generic names like "states" instead of theme-specific names

Reviewed changes

Copilot reviewed 54 out of 220 changed files in this pull request and generated no comments.

File Description
packages/e2e/utils/when-stable.ts New utility function to wait for element stability and visibility
packages/e2e/utils/index.ts Export the new e2eWhenStable function
packages/components/*/e2e.playwright-spec.ts Refactored tests to combine light/dark theme screenshots in single test cases
packages/components//screenshots/.png New screenshot files with explicit naming convention

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

@artembelik artembelik merged commit 2e18a40 into main Jan 20, 2026
14 checks passed
@artembelik artembelik deleted the chore/DS-4589 branch January 20, 2026 13:39
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.

4 participants