Skip to content

Commit d62b54e

Browse files
committed
fix(e2e): Remove global VITE_SENTRY_SPOTLIGHT from run.ts
Setting VITE_SENTRY_SPOTLIGHT=true globally causes Spotlight to be auto-enabled in all E2E tests, leading to test pollution. Spotlight env vars should only be set for test apps that specifically test Spotlight functionality. This makes local test behavior consistent with CI (where Spotlight env vars are only set for tests containing 'spotlight' in their name).
1 parent bb65b10 commit d62b54e

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

dev-packages/e2e-tests/run.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,8 @@ async function run(): Promise<void> {
174174
PUBLIC_E2E_TEST_DSN: dsn,
175175
REACT_APP_E2E_TEST_DSN: dsn,
176176
VITE_E2E_TEST_DSN: dsn,
177-
VITE_SENTRY_SPOTLIGHT: 'true',
177+
// Note: Spotlight env vars (VITE_SENTRY_SPOTLIGHT, etc.) are NOT set globally.
178+
// They should only be set in test apps that specifically test Spotlight functionality.
178179
E2E_TEST_SENTRY_ORG_SLUG: process.env.E2E_TEST_SENTRY_ORG_SLUG || DEFAULT_SENTRY_ORG_SLUG,
179180
E2E_TEST_SENTRY_PROJECT: process.env.E2E_TEST_SENTRY_PROJECT || DEFAULT_SENTRY_PROJECT,
180181
// Pass workspace root so tests copied to temp dirs can find local packages

0 commit comments

Comments
 (0)