chore: test

Author: nroscino

tests/tools/extensions.test.ts

@@ -122,9 +122,6 @@ describe('extension', () => {
       assert.ok(list.length === 1, 'List should have only one extension');
       const reinstalled = list.find(e => e.id === extensionId);
       assert.ok(reinstalled, 'Extension should be present after reload');
-      try {
-        await context.uninstallExtension(EXTENSION_WITH_SW_PATH);
-      } catch (e) {}
     });
   });
   it('triggers an extension action', async () => {
@@ -150,9 +147,6 @@ describe('extension', () => {
           t => t.type() === 'page' && t.url().includes(extensionId),
         );
         assert.ok(pageTargetAfter, 'Page should exist after action');
-        try {
-          await context.uninstallExtension(EXTENSION_WITH_SW_PATH);
-        } catch (e) {}
       },
       {},
       {

tests/tools/pages.test.ts

@@ -197,7 +197,9 @@ describe('pages', () => {
           t.assert.snapshot?.(text);
           await context.uninstallExtension(extensionId);
         },
-        {},
+        {
+          userDataDir: '/tmp/chrome-test',
+        },
         {
           categoryExtensions: true,
         } as ParsedArguments,

tests/utils.ts

@@ -62,6 +62,7 @@ export async function withBrowser(
     debug?: boolean;
     autoOpenDevTools?: boolean;
     executablePath?: string;
+    userDataDir?: string;
   } = {},
 ) {
   const launchOptions: LaunchOptions = {
@@ -74,6 +75,7 @@ export async function withBrowser(
     handleDevToolsAsPage: true,
     args: ['--screen-info={3840x2160}'],
     enableExtensions: true,
+    userDataDir: options.userDataDir,
   };
   const key = JSON.stringify(launchOptions);
 
@@ -102,6 +104,7 @@ export async function withMcpContext(
     autoOpenDevTools?: boolean;
     performanceCrux?: boolean;
     executablePath?: string;
+    userDataDir?: string;
   } = {},
   args: ParsedArguments = {} as ParsedArguments,
 ) {