Skip to content

test: add playwright suite.#7

Merged
knightedcodemonkey merged 3 commits intomainfrom
bananas
Mar 16, 2026
Merged

test: add playwright suite.#7
knightedcodemonkey merged 3 commits intomainfrom
bananas

Conversation

@knightedcodemonkey
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings March 16, 2026 18:02
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

Adds an end-to-end (E2E) test setup using Playwright for the browser-based playground, along with supporting TypeScript + ESLint configuration and CI automation.

Changes:

  • Introduces a Playwright E2E spec covering key playground behaviors (rendering, mode/style switches, dialogs, persistence).
  • Adds Playwright configuration + CI workflow to run tests (Chromium locally; Chromium + WebKit on CI).
  • Adds TS config/type-check script and an ESLint flat config for Playwright files.

Reviewed changes

Copilot reviewed 6 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tsconfig.json TypeScript config scoped to Playwright tests/config for check-types.
playwright/app.spec.ts New Playwright E2E suite for the playground UI flows.
playwright.config.ts Playwright runner config including webServer boot + CI project matrix.
package.json Adds Playwright/ESLint/TS dev deps and scripts for typecheck/lint/e2e.
package-lock.json Lockfile updates for newly added tooling dependencies.
eslint.config.js Flat ESLint config to lint Playwright tests with eslint-plugin-playwright.
README.md Documents how to run the new E2E suite locally.
.gitignore Ignores Playwright outputs (playwright-report, test-results).
.github/workflows/playwright.yml New GitHub Actions workflow to run Playwright in CI.

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

Copilot AI review requested due to automatic review settings March 16, 2026 18:14
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

Adds a Playwright-based end-to-end test suite for the browser playground, along with TypeScript and ESLint wiring plus CI automation to run the suite.

Changes:

  • Introduces a Playwright E2E spec covering core playground behaviors (rendering, modes, theme/layout, clear actions).
  • Adds Playwright/TypeScript/ESLint tooling and scripts (typecheck, lint, e2e commands) plus Playwright config + web server setup.
  • Adds a dedicated GitHub Actions workflow for Playwright and documents local setup in the README.

Reviewed changes

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

Show a summary per file
File Description
tsconfig.json Adds TS project config for Playwright tests/config typechecking.
playwright/app.spec.ts New Playwright E2E coverage for key UI flows.
playwright.config.ts Configures Playwright projects, baseURL, and local webServer.
eslint.config.js Adds ESLint flat config targeting Playwright files with Playwright plugin rules.
package.json Adds scripts and devDependencies for Playwright, ESLint, and TypeScript.
package-lock.json Locks newly added tooling dependencies.
README.md Documents Playwright browser installation and how to run E2E tests.
.gitignore Ignores Playwright reports/results.
.github/workflows/playwright.yml Adds CI job to run Playwright tests on main/manual.
.github/workflows/deploy.yml Changes deploy trigger behavior (push trigger removed).

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

@knightedcodemonkey knightedcodemonkey merged commit 5283d66 into main Mar 16, 2026
5 checks passed
@knightedcodemonkey knightedcodemonkey deleted the bananas branch March 16, 2026 18:20
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