add tests for trace context data

Author: s1gr1d

dev-packages/browser-integration-tests/suites/profiling/legacyMode/test.ts

@@ -36,7 +36,9 @@ sentryTest('sends profile envelope in legacy mode', async ({ page, getLocalTestU
   const url = await getLocalTestUrl({ testDir: __dirname, responseHeaders: { 'Document-Policy': 'js-profiling' } });
 
   const req = await waitForTransactionRequestOnUrl(page, url);
+  const transactionEvent = properEnvelopeRequestParser<Event>(req, 0);
   const profileEvent = properEnvelopeRequestParser<Profile>(req, 1);
+
   expect(profileEvent).toBeDefined();
 
   const profile = profileEvent.profile;
@@ -54,4 +56,8 @@ sentryTest('sends profile envelope in legacy mode', async ({ page, getLocalTestU
     minSampleDurationMs: 20,
     isChunkFormat: false,
   });
+
+  // contexts.trace.data must include thread.id to identify which thread is associated with the transaction
+  expect(transactionEvent?.contexts?.trace?.data?.['thread.id']).toBe('0');
+  expect(transactionEvent?.contexts?.trace?.data?.['thread.name']).toBe('main');
 });

dev-packages/browser-integration-tests/suites/profiling/traceLifecycleMode_multiple-chunks/test.ts

@@ -1,11 +1,13 @@
 import { expect } from '@playwright/test';
-import type { ProfileChunkEnvelope } from '@sentry/core';
+import type { Event, ProfileChunkEnvelope } from '@sentry/core';
 import { sentryTest } from '../../../utils/fixtures';
 import {
   countEnvelopes,
   getMultipleSentryEnvelopeRequests,
+  properEnvelopeRequestParser,
   properFullEnvelopeRequestParser,
   shouldSkipTracingTest,
+  waitForTransactionRequestOnUrl,
 } from '../../../utils/helpers';
 import { validateProfile, validateProfilePayloadMetadata } from '../test-utils';
 
@@ -96,3 +98,25 @@ sentryTest(
     });
   },
 );
+
+sentryTest(
+  'attaches profiler_id and thread data to transaction events (trace mode)',
+  async ({ page, getLocalTestUrl, browserName }) => {
+    if (shouldSkipTracingTest() || browserName !== 'chromium') {
+      sentryTest.skip();
+    }
+
+    const url = await getLocalTestUrl({ testDir: __dirname, responseHeaders: { 'Document-Policy': 'js-profiling' } });
+
+    const req = await waitForTransactionRequestOnUrl(page, url);
+    const transactionEvent = properEnvelopeRequestParser<Event>(req, 0) as any;
+
+    expect(transactionEvent?.type).toBe('transaction');
+
+    expect(transactionEvent?.contexts?.profile?.profiler_id).toMatch(/^[a-f\d]{32}$/);
+
+    // contexts.trace.data must include thread.id to identify which thread is associated with the transaction
+    expect(transactionEvent?.contexts?.trace?.data?.['thread.id']).toBe('0');
+    expect(transactionEvent?.contexts?.trace?.data?.['thread.name']).toBe('main');
+  },
+);

dev-packages/browser-integration-tests/suites/profiling/traceLifecycleMode_overlapping-spans/test.ts

@@ -89,9 +89,12 @@ sentryTest('attaches thread data to child spans (trace mode)', async ({ page, ge
 
   const profilerId = rootSpan?.contexts?.profile?.profiler_id as string | undefined;
   expect(typeof profilerId).toBe('string');
-
   expect(profilerId).toMatch(/^[a-f\d]{32}$/);
 
+  // contexts.trace.data must include thread.id to identify which thread is associated with the transaction
+  expect(rootSpan?.contexts?.trace?.data?.['thread.id']).toBe('0');
+  expect(rootSpan?.contexts?.trace?.data?.['thread.name']).toBe('main');
+
   const spans = (rootSpan?.spans ?? []) as Array<{ data?: Record<string, unknown> }>;
   expect(spans.length).toBeGreaterThan(0);
   for (const span of spans) {