fix

Author: chargome

dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/spotlight-interaction-filter/test.ts

@@ -33,14 +33,22 @@ sentryTest(
     const spotlightInteractionSpans = spotlightTransaction.spans?.filter(span => span.op === 'ui.interaction.click');
     expect(spotlightInteractionSpans).toHaveLength(0);
 
-    // Click on the regular button — interaction span should be kept
-    const regularTxnPromise = waitForTransactionRequest(page, txn => txn.contexts?.trace?.op === 'ui.action.click');
+    // Let the first idle span fully settle before clicking again
+    await page.waitForTimeout(1000);
+
+    // Click on the regular button — wait specifically for a transaction that contains
+    // a ui.interaction.click child span, since the PerformanceObserver may deliver
+    // the event entry asynchronously
+    const regularTxnPromise = waitForTransactionRequest(
+      page,
+      txn =>
+        txn.contexts?.trace?.op === 'ui.action.click' &&
+        (txn.spans?.some(span => span.op === 'ui.interaction.click') ?? false),
+    );
     await page.locator('[data-test-id=regular-button]').click();
     await page.locator('.clicked[data-test-id=regular-button]').isVisible();
     const regularTransaction = envelopeRequestParser<TransactionEvent>(await regularTxnPromise);
 
-    expect(regularTransaction.contexts?.trace?.op).toBe('ui.action.click');
-
     const regularInteractionSpans = regularTransaction.spans?.filter(span => span.op === 'ui.interaction.click');
     expect(regularInteractionSpans?.length).toBeGreaterThanOrEqual(1);
     expect(regularInteractionSpans![0]!.description).toContain('button');