Update

Author: inecik

tests/browser-application-retirement-contract.test.mjs

@@ -31,6 +31,31 @@ test('every browser spec owns production application retirement', async () => {
   }
 });
 
+test('every explicit browser page or context retires before closure', async () => {
+  const specFiles = (await readdir(browserDir))
+    .filter(name => name.endsWith('.spec.mjs'))
+    .sort();
+
+  for (const filename of specFiles) {
+    const source = await readFile(path.join(browserDir, filename), 'utf8');
+    if (/browser\.newContext\(/.test(source)) {
+      assert.match(
+        source,
+        /closeContextWithApplicationRetirement/,
+        `${filename} closes a custom context without application retirement`,
+      );
+      assert.doesNotMatch(source, /\bcontext\?*\.close\(/);
+    }
+    if (/context\.newPage\(/.test(source)) {
+      assert.match(
+        source,
+        /close(?:Page|Context)WithApplicationRetirement/,
+        `${filename} closes an explicit page without application retirement`,
+      );
+    }
+  }
+});
+
 test('the browser fixture retires even a failed test through the stable owner', async () => {
   const source = await readFile(
     path.join(browserDir, 'helpers', 'test.mjs'),
@@ -40,7 +65,8 @@ test('the browser fixture retires even a failed test through the stable owner',
     source,
     /finally\s*\{[\s\S]*retireContextApplications\(context\)/,
   );
-  assert.match(source, /applicationRetirement:[\s\S]*auto:\s*true/);
+  assert.match(source, /page:\s*async\s*\(\{\s*context,\s*page\s*\},\s*use\)/);
+  assert.doesNotMatch(source, /applicationRetirement:[\s\S]*auto:\s*true/);
   assert.match(source, /window\._cellucidDispose/);
   assert.match(source, /secondTask\s*!==\s*firstTask/);
   assert.match(

tests/browser/community-annotation-oauth-worker-lifecycle.spec.mjs

@@ -2,7 +2,11 @@ import { once } from 'node:events';
 import { createPrivateKey } from 'node:crypto';
 import { createServer } from 'node:https';
 
-import { expect, test } from './helpers/test.mjs';
+import {
+  closeContextWithApplicationRetirement,
+  expect,
+  test,
+} from './helpers/test.mjs';
 
 import worker from '../../assets/js/app/community-annotations/_worker-code.js';
 import {
@@ -180,6 +184,7 @@ test('OAuth owner cookies preserve exact raw Set-Cookie bytes', async () => {
 test(
   'state-specific OAuth cookies coexist and retire independently',
   async ({ browser }) => {
+    expect(browser.contexts()).toHaveLength(0);
     const previousFetch = globalThis.fetch;
     globalThis.fetch = async (_url, options) => {
       const body = parseExactJson(options.body);
@@ -290,7 +295,9 @@ test(
       expect(workerServer.serverErrors).toEqual([]);
     } finally {
       const cleanup = await Promise.allSettled([
-        context?.close(),
+        context === null
+          ? undefined
+          : closeContextWithApplicationRetirement(context),
         workerServer?.close(),
       ]);
       globalThis.fetch = previousFetch;

tests/browser/helpers/test.mjs

@@ -109,13 +109,18 @@ export async function closeContextWithApplicationRetirement(context) {
 }
 
 export const test = playwrightTest.extend({
-  applicationRetirement: [async ({ context }, use) => {
+  // Override the lazy page fixture instead of installing an auto context
+  // fixture. An auto fixture materializes an otherwise unused default context
+  // in browser-only tests, and its teardown can run after Playwright has
+  // already closed the base page. This boundary retires the application while
+  // both its document and context are still live.
+  page: async ({ context, page }, use) => {
     try {
-      await use();
+      await use(page);
     } finally {
       await retireContextApplications(context);
     }
-  }, { auto: true }],
+  },
 });
 
 export { expect };

tests/browser/reduced-motion-camera-path.spec.mjs

@@ -139,6 +139,10 @@ async function playAndMeasure(page, reducedMotion) {
 test('camera path playback honours a reduced-motion preference', async ({
   browser
 }) => {
+  // This test deliberately owns custom contexts so it can compare two media
+  // preferences in one browser generation. Its fixture must not materialize
+  // an unused default context alongside them.
+  expect(browser.contexts()).toHaveLength(0);
   const measurements = {};
   const errors = {};