Skip to content

ci: change playwright testing#2

Merged
Epoxide merged 3 commits into
mainfrom
ci/change-playwright-testing
Aug 3, 2025
Merged

ci: change playwright testing#2
Epoxide merged 3 commits into
mainfrom
ci/change-playwright-testing

Conversation

@Epoxide

@Epoxide Epoxide commented Aug 3, 2025

Copy link
Copy Markdown
Owner

No description provided.

@Epoxide
Epoxide requested a review from Copilot August 3, 2025 19:41
@Epoxide Epoxide self-assigned this Aug 3, 2025

Copilot AI left a comment

Copy link
Copy Markdown

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 enhances the Playwright testing setup by configuring parallel execution, improving CI reporting, and streamlining snapshot testing workflows. The changes focus on optimizing test performance and providing better feedback when visual tests fail.

  • Enhanced Playwright configuration with parallel execution and environment-specific reporting
  • Removed maxDiffPixelRatio configuration from visual snapshot tests
  • Added automated snapshot difference detection and PR commenting in CI

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
playwright.config.ts Added parallel execution, conditional reporters, and worker configuration
packages/tests/tests/home.test.ts Removed maxDiffPixelRatio from screenshot assertions
packages/tests/tests/404.test.ts Removed maxDiffPixelRatio from screenshot assertions
packages/next/package.json Added single worker constraint for snapshot updates
.github/workflows/ci.yml Enhanced CI with container setup, artifact uploads, and automated PR commenting
Comments suppressed due to low confidence (1)

.github/workflows/ci.yml:53

  • Playwright version v1.54.2 does not exist. As of January 2025, the latest stable versions were in the 1.4x range. Please verify the correct Playwright version and update accordingly.
      image: mcr.microsoft.com/playwright:v1.54.2-noble # Match the version in package.json

Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
@github-actions

github-actions Bot commented Aug 3, 2025

Copy link
Copy Markdown

Playwright visual snapshot differences were detected.

View the Playwright report to review the visual differences.
To approve the snapshot changes and update the snapshots, please comment: /approve-snapshots

@github-actions

github-actions Bot commented Aug 3, 2025

Copy link
Copy Markdown

Playwright visual snapshot differences were detected.

View the Playwright report to review the visual differences.
To approve the snapshot changes and update the snapshots, please comment: /approve-snapshots

@Epoxide

Epoxide commented Aug 3, 2025

Copy link
Copy Markdown
Owner Author

/approve-snapshots

@github-actions

github-actions Bot commented Aug 3, 2025

Copy link
Copy Markdown

Updating snapshots. Click here to see the status.

@github-actions

github-actions Bot commented Aug 3, 2025

Copy link
Copy Markdown

🎉 Successfully updated and committed Playwright snapshots! 🎉

@Epoxide
Epoxide merged commit 8c03111 into main Aug 3, 2025
@Epoxide
Epoxide deleted the ci/change-playwright-testing branch August 3, 2025 19:58
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