add otel test

Author: mydea

dev-packages/node-integration-tests/suites/tracing/tracer-start-active-span-error/instrument.mjs

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

dev-packages/node-integration-tests/suites/tracing/tracer-start-active-span-error/scenario.mjs

@@ -0,0 +1,15 @@
+import * as Sentry from '@sentry/node';
+
+const tracer = Sentry.getClient().tracer;
+
+async function run() {
+  await tracer.startActiveSpan('test span name', async span => {
+    try {
+      throw new Error('Test error from tracer.startActiveSpan');
+    } finally {
+      span.end();
+    }
+  });
+}
+
+run();

dev-packages/node-integration-tests/suites/tracing/tracer-start-active-span-error/test.ts

@@ -0,0 +1,33 @@
+import { afterAll, describe, expect } from 'vitest';
+import { cleanupChildProcesses, createEsmAndCjsTests } from '../../../utils/runner';
+
+describe('tracer.startActiveSpan errors', () => {
+  afterAll(() => {
+    cleanupChildProcesses();
+  });
+
+  createEsmAndCjsTests(__dirname, 'scenario.mjs', 'instrument.mjs', (createRunner, test) => {
+    // When a callback to raw OTel `tracer.startActiveSpan` throws and the error propagates
+    // (uncaught), the span status is NOT automatically marked as errored. The user's `finally`
+    // calls `span.end()` before the error propagates, and an OTel span becomes immutable on end.
+    //
+    // Users who want auto-status-on-error should use `Sentry.startSpan` instead, or follow the
+    // OTel-idiomatic pattern: `span.recordException(err); span.setStatus({ code: ERROR })` in a
+    // `catch` inside the callback.
+    test('does NOT mark span errored when uncaught error escapes raw tracer.startActiveSpan callback (status quo)', async () => {
+      await createRunner()
+        .expect({
+          transaction: {
+            transaction: 'test span name',
+            contexts: {
+              trace: {
+                status: 'ok',
+              },
+            },
+          },
+        })
+        .start()
+        .completed();
+    });
+  });
+});