add more tests

Author: mydea

dev-packages/node-integration-tests/suites/express/with-http/instrument.mjs

@@ -0,0 +1,9 @@
+import * as Sentry from '@sentry/node';
+import { loggingTransport } from '@sentry-internal/node-integration-tests';
+
+Sentry.init({
+  dsn: 'https://[email protected]/1337',
+  release: '1.0',
+  tracesSampleRate: 1.0,
+  transport: loggingTransport,
+});

dev-packages/node-integration-tests/suites/express/with-http/scenario.mjs

@@ -0,0 +1,28 @@
+import * as Sentry from '@sentry/node';
+import { startExpressServerAndSendPortToRunner } from '@sentry-internal/node-integration-tests';
+import express from 'express';
+import http from 'http';
+
+const app = express();
+
+app.get('/test', (_req, res) => {
+  http.get(`http://localhost:${app.port}/test2`, httpRes => {
+    httpRes.on('data', () => {
+      setTimeout(() => {
+        res.send({ response: 'response 1' });
+      }, 200);
+    });
+  });
+});
+
+app.get('/test2', (_req, res) => {
+  res.send({ response: 'response 2' });
+});
+
+app.get('/test3', (_req, res) => {
+  res.send({ response: 'response 3' });
+});
+
+Sentry.setupExpressErrorHandler(app);
+
+startExpressServerAndSendPortToRunner(app);

dev-packages/node-integration-tests/suites/express/with-http/test.ts

@@ -0,0 +1,41 @@
+import { afterAll, describe, test } from 'vitest';
+import { cleanupChildProcesses, createEsmAndCjsTests } from '../../../utils/runner';
+
+describe('express with http import xxx', () => {
+  afterAll(() => {
+    cleanupChildProcesses();
+  });
+
+  createEsmAndCjsTests(
+    __dirname,
+    'scenario.mjs',
+    'instrument.mjs',
+    createRunner => {
+      test('it works when importing the http module', async () => {
+        const runner = createRunner()
+          .expect({
+            transaction: {
+              transaction: 'GET /test2',
+            },
+          })
+          .expect({
+            transaction: {
+              transaction: 'GET /test',
+            },
+          })
+          .expect({
+            transaction: {
+              transaction: 'GET /test3',
+            },
+          })
+          .start();
+        await runner.makeRequest('get', '/test');
+        await runner.makeRequest('get', '/test3');
+        await runner.completed();
+      });
+      // TODO: This is failing on ESM because importing http is triggering the http spans twice :(
+      // We need to fix this!
+    },
+    { skipEsm: true },
+  );
+});

dev-packages/node-integration-tests/suites/tracing/ai/scenario.js

@@ -1,6 +1,6 @@
 import { join } from 'node:path';
 import { afterAll, describe, expect, test } from 'vitest';
-import { cleanupChildProcesses, createRunner } from '../../../utils/runner';
+import { cleanupChildProcesses, createEsmAndCjsTests } from '../../../utils/runner';
 
 // `ai` SDK only support Node 18+
 describe('ai', () => {