From c1fbeae8ae5d8034c10fd36544ad44db2ea007ac Mon Sep 17 00:00:00 2001 From: Abhijeet Prasad Date: Wed, 17 Aug 2022 16:03:26 -0400 Subject: [PATCH 1/2] ref(tracing): Update long task description Give more context about what the issue is. --- packages/tracing/src/browser/metrics/index.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/tracing/src/browser/metrics/index.ts b/packages/tracing/src/browser/metrics/index.ts index ee94a66d1957..3bc3c9282f32 100644 --- a/packages/tracing/src/browser/metrics/index.ts +++ b/packages/tracing/src/browser/metrics/index.ts @@ -52,7 +52,7 @@ export function startTrackingLongTasks(): void { const startTime = msToSec((browserPerformanceTimeOrigin as number) + entry.startTime); const duration = msToSec(entry.duration); transaction.startChild({ - description: 'Long Task', + description: 'Main UI thread blocked', op: 'ui.long-task', startTimestamp: startTime, endTimestamp: startTime + duration, From 5e75e7b6991ab2bc1b5cac9c5bfc51ca5937c084 Mon Sep 17 00:00:00 2001 From: Abhijeet Prasad Date: Wed, 17 Aug 2022 17:14:16 -0400 Subject: [PATCH 2/2] fix tests --- .../suites/tracing/browsertracing/long-tasks-enabled/test.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/integration-tests/suites/tracing/browsertracing/long-tasks-enabled/test.ts b/packages/integration-tests/suites/tracing/browsertracing/long-tasks-enabled/test.ts index 91a8f6087376..7390d390224a 100644 --- a/packages/integration-tests/suites/tracing/browsertracing/long-tasks-enabled/test.ts +++ b/packages/integration-tests/suites/tracing/browsertracing/long-tasks-enabled/test.ts @@ -23,7 +23,7 @@ sentryTest('should capture long task.', async ({ browserName, getLocalTestPath, expect(firstUISpan).toEqual( expect.objectContaining({ op: 'ui.long-task', - description: 'Long Task', + description: 'Main UI thread blocked', parent_span_id: eventData.contexts?.trace.span_id, }), );