Skip to content

test: isolate suite from local environment - #217

Open
hxy91819 wants to merge 1 commit into
vincentkoc:mainfrom
hxy91819:feature/test-local-environment-isolation
Open

test: isolate suite from local environment#217
hxy91819 wants to merge 1 commit into
vincentkoc:mainfrom
hxy91819:feature/test-local-environment-isolation

Conversation

@hxy91819

Copy link
Copy Markdown
Contributor

Summary

  • run wrap fixtures with the active Node executable instead of a PATH lookup
  • write child fixture output synchronously so immediate exits cannot drop buffered data
  • point aggregate doctor tests at their temporary project instead of the developer working tree

Root cause

Host tests mutate PATH and aggregate doctor scans project-scoped integrations. Concurrent tests could therefore execute a fake Node binary or detect a real local host configuration such as .kiro, making the suite depend on the machine running it.

Tests

  • env -u TOKENJUICE_NO_OMISSION pnpm exec vitest run test/core/wrap.test.ts test/hosts/claude-code.test.ts
  • pnpm lint
  • pnpm typecheck

Impact

Test-only change; no runtime, documentation, packaging, or release impact.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

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