test(e2e): Enable ESM loader test

Author: Luca Forstner

.github/workflows/build.yml

@@ -1003,7 +1003,7 @@ jobs:
             'create-remix-app-express',
             'create-remix-app-express-vite-dev',
             'debug-id-sourcemaps',
-            # 'esm-loader-node-express-app', # This is currently broken for upstream reasons. See https://github.com/getsentry/sentry-javascript/pull/11338#issuecomment-2025450675
+            'esm-loader-node-express-app',
             'nextjs-app-dir',
             'nextjs-14',
             'react-create-hash-router',

dev-packages/e2e-tests/test-applications/esm-loader-node-express-app/package.json

@@ -2,24 +2,21 @@
   "name": "node-express-app",
   "version": "1.0.0",
   "private": true,
+  "type": "module",
   "scripts": {
-    "start": "node --import=@sentry/node/import-hook src/app.mjs",
+    "start": "node --import=@sentry/node/import src/app.mjs",
     "clean": "npx rimraf node_modules,pnpm-lock.yaml",
     "test:build": "pnpm install",
     "test:assert": "playwright test"
   },
   "dependencies": {
     "@sentry/node": "latest || *",
-    "@sentry/types": "latest || *",
-    "express": "4.19.2",
-    "@types/express": "4.17.17",
-    "@types/node": "18.15.1",
-    "typescript": "4.9.5"
+    "@sentry/opentelemetry": "latest || *",
+    "express": "4.19.2"
   },
   "devDependencies": {
     "@sentry-internal/event-proxy-server": "link:../../../event-proxy-server",
-    "@playwright/test": "^1.27.1",
-    "ts-node": "10.9.1"
+    "@playwright/test": "^1.27.1"
   },
   "volta": {
     "extends": "../../package.json",

dev-packages/e2e-tests/test-applications/esm-loader-node-express-app/playwright.config.ts

@@ -53,7 +53,7 @@ const config: PlaywrightTestConfig = {
   /* Run your local dev server before starting the tests */
   webServer: [
     {
-      command: 'pnpm ts-node-script start-event-proxy.ts',
+      command: 'pnpm ts-node-script start-event-proxy.mjs',
       port: eventProxyPort,
     },
     {

dev-packages/e2e-tests/test-applications/esm-loader-node-express-app/src/app.mjs

@@ -1,13 +1,6 @@
-import * as Sentry from '@sentry/node';
-
-Sentry.init({
-  environment: 'qa', // dynamic sampling bias to keep transactions
-  dsn: process.env.E2E_TEST_DSN,
-  debug: true,
-  tunnel: `http://localhost:3031/`, // proxy server
-  tracesSampleRate: 1,
-});
+// @ts-check
 
+import * as Sentry from '@sentry/node';
 import express from 'express';
 
 const app = express();
@@ -36,7 +29,6 @@ app.get('/test-error', function (req, res) {
 
 Sentry.setupExpressErrorHandler(app);
 
-// @ts-ignore
 app.use(function onError(err, req, res, next) {
   // The error id is attached to `res.sentry` to be returned
   // and optionally displayed to the user for support.

dev-packages/e2e-tests/test-applications/esm-loader-node-express-app/src/instrument.mjs

@@ -0,0 +1,9 @@
+import * as Sentry from '@sentry/node';
+
+Sentry.init({
+  environment: 'qa', // dynamic sampling bias to keep transactions
+  dsn: process.env.E2E_TEST_DSN,
+  debug: true,
+  tunnel: `http://localhost:3031/`, // proxy server
+  tracesSampleRate: 1,
+});

dev-packages/e2e-tests/test-applications/esm-loader-node-express-app/start-event-proxy.ts renamed to dev-packages/e2e-tests/test-applications/esm-loader-node-express-app/start-event-proxy.mjs

@@ -13,7 +13,6 @@ test('Should record exceptions captured inside handlers', async ({ request }) =>
 
 test('Should record a transaction for a parameterless route', async ({ request }) => {
   const transactionEventPromise = waitForTransaction('esm-loader-node-express-app', transactionEvent => {
-    console.log('txn', transactionEvent.transaction);
     return transactionEvent?.transaction === 'GET /test-success';
   });