test(e2e): Migrate Express 4 and TSX to span streaming

Co-Authored-By: GPT-6 <codex@openai.com>

Author: msonnb

dev-packages/e2e-tests/test-applications/node-express/src/app.ts

@@ -9,7 +9,6 @@ declare global {
 }
 
 Sentry.init({
-  traceLifecycle: 'static',
   environment: 'qa', // dynamic sampling bias to keep transactions
   dsn: process.env.E2E_TEST_DSN,
   includeLocalVariables: true,

dev-packages/e2e-tests/test-applications/node-express/tests/errors.test.ts

@@ -1,19 +1,20 @@
 import { expect, test } from '@playwright/test';
-import { waitForError, waitForTransaction } from '@sentry-internal/test-utils';
+import { waitForError, collectStreamedSpansUntilSegment } from '@sentry-internal/test-utils';
 
 test('Sends correct error event', async ({ baseURL }) => {
   const errorEventPromise = waitForError('node-express', event => {
     return !event.type && event.exception?.values?.[0]?.value === 'This is an exception with id 123';
   });
 
-  const transactionEventPromise = waitForTransaction('node-express', event => {
-    return event.transaction === 'GET /test-exception/:id';
-  });
+  const segmentEventPromise = collectStreamedSpansUntilSegment('node-express', 'GET /test-exception/:id');
 
   await fetch(`${baseURL}/test-exception/123`);
 
   const errorEvent = await errorEventPromise;
-  const transactionEvent = await transactionEventPromise;
+  const segmentEventSpans = await segmentEventPromise;
+  const segmentEvent = segmentEventSpans.find(
+    segment => segment.is_segment && segment.name === 'GET /test-exception/:id',
+  )!;
 
   expect(errorEvent.exception?.values).toHaveLength(1);
   const exception = errorEvent.exception?.values?.[0];
@@ -38,13 +39,18 @@ test('Sends correct error event', async ({ baseURL }) => {
     parent_span_id: expect.stringMatching(/[a-f0-9]{16}/),
   });
 
-  // The error is attached to the same trace as the request transaction, and to a
-  // span that belongs to that transaction (its root span or one of its children).
-  const transactionTrace = transactionEvent.contexts?.trace;
-  expect(errorEvent.contexts?.trace?.trace_id).toBe(transactionTrace?.trace_id);
+  // The error is attached to the same trace as the request segment, and to a
+  // span in that segment.
+  const segmentTrace = segmentEvent;
+  expect(errorEvent.contexts?.trace?.trace_id).toBe(segmentTrace?.trace_id);
 
-  const transactionSpanIds = [transactionTrace?.span_id, ...(transactionEvent.spans ?? []).map(span => span.span_id)];
-  expect(transactionSpanIds).toContain(errorEvent.contexts?.trace?.span_id);
+  const segmentSpanIds = [
+    segmentTrace?.span_id,
+    ...segmentEventSpans
+      .filter(span => !span.is_segment && span.attributes['sentry.segment.id']?.value === segmentEvent.span_id)
+      .map(span => span.span_id),
+  ];
+  expect(segmentSpanIds).toContain(errorEvent.contexts?.trace?.span_id);
 });
 
 test('Should record caught exceptions with local variable', async ({ baseURL }) => {