test(node): Migrate tracing integration tests to createEsmAndCjsTests (#20961)

## Summary

Migrates the remaining integration-specific tracing tests under
`dev-packages/node-integration-tests/suites/tracing/` to use the
`createEsmAndCjsTests` helper, which auto-runs each scenario in both ESM
and CJS mode from a single `.mjs` source.

For each folder:
- `scenario*.js` → `scenario*.mjs` (ESM imports)
- Sentry.init extracted into a separate `instrument.mjs`
- `test.ts` switched to `createEsmAndCjsTests(__dirname, scenarioPath,
instrumentPath, callback, options?)`
- `failsOnEsm: true` added only where CJS passed and ESM legitimately
fails (i.e. instrumentation only patches the CJS module shape)

## Migrated folders

| Folder | Notes |
|---|---|
| `lru-memoizer` | `failsOnEsm: true` — OTel
`instrumentation-lru-memoizer` only patches the CJS function-shaped
export. |
| `mongodb` | mongodb v3 is CJS-only and has no named ESM exports — uses
`import mongodb from 'mongodb'` + destructure. |
| `mongoose` | Straight migration. |
| `mysql2` | Straight migration (docker). |
| `mysql` | `failsOnEsm: true` on all 3 scenarios — mysql v2 is
CJS-only. Three scenarios kept (`withConnect`, `withoutCallback`,
`withoutConnect`). |
| `postgres` | 3 scenarios. `ignoreConnectSpans` reuses the default
scenario but with a separate `instrument-ignoreConnect.mjs`. `pg-native`
scenario migrated as-is (still relies on local `setupCommand: 'yarn'` to
build the native bindings). |
| `postgresjs` | Consolidated previously-parallel `.cjs`/`.mjs`
scenarios into a single `.mjs` each. The `wait-for-postgres.js` helper
is still required from the scenarios via
`createRequire(import.meta.url)` (revert of the .mjs-helper conversion
approach — see notes below). The error stacktrace assertion uses
`expect.stringMatching(/postgres(\.cjs)?\.src:connection/)` so the same
expectation matches both modes. |
| `redis` | Straight migration (docker). |
| `redis-cache` | 2 scenarios (`ioredis`, `redis-4`). Each scenario has
its own `instrument-*.mjs` because they pass different `cachePrefixes`
to `redisIntegration`. |
| `redis-dc` | Uses `await import('redis-5')` inside `run()` to preserve
the comment-documented ordering where the DC subscriber must be
registered (via `Promise.resolve().then`) before node-redis eagerly
creates its native TracingChannels on require/import. |
| `tedious` | Migrated but kept `describe.skip` (the test was previously
skipped as flaky in #15798). Manually verified both modes pass when
un-skipped. |
| `apollo-graphql` | 3 scenarios (query, mutation, error). Shared
`apollo-server.mjs` helper copied into the tmp dir via `copyPaths:
['apollo-server.mjs']` and loaded with `await
import('./apollo-server.mjs')` from each scenario. |
| `apollo-graphql/useOperationNameForRootSpan` | 6 scenarios. Reuses the
parent `apollo-server.mjs` via `../../apollo-server.mjs` (the scenarios
run from `useOperationNameForRootSpan/tmp_xxx/`, so this resolves back
to the apollo-graphql folder). Scenarios use `Sentry.getClient().tracer`
instead of capturing the client returned by `Sentry.init` (since init
now lives in `instrument.mjs`). |

## Out of scope

The following folders under `tracing/` are still using `createRunner`
directly — they are not integrations for a specific package but general
tracing-behavior tests, so they're intentionally left alone:

- `meta-tags*`, `meta-tags-twp*`
- `sample-rate-propagation/*`, `sample-rand-propagation`
- `tracePropagationTargets/*`
- `traceid-recycling*`
- `dsc-txn-name-update`
- `http-client-spans/*`
- `maxSpans`, `envelope-header/*`, `linking`

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: mydea

…/tracing/apollo-graphql/apollo-server.js …tracing/apollo-graphql/apollo-server.mjsdev-packages/node-integration-tests/suites/tracing/apollo-graphql/apollo-server.js renamed to dev-packages/node-integration-tests/suites/tracing/apollo-graphql/apollo-server.mjs dev-packages/node-integration-tests/suites/tracing/apollo-graphql/apollo-server.js renamed to dev-packages/node-integration-tests/suites/tracing/apollo-graphql/apollo-server.mjs

@@ -1,8 +1,8 @@
-const { ApolloServer } = require('@apollo/server');
-const gql = require('graphql-tag');
-const Sentry = require('@sentry/node');
+import { ApolloServer } from '@apollo/server';
+import * as Sentry from '@sentry/node';
+import gql from 'graphql-tag';
 
-module.exports = () => {
+export function createApolloServer() {
   return Sentry.startSpan({ name: 'Test Server Start' }, () => {
     return new ApolloServer({
       typeDefs: gql`
@@ -32,4 +32,4 @@ module.exports = () => {
       introspection: false,
     });
   });
-};
+}

…suites/tracing/postgresjs/instrument.cjs …es/tracing/apollo-graphql/instrument.mjsdev-packages/node-integration-tests/suites/tracing/postgresjs/instrument.cjs renamed to dev-packages/node-integration-tests/suites/tracing/apollo-graphql/instrument.mjs dev-packages/node-integration-tests/suites/tracing/postgresjs/instrument.cjs renamed to dev-packages/node-integration-tests/suites/tracing/apollo-graphql/instrument.mjs

@@ -1,5 +1,5 @@
-const Sentry = require('@sentry/node');
-const { loggingTransport } = require('@sentry-internal/node-integration-tests');
+import * as Sentry from '@sentry/node';
+import { loggingTransport } from '@sentry-internal/node-integration-tests';
 
 Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',

…tracing/apollo-graphql/scenario-error.js …racing/apollo-graphql/scenario-error.mjsdev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-error.js renamed to dev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-error.mjs dev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-error.js renamed to dev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-error.mjs

@@ -1,16 +1,9 @@
-const Sentry = require('@sentry/node');
-const { loggingTransport } = require('@sentry-internal/node-integration-tests');
-
-Sentry.init({
-  dsn: 'https://public@dsn.ingest.sentry.io/1337',
-  release: '1.0',
-  tracesSampleRate: 1.0,
-  transport: loggingTransport,
-});
+import * as Sentry from '@sentry/node';
+import gql from 'graphql-tag';
 
 async function run() {
-  const gql = require('graphql-tag');
-  const server = require('./apollo-server')();
+  const { createApolloServer } = await import('./apollo-server.mjs');
+  const server = createApolloServer();
 
   await Sentry.startSpan(
     {

…cing/apollo-graphql/scenario-mutation.js …ing/apollo-graphql/scenario-mutation.mjsdev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-mutation.js renamed to dev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-mutation.mjs dev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-mutation.js renamed to dev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-mutation.mjs

@@ -1,16 +1,9 @@
-const Sentry = require('@sentry/node');
-const { loggingTransport } = require('@sentry-internal/node-integration-tests');
-
-Sentry.init({
-  dsn: 'https://public@dsn.ingest.sentry.io/1337',
-  release: '1.0',
-  tracesSampleRate: 1.0,
-  transport: loggingTransport,
-});
+import * as Sentry from '@sentry/node';
+import gql from 'graphql-tag';
 
 async function run() {
-  const gql = require('graphql-tag');
-  const server = require('./apollo-server')();
+  const { createApolloServer } = await import('./apollo-server.mjs');
+  const server = createApolloServer();
 
   await Sentry.startSpan(
     {

…tracing/apollo-graphql/scenario-query.js …racing/apollo-graphql/scenario-query.mjsdev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-query.js renamed to dev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-query.mjs dev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-query.js renamed to dev-packages/node-integration-tests/suites/tracing/apollo-graphql/scenario-query.mjs

@@ -1,15 +1,8 @@
-const Sentry = require('@sentry/node');
-const { loggingTransport } = require('@sentry-internal/node-integration-tests');
-
-Sentry.init({
-  dsn: 'https://public@dsn.ingest.sentry.io/1337',
-  release: '1.0',
-  tracesSampleRate: 1.0,
-  transport: loggingTransport,
-});
+import * as Sentry from '@sentry/node';
 
 async function run() {
-  const server = require('./apollo-server')();
+  const { createApolloServer } = await import('./apollo-server.mjs');
+  const server = createApolloServer();
 
   await Sentry.startSpan(
     {

dev-packages/node-integration-tests/suites/tracing/apollo-graphql/test.ts

@@ -1,13 +1,17 @@
-import { describe, expect, test } from 'vitest';
-import { createRunner } from '../../../utils/runner';
+import { afterAll, describe, expect } from 'vitest';
+import { cleanupChildProcesses, createEsmAndCjsTests } from '../../../utils/runner';
 
 // Server start transaction (Apollo Server v5 no longer runs introspection query on start)
 const EXPECTED_START_SERVER_TRANSACTION = {
   transaction: 'Test Server Start',
 };
 
 describe('GraphQL/Apollo Tests', () => {
-  test('should instrument GraphQL queries used from Apollo Server.', async () => {
+  afterAll(() => {
+    cleanupChildProcesses();
+  });
+
+  describe('query', () => {
     const EXPECTED_TRANSACTION = {
       transaction: 'Test Transaction (query)',
       spans: expect.arrayContaining([
@@ -24,14 +28,24 @@ describe('GraphQL/Apollo Tests', () => {
       ]),
     };
 
-    await createRunner(__dirname, 'scenario-query.js')
-      .expect({ transaction: EXPECTED_START_SERVER_TRANSACTION })
-      .expect({ transaction: EXPECTED_TRANSACTION })
-      .start()
-      .completed();
+    createEsmAndCjsTests(
+      __dirname,
+      'scenario-query.mjs',
+      'instrument.mjs',
+      (createTestRunner, test) => {
+        test('should instrument GraphQL queries used from Apollo Server.', async () => {
+          await createTestRunner()
+            .expect({ transaction: EXPECTED_START_SERVER_TRANSACTION })
+            .expect({ transaction: EXPECTED_TRANSACTION })
+            .start()
+            .completed();
+        });
+      },
+      { copyPaths: ['apollo-server.mjs'] },
+    );
   });
 
-  test('should instrument GraphQL mutations used from Apollo Server.', async () => {
+  describe('mutation', () => {
     const EXPECTED_TRANSACTION = {
       transaction: 'Test Transaction (mutation Mutation)',
       spans: expect.arrayContaining([
@@ -49,14 +63,24 @@ describe('GraphQL/Apollo Tests', () => {
       ]),
     };
 
-    await createRunner(__dirname, 'scenario-mutation.js')
-      .expect({ transaction: EXPECTED_START_SERVER_TRANSACTION })
-      .expect({ transaction: EXPECTED_TRANSACTION })
-      .start()
-      .completed();
+    createEsmAndCjsTests(
+      __dirname,
+      'scenario-mutation.mjs',
+      'instrument.mjs',
+      (createTestRunner, test) => {
+        test('should instrument GraphQL mutations used from Apollo Server.', async () => {
+          await createTestRunner()
+            .expect({ transaction: EXPECTED_START_SERVER_TRANSACTION })
+            .expect({ transaction: EXPECTED_TRANSACTION })
+            .start()
+            .completed();
+        });
+      },
+      { copyPaths: ['apollo-server.mjs'] },
+    );
   });
 
-  test('should handle GraphQL errors.', async () => {
+  describe('error', () => {
     const EXPECTED_TRANSACTION = {
       transaction: 'Test Transaction (mutation Mutation)',
       spans: expect.arrayContaining([
@@ -74,10 +98,20 @@ describe('GraphQL/Apollo Tests', () => {
       ]),
     };
 
-    await createRunner(__dirname, 'scenario-error.js')
-      .expect({ transaction: EXPECTED_START_SERVER_TRANSACTION })
-      .expect({ transaction: EXPECTED_TRANSACTION })
-      .start()
-      .completed();
+    createEsmAndCjsTests(
+      __dirname,
+      'scenario-error.mjs',
+      'instrument.mjs',
+      (createTestRunner, test) => {
+        test('should handle GraphQL errors.', async () => {
+          await createTestRunner()
+            .expect({ transaction: EXPECTED_START_SERVER_TRANSACTION })
+            .expect({ transaction: EXPECTED_TRANSACTION })
+            .start()
+            .completed();
+        });
+      },
+      { copyPaths: ['apollo-server.mjs'] },
+    );
   });
 });

dev-packages/node-integration-tests/suites/tracing/apollo-graphql/useOperationNameForRootSpan/instrument.mjs

@@ -0,0 +1,10 @@
+import * as Sentry from '@sentry/node';
+import { loggingTransport } from '@sentry-internal/node-integration-tests';
+
+Sentry.init({
+  dsn: 'https://public@dsn.ingest.sentry.io/1337',
+  release: '1.0',
+  tracesSampleRate: 1.0,
+  integrations: [Sentry.graphqlIntegration({ useOperationNameForRootSpan: true })],
+  transport: loggingTransport,
+});

dev-packages/node-integration-tests/suites/tracing/apollo-graphql/useOperationNameForRootSpan/scenario-invalid-root-span.js

@@ -0,0 +1,22 @@
+import * as Sentry from '@sentry/node';
+
+const tracer = Sentry.getClient().tracer;
+
+async function run() {
+  const { createApolloServer } = await import('../../apollo-server.mjs');
+  const server = createApolloServer();
+
+  await tracer.startActiveSpan('test span name', async span => {
+    // Ref: https://www.apollographql.com/docs/apollo-server/testing/testing/#testing-using-executeoperation
+    await server.executeOperation({
+      query: 'query GetHello {hello}',
+    });
+
+    setTimeout(() => {
+      span.end();
+      server.stop();
+    }, 500);
+  });
+}
+
+run();

dev-packages/node-integration-tests/suites/tracing/apollo-graphql/useOperationNameForRootSpan/scenario-invalid-root-span.mjs

@@ -1,18 +1,10 @@
-const Sentry = require('@sentry/node');
-const { loggingTransport } = require('@sentry-internal/node-integration-tests');
+import * as Sentry from '@sentry/node';
 
-const client = Sentry.init({
-  dsn: 'https://public@dsn.ingest.sentry.io/1337',
-  release: '1.0',
-  tracesSampleRate: 1.0,
-  integrations: [Sentry.graphqlIntegration({ useOperationNameForRootSpan: true })],
-  transport: loggingTransport,
-});
-
-const tracer = client.tracer;
+const tracer = Sentry.getClient().tracer;
 
 async function run() {
-  const server = require('../apollo-server')();
+  const { createApolloServer } = await import('../../apollo-server.mjs');
+  const server = createApolloServer();
 
   await tracer.startActiveSpan(
     'test span name',