add error session update test for page lifecycle

Lms24 · Lms24 · commit 5a94cadb5558 · 2026-02-18T10:43:34.000+01:00
diff --git a/dev-packages/browser-integration-tests/suites/sessions/page-lifecycle/subject.js b/dev-packages/browser-integration-tests/suites/sessions/page-lifecycle/subject.js
@@ -10,3 +10,7 @@ document.getElementById('manual-session').addEventListener('click', () => {
   Sentry.startSession();
   Sentry.captureException('Test error from manual session');
 });
+
+document.getElementById('error').addEventListener('click', () => {
+  throw new Error('Test error from error button');
+});
diff --git a/dev-packages/browser-integration-tests/suites/sessions/page-lifecycle/template.html b/dev-packages/browser-integration-tests/suites/sessions/page-lifecycle/template.html
@@ -6,5 +6,6 @@
   <body>
     <button id="navigate">Navigate via pushState</button>
     <button id="manual-session">Manual session</button>
+    <button id="error">Throw error</button>
   </body>
 </html>
diff --git a/dev-packages/browser-integration-tests/suites/sessions/page-lifecycle/test.ts b/dev-packages/browser-integration-tests/suites/sessions/page-lifecycle/test.ts
@@ -78,3 +78,28 @@ sentryTest(
     );
   },
 );
+
+sentryTest('Updates the session when an error is thrown', async ({ getLocalTestUrl, page }) => {
+  const url = await getLocalTestUrl({ testDir: __dirname });
+
+  const initialSessionPromise = waitForSession(page, s => !!s.init && s.status === 'ok');
+  await page.goto(url);
+  const initialSession = await initialSessionPromise;
+
+  // for good measure, throw in a few navigations
+  await page.locator('#navigate').click();
+  await page.locator('#navigate').click();
+  await page.locator('#navigate').click();
+
+  const updatedSessionPromise = waitForSession(page, s => !s.init && s.status !== 'ok');
+  await page.locator('#error').click();
+  const updatedSession = await updatedSessionPromise;
+
+  expect(updatedSession).toEqual({
+    ...initialSession,
+    errors: 1,
+    init: false,
+    status: 'crashed',
+    timestamp: expect.any(String),
+  });
+});
