Skip to content

Commit 2ea8c45

Browse files
committed
fix(e2e): revert Vite app to use production build with dev SDK exports
Reverted browser-webworker-vite to use 'vite build' + 'vite preview' because the errors.test.ts tests expect production-style filenames (bundled .js files). Added resolve.conditions to get SDK development exports during build, enabling Spotlight auto-enablement while keeping production-like behavior for worker tests.
1 parent c12edfc commit 2ea8c45

File tree

3 files changed

+8
-3
lines changed

3 files changed

+8
-3
lines changed

dev-packages/e2e-tests/test-applications/browser-webworker-vite/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
"build": "rm -rf dist && tsc && vite build",
99
"preview": "vite preview --port 3030",
1010
"test": "playwright test",
11-
"test:build": "pnpm install",
11+
"test:build": "pnpm install && pnpm build",
1212
"test:assert": "pnpm test"
1313
},
1414
"devDependencies": {

dev-packages/e2e-tests/test-applications/browser-webworker-vite/playwright.config.mjs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
import { getPlaywrightConfig } from '@sentry-internal/test-utils';
22

33
const config = getPlaywrightConfig({
4-
// Use vite dev server to test development-mode behavior (e.g., Spotlight auto-enablement)
5-
startCommand: `pnpm dev`,
4+
startCommand: `pnpm preview`,
65
eventProxyFile: 'start-event-proxy.mjs',
76
eventProxyPort: 3031,
87
port: 3030,

dev-packages/e2e-tests/test-applications/browser-webworker-vite/vite.config.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,12 @@ export default defineConfig({
1313
},
1414
},
1515

16+
// Use development exports from @sentry/* packages to include Spotlight auto-enablement code
17+
// The SDK has conditional exports: "development" includes Spotlight, "production" strips it
18+
resolve: {
19+
conditions: ['development', 'module', 'browser', 'import'],
20+
},
21+
1622
plugins: [
1723
sentryVitePlugin({
1824
org: process.env.E2E_TEST_SENTRY_ORG_SLUG,

0 commit comments

Comments
 (0)