From e9ef9e7a2b6f5dfa3066fc3d2e77b78892fbf32e Mon Sep 17 00:00:00 2001 From: Francesco Gringl-Novy Date: Mon, 4 May 2026 15:46:41 +0200 Subject: [PATCH] test(browser): Fix flaky loader test --- .../loader/noOnLoad/sdkLoadedInMeanwhile/test.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/dev-packages/browser-integration-tests/loader-suites/loader/noOnLoad/sdkLoadedInMeanwhile/test.ts b/dev-packages/browser-integration-tests/loader-suites/loader/noOnLoad/sdkLoadedInMeanwhile/test.ts index d758ec5e7901..276bda3227ac 100644 --- a/dev-packages/browser-integration-tests/loader-suites/loader/noOnLoad/sdkLoadedInMeanwhile/test.ts +++ b/dev-packages/browser-integration-tests/loader-suites/loader/noOnLoad/sdkLoadedInMeanwhile/test.ts @@ -53,8 +53,9 @@ sentryTest('it does not download the SDK if the SDK was loaded in the meanwhile' // Still loaded the CDN bundle twice await expect.poll(() => cdnLoadedCount, { timeout: 15_000 }).toBe(2); - // But only sent to Sentry once - expect(sentryEventCount).toBe(1); + // But only sent to Sentry once (`waitForErrorRequest` can resolve before the DSN + // `page.route` handler increments — poll until the intercept has run) + await expect.poll(() => sentryEventCount, { timeout: 15_000 }).toBe(1); // Ensure loader does not overwrite init/config const options = await page.evaluate(() => (window as any).Sentry.getClient()?.getOptions());