Skip to content

ci: add browser environment test suite#150

Merged
DecimalTurn merged 9 commits intolatestfrom
dev-browser-test
Apr 9, 2026
Merged

ci: add browser environment test suite#150
DecimalTurn merged 9 commits intolatestfrom
dev-browser-test

Conversation

@DecimalTurn
Copy link
Copy Markdown
Owner

No description provided.

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

Adds a dedicated Jest “browser-like” test run (jsdom) to CI to complement existing Node and ESM integration tests.

Changes:

  • Add jest-environment-jsdom dependency and a new jest.config.browser.mjs.
  • Add test:browser script and run it in CI and local dev.
  • Ignore <rootDir>/worktrees/ in Jest configs to avoid picking up tests from worktree checkouts.

Reviewed changes

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

Show a summary per file
File Description
package.json Adds jest-environment-jsdom and wires up test:browser in scripts/dev workflow.
jest.config.browser.mjs New Jest config to run existing .mjs integration tests under jsdom.
jest.config.mjs Adds testPathIgnorePatterns for worktrees.
jest.config.json Adds testPathIgnorePatterns for worktrees.
.github/workflows/test-and-build.yml Runs the new browser test suite in CI.
pnpm-lock.yaml Lockfile updates for jest-environment-jsdom/jsdom and transitive deps.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

@DecimalTurn DecimalTurn requested a review from Copilot April 9, 2026 23:00
@DecimalTurn DecimalTurn changed the title ci: add browser environment test suite using jest-environment-jsdom ci: add browser environment test suite Apr 9, 2026
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

Copilot reviewed 6 out of 8 changed files in this pull request and generated 3 comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

@DecimalTurn DecimalTurn merged commit 15ad00b into latest Apr 9, 2026
3 checks passed
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