unflake some tests?

Author: mydea

dev-packages/browser-integration-tests/scripts/detectFlakyTests.ts

@@ -100,7 +100,7 @@ ${changedPaths.join('\n')}
  * Returns how many time one test should run based on the chosen mode and a bunch of heuristics
  */
 function getPerTestRunCount(testPaths: string[]) {
-  if (process.env.TEST_RUN_COUNT === 'AUTO' && testPaths.length > 0) {
+  if ((!process.env.TEST_RUN_COUNT || process.env.TEST_RUN_COUNT === 'AUTO') && testPaths.length > 0) {
     // Run everything up to 100x, assuming that total runtime is less than 60min.
     // We assume an average runtime of 3s per test, times 4 (for different browsers) = 12s per detected testPaths
     // We want to keep overall runtime under 30min

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-animation-frame-enabled/test.ts

@@ -30,9 +30,11 @@ sentryTest(
 
     const uiSpans = eventData.spans?.filter(({ op }) => op?.startsWith('ui.long-animation-frame'));
 
-    expect(uiSpans?.length).toEqual(1);
+    expect(uiSpans?.length).toBeGreaterThanOrEqual(1);
 
-    const [topLevelUISpan] = uiSpans || [];
+    const topLevelUISpan = (uiSpans || []).find(
+      span => span.data?.['browser.script.invoker'] === 'https://example.com/path/to/script.js',
+    )!;
     expect(topLevelUISpan).toEqual(
       expect.objectContaining({
         op: 'ui.long-animation-frame',
@@ -84,10 +86,11 @@ sentryTest(
 
     const uiSpans = eventData.spans?.filter(({ op }) => op?.startsWith('ui.long-animation-frame'));
 
-    expect(uiSpans?.length).toEqual(2);
+    expect(uiSpans?.length).toBeGreaterThanOrEqual(2);
 
-    // ignore the first ui span (top-level long animation frame)
-    const [, eventListenerUISpan] = uiSpans || [];
+    const eventListenerUISpan = (uiSpans || []).find(
+      span => span.data?.['browser.script.invoker'] === 'BUTTON#clickme.onclick',
+    )!;
 
     expect(eventListenerUISpan).toEqual(
       expect.objectContaining({

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-and-animation-frame-enabled/test.ts

@@ -32,9 +32,11 @@ sentryTest(
 
     const uiSpans = eventData.spans?.filter(({ op }) => op?.startsWith('ui.long-animation-frame'));
 
-    expect(uiSpans?.length).toEqual(1);
+    expect(uiSpans?.length).toBeGreaterThanOrEqual(1);
 
-    const [topLevelUISpan] = uiSpans || [];
+    const topLevelUISpan = (uiSpans || []).find(
+      span => span.data?.['browser.script.invoker'] === 'https://example.com/path/to/script.js',
+    )!;
     expect(topLevelUISpan).toEqual(
       expect.objectContaining({
         op: 'ui.long-animation-frame',
@@ -86,10 +88,11 @@ sentryTest(
 
     const uiSpans = eventData.spans?.filter(({ op }) => op?.startsWith('ui.long-animation-frame'));
 
-    expect(uiSpans?.length).toEqual(2);
+    expect(uiSpans?.length).toBeGreaterThanOrEqual(2);
 
-    // ignore the first ui span (top-level long animation frame)
-    const [, eventListenerUISpan] = uiSpans || [];
+    const eventListenerUISpan = (uiSpans || []).find(
+      span => span.data?.['browser.script.invoker'] === 'BUTTON#clickme.onclick',
+    )!;
 
     expect(eventListenerUISpan).toEqual(
       expect.objectContaining({