fix long-task tests

getsentry · Aug 7, 2024 · 7a42707 · 7a42707
1 parent 47842da
commit 7a42707
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/...ser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-enabled/init.js b/...ser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-enabled/init.js
@@ -7,6 +7,7 @@ Sentry.init({
   integrations: [
     Sentry.browserTracingIntegration({
       idleTimeout: 9000,
+      enableLongAnimationFrame: false,
     }),
   ],
   tracesSampleRate: 1,

diff --git a/packages/ember/tests/helpers/utils.ts b/packages/ember/tests/helpers/utils.ts
@@ -64,11 +64,15 @@ export function assertSentryTransactions(
 
   // instead of checking the specific order of runloop spans (which is brittle),
   // we check (below) that _any_ runloop spans are added
-  // Also we ignore ui.long-task spans, as they are brittle and may or may not appear
+  // Also we ignore ui.long-task spans and ui.long-animation-frame, as they are brittle and may or may not appear
   const filteredSpans = spans
     .filter(span => {
       const op = span.op;
-      return !op?.startsWith('ui.ember.runloop.') && !op?.startsWith('ui.long-task');
+      return (
+        !op?.startsWith('ui.ember.runloop.') &&
+        !op?.startsWith('ui.long-task') &&
+        !op?.startsWith('ui.long-animation-frame')
+      );
     })
     .map(spanJson => {
       return `${spanJson.op} | ${spanJson.description}`;