Skip to content

Add automated playwright tests#113

Open
edmcman wants to merge 3 commits intolydell:mainfrom
edmcman:main
Open

Add automated playwright tests#113
edmcman wants to merge 3 commits intolydell:mainfrom
edmcman:main

Conversation

@edmcman
Copy link
Copy Markdown

@edmcman edmcman commented Dec 31, 2025

Almost all changes are new tests.

The exceptions are:

  1. Small changes to CI to add the new tests
  2. The shadow DOM is left open in the development build to enable the new tests

Copilot AI review requested due to automatic review settings December 31, 2025 13:37
Copy link
Copy Markdown

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 adds comprehensive end-to-end testing infrastructure using Playwright to test the LinkHints browser extension. The tests verify the tutorial workflow by simulating user interactions and checking that hints appear correctly and navigation works as expected.

Key changes:

  • Adds automated Playwright test suite for the tutorial workflow
  • Modifies shadow DOM mode to be "open" in development builds to enable test access
  • Updates CI/CD pipeline to run tests on all branches with a dedicated test job

Reviewed changes

Copilot reviewed 18 out of 29 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tests/tutorial.spec.ts New comprehensive e2e test file covering the tutorial workflow with hint activation, navigation, and interactions
tests/-snapshots/.html HTML snapshot files for visual regression testing of shadow DOM content
tests/-snapshots/.png Screenshot snapshots for visual regression testing
playwright.config.ts Playwright configuration with Firefox project, timeouts, and snapshot settings
src/renderer/Program.ts Changes shadow DOM mode from "closed" to "open" in development for test access
package.json Adds Playwright dependencies and test:playwright script
package-lock.json Lock file updates for new Playwright dependencies
.github/workflows/ci.yml Restructures CI with separate deploy, check, and test jobs; adds Playwright test execution
CONTRIBUTING.md Documents the new automated testing workflow
.prettierignore Excludes test results and snapshot directories from formatting
.gitignore Excludes test results and tool version files
.github/copilot-instructions.md New comprehensive project documentation for AI assistants
.devcontainer/devcontainer.json New devcontainer configuration with Playwright setup

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

Comment thread tests/tutorial.spec.ts
Comment thread tests/tutorial.spec.ts
Comment thread tests/tutorial.spec.ts
Comment thread tests/tutorial.spec.ts
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