Skip to content

Migrate tests to import from @jest/globals instead of using globals#826

Merged
ecraig12345 merged 2 commits intomainfrom
copilot/update-tests-import-jest-globals
Mar 26, 2026
Merged

Migrate tests to import from @jest/globals instead of using globals#826
ecraig12345 merged 2 commits intomainfrom
copilot/update-tests-import-jest-globals

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 26, 2026

  • Add @jest/globals to root devDependencies and install
  • Remove "jest" from types in root tsconfig.json
  • Add injectGlobals: false to scripts/jest.config.js
  • Update test helper files (getMockScript.ts, getNormalizedSpawnArgs.ts) to import from @jest/globals
  • Update just-task test files (5 files) to import from @jest/globals
  • Update just-scripts test files (8 files) to import from @jest/globals
  • Run build and tests to verify all changes work
  • Code review and security scan
  • Create pull request

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@ecraig12345 ecraig12345 marked this pull request as ready for review March 26, 2026 13:08
@ecraig12345 ecraig12345 merged commit ee6ca22 into main Mar 26, 2026
6 checks passed
@ecraig12345 ecraig12345 deleted the copilot/update-tests-import-jest-globals branch March 26, 2026 13:08
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