Skip to content

test: opt the test suite out of telemetry #1668

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ryandemelo wants to merge 1 commit into
base: main
Choose a base branch
from
+10 −0
Open

test: opt the test suite out of telemetry #1668

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions vitest.config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,16 @@ export default defineConfig({
globals: true,
environment: 'node',
globalSetup: './vitest.setup.ts',
// Opt the suite out of telemetry. Many tests spawn the real CLI, which runs
// the preAction hook like any user invocation: it would persist an
// anonymousId into the developer's *real* global config and POST a
// command_executed event per spawn. Workers inherit this env, and so do the
// CLI child processes they spawn. Telemetry's own tests delete these vars
// before asserting, so they are unaffected.
env: {
OPENSPEC_TELEMETRY: '0',
DO_NOT_TRACK: '1',
},
// Tests rely on per-file process isolation (e.g., `process.cwd()` assumptions).
pool: 'forks',
maxWorkers: resolveMaxWorkers(),
Expand Down