test: add tests for --skipUnresponsiveTabs flag

- CLI test: verify the flag is parsed correctly and defaults to false
- McpContext test: verify page enumeration works with the flag enabled
- Update withMcpContext test helper to accept the new option

Author: jaidhyani

tests/McpContext.test.ts

@@ -37,6 +37,17 @@ describe('McpContext', () => {
     });
   });
 
+  it('list pages with skipUnresponsiveTabs enabled', async () => {
+    await withMcpContext(
+      async (_response, context) => {
+        const page = context.getSelectedMcpPage();
+        assert.ok(page, 'Should have a selected page');
+        assert.ok(page.pptrPage.url(), 'Page should have a URL');
+      },
+      {skipUnresponsiveTabs: true},
+    );
+  });
+
   it('can store and retrieve the latest performance trace', async () => {
     await withMcpContext(async (_response, context) => {
       const fakeTrace1 = {} as unknown as TraceResult;

tests/cli.test.ts

@@ -23,6 +23,8 @@ describe('cli args parsing', () => {
     performanceCrux: true,
     'usage-statistics': true,
     usageStatistics: true,
+    'skip-unresponsive-tabs': false,
+    skipUnresponsiveTabs: false,
   };
 
   it('parses with default args', async () => {
@@ -54,6 +56,17 @@ describe('cli args parsing', () => {
     });
   });
 
+  it('parses --skipUnresponsiveTabs', async () => {
+    const args = parseArguments('1.0.0', [
+      'node',
+      'main.js',
+      '--browserUrl',
+      'http://localhost:9222',
+      '--skipUnresponsiveTabs',
+    ]);
+    assert.strictEqual(args.skipUnresponsiveTabs, true);
+  });
+
   it('parses with user data dir', async () => {
     const args = parseArguments('1.0.0', [
       'node',

tests/utils.ts

@@ -103,6 +103,7 @@ export async function withMcpContext(
     debug?: boolean;
     autoOpenDevTools?: boolean;
     performanceCrux?: boolean;
+    skipUnresponsiveTabs?: boolean;
     executablePath?: string;
   } = {},
   args: ParsedArguments = {} as ParsedArguments,
@@ -118,6 +119,7 @@ export async function withMcpContext(
       {
         experimentalDevToolsDebugging: false,
         performanceCrux: options.performanceCrux ?? true,
+        skipUnresponsiveTabs: options.skipUnresponsiveTabs,
       },
       Locator,
     );