fix: address Playwright 1.56.0 test failures

- Update userAgent to match Chromium 141 in playwright-config.ts
- Filter out favicon resource spans in react-create-*-router tests
- Add retry logic for flaky navigation in browser-integration-tests
- Handle error status code navigation in astro-4/5 SSR error tests

Author: BYK

dev-packages/browser-integration-tests/suites/sessions/initial-scope/test.ts

@@ -18,11 +18,12 @@ sentryTest('should start a new session on pageload.', async ({ getLocalTestUrl,
 sentryTest('should start a new session with navigation.', async ({ getLocalTestUrl, page }) => {
   const url = await getLocalTestUrl({ testDir: __dirname });
 
+  // Route must be set up before any navigation to avoid race conditions
   await page.route('**/foo', (route: Route) => route.continue({ url }));
 
   const initSession = await getFirstSentryEnvelopeRequest<SessionContext>(page, url);
 
-  await page.click('#navigate');
+  await page.locator('#navigate').click();
 
   const newSession = await getFirstSentryEnvelopeRequest<SessionContext>(page, url);
 

dev-packages/browser-integration-tests/utils/helpers.ts

@@ -149,7 +149,13 @@ export const countEnvelopes = async (
   });
 
   if (options?.url) {
-    await page.goto(options.url);
+    // Retry navigation once if it fails due to route handler timing issues
+    try {
+      await page.goto(options.url);
+    } catch (e) {
+      // Retry once for flaky navigation errors (e.g., route handler not ready in time)
+      await page.goto(options.url);
+    }
   }
 
   return countPromise;
@@ -404,7 +410,13 @@ async function getMultipleRequests<T>(
   });
 
   if (options?.url) {
-    await page.goto(options.url);
+    // Retry navigation once if it fails due to route handler timing issues
+    try {
+      await page.goto(options.url);
+    } catch (e) {
+      // Retry once for flaky navigation errors (e.g., route handler not ready in time)
+      await page.goto(options.url);
+    }
   }
 
   return requests;

dev-packages/e2e-tests/test-applications/astro-4/tests/errors.server.test.ts

@@ -11,7 +11,10 @@ test.describe('server-side errors', () => {
       return transactionEvent.transaction === 'GET /ssr-error';
     });
 
-    await page.goto('/ssr-error');
+    // This page returns an error status code, so we need to catch the navigation error
+    await page.goto('/ssr-error').catch(() => {
+      // Expected to fail with net::ERR_HTTP_RESPONSE_CODE_FAILURE in newer Chromium versions
+    });
 
     const errorEvent = await errorEventPromise;
     const transactionEvent = await transactionEventPromise;

dev-packages/e2e-tests/test-applications/astro-5/tests/errors.server.test.ts

@@ -11,7 +11,10 @@ test.describe('server-side errors', () => {
       return transactionEvent.transaction === 'GET /ssr-error';
     });
 
-    await page.goto('/ssr-error');
+    // This page returns an error status code, so we need to catch the navigation error
+    await page.goto('/ssr-error').catch(() => {
+      // Expected to fail with net::ERR_HTTP_RESPONSE_CODE_FAILURE in newer Chromium versions
+    });
 
     const errorEvent = await errorEventPromise;
     const transactionEvent = await transactionEventPromise;

dev-packages/e2e-tests/test-applications/react-create-browser-router/tests/transactions.test.ts

@@ -86,7 +86,9 @@ test('Captures a navigation transaction', async ({ page }) => {
     }),
   );
 
-  expect(transactionEvent.spans).toEqual([]);
+  // Filter out favicon spans which may or may not be present depending on the browser version
+  const spans = (transactionEvent.spans || []).filter(span => !span.description?.includes('favicon'));
+  expect(spans).toEqual([]);
 });
 
 test('Captures a lazy pageload transaction', async ({ page }) => {

dev-packages/e2e-tests/test-applications/react-create-hash-router/tests/transactions.test.ts

@@ -150,7 +150,9 @@ test('Captures a navigation transaction', async ({ page }) => {
     }),
   );
 
-  expect(transactionEvent.spans).toEqual([]);
+  // Filter out favicon spans which may or may not be present depending on the browser version
+  const spans = (transactionEvent.spans || []).filter(span => !span.description?.includes('favicon'));
+  expect(spans).toEqual([]);
 });
 
 test('Captures a parameterized path pageload transaction', async ({ page }) => {

dev-packages/test-utils/src/playwright-config.ts

@@ -54,11 +54,11 @@ export function getPlaywrightConfig(
       {
         name: 'chromium',
         use: {
-          // This comes from `devices["Desktop Chrome"]
+          // This comes from `devices["Desktop Chrome"]` in Playwright 1.56.0
           // We inline this instead of importing this,
           // because playwright otherwise complains that it was imported twice :(
           userAgent:
-            'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/125.0.6422.26 Safari/537.36',
+            'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/141.0.7390.37 Safari/537.36',
           viewport: { width: 1280, height: 720 },
           deviceScaleFactor: 1,
           isMobile: false,