clean up e2e

getsentry · Dec 12, 2024 · fb2fbd3 · fb2fbd3
1 parent fd618e9
commit fb2fbd3
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 16 deletions.
diff --git a/...est-applications/node-profiling/build.mjs → ...applications/node-profiling/build-cjs.mjs b/...est-applications/node-profiling/build.mjs → ...applications/node-profiling/build-cjs.mjs
@@ -11,7 +11,7 @@ console.log('Running build using esbuild version', esbuild.version);
 esbuild.buildSync({
   platform: 'node',
   entryPoints: ['./index.ts'],
-  outdir: './dist',
+  outfile: './dist/cjs/index.js',
   target: 'esnext',
   format: 'cjs',
   bundle: true,

diff --git a/...ications/node-profiling/build.shimmed.mjs → ...applications/node-profiling/build-esm.mjs b/...ications/node-profiling/build.shimmed.mjs → ...applications/node-profiling/build-esm.mjs
@@ -11,19 +11,9 @@ console.log('Running build using esbuild version', esbuild.version);
 esbuild.buildSync({
   platform: 'node',
   entryPoints: ['./index.ts'],
-  outfile: './dist/index.shimmed.mjs',
+  outfile: './dist/esm/index.mjs',
   target: 'esnext',
   format: 'esm',
   bundle: true,
   loader: { '.node': 'copy' },
-  banner: {
-    js: `
-        import { dirname } from 'node:path';
-        import { fileURLToPath } from 'node:url';
-        import { createRequire } from 'node:module';
-        const require = createRequire(import.meta.url);
-        const __filename = fileURLToPath(import.meta.url);
-        const __dirname = dirname(__filename);
-        `,
-  },
 });
diff --git a/dev-packages/e2e-tests/test-applications/node-profiling/index.mjs b/dev-packages/e2e-tests/test-applications/node-profiling/index.mjs
@@ -8,7 +8,7 @@ console.log('🧵 Starting ESM test');
 
 const wait = ms => new Promise(resolve => setTimeout(resolve, ms));
 
-function assertUnpatechedRequire() {
+function assertUnpatchedRequire() {
   if (typeof require !== 'undefined') {
     // Test that globalThis.require is not defined by any side effects of the profiling
     // https://github.com/getsentry/sentry-javascript/issues/13662
@@ -30,5 +30,5 @@ Sentry.startSpan({ name: 'Precompile test' }, async () => {
   await wait(500);
 });
 
-assertUnpatechedRequire();
+assertUnpatchedRequire();
 console.log('✅ Require is not patched');
diff --git a/dev-packages/e2e-tests/test-applications/node-profiling/package.json b/dev-packages/e2e-tests/test-applications/node-profiling/package.json
@@ -4,8 +4,8 @@
   "private": true,
   "scripts": {
     "typecheck": "tsc --noEmit",
-    "build": "node build.mjs && node build.shimmed.mjs",
-    "test": "node dist/index.js && node --experimental-require-module dist/index.js && node dist/index.shimmed.mjs",
+    "build": "node build-cjs.mjs && node build-esm.mjs",
+    "test": "node dist/index.js && node --experimental-require-module dist/cjs/index.js && node dist/esm/index.mjs",
     "clean": "npx rimraf node_modules dist",
     "test:electron": "$(pnpm bin)/electron-rebuild && playwright test",
     "test:build": "pnpm run typecheck && pnpm run build",