fix: update puppeteer to v25 (#3135)

* fix: update puppeteer to v25

* fix: replace removed isConnected() with a new prop

* fix: prevent bfcache from caching test page with open WS connection

Author: bashmish

.changeset/silly-years-rescue.md

@@ -0,0 +1,7 @@
+---
+'@web/dev-server-core': patch
+'@web/test-runner-puppeteer': patch
+'@web/test-runner-chrome': patch
+---
+
+update puppeteer to v25

package-lock.json

@@ -50,6 +50,6 @@
     "errorstacks": "^2.4.1"
   },
   "devDependencies": {
-    "puppeteer": "^24.0.0"
+    "puppeteer": "^25.0.0"
   }
 }

packages/browser-logs/package.json

@@ -127,6 +127,7 @@ export function webSocketsPlugin(): Plugin {
           }
         }
 
+window.addEventListener('unload', () => {});
 
 const { protocol, host } = new URL(import.meta.url);
 const webSocketUrl = \`ws\${protocol === 'https:' ? 's' : ''}://\${host}/${NAME_WEB_SOCKET_API}\`;

packages/dev-server-core/src/web-sockets/webSocketsPlugin.ts

@@ -44,6 +44,6 @@
   },
   "devDependencies": {
     "lit-html": "^2.7.3 || ^3.0.0",
-    "puppeteer": "^24.0.0"
+    "puppeteer": "^25.0.0"
   }
 }

packages/dev-server-hmr/package.json

@@ -73,6 +73,6 @@
   "devDependencies": {
     "@types/command-line-usage": "^5.0.1",
     "lit-html": "^2.7.3 || ^3.0.0",
-    "puppeteer": "^24.0.0"
+    "puppeteer": "^25.0.0"
   }
 }

packages/dev-server/package.json

@@ -49,7 +49,7 @@
     "@web/test-runner-core": "^1.0.0",
     "@web/test-runner-coverage-v8": "^1.0.0",
     "chrome-launcher": "^0.15.0",
-    "puppeteer-core": "^24.0.0"
+    "puppeteer-core": "^25.0.0"
   },
   "devDependencies": {
     "@types/istanbul-reports": "^3.0.0",

packages/test-runner-chrome/package.json

@@ -123,10 +123,10 @@ export class ChromeLauncher implements BrowserLauncher {
   }
 
   async stop() {
-    if (this.browser?.isConnected()) {
+    if (this.browser?.connected) {
       await this.browser.close();
     }
-    if (this.debugBrowser?.isConnected()) {
+    if (this.debugBrowser?.connected) {
       await this.debugBrowser.close();
     }
   }
@@ -210,7 +210,7 @@ export class ChromeLauncher implements BrowserLauncher {
       return this.__startBrowserPromise;
     }
 
-    if (!this.browser || !this.browser?.isConnected() || !this.browserContext) {
+    if (!this.browser || !this.browser?.connected || !this.browserContext) {
       this.__startBrowserPromise = this.startBrowser();
       const { browser, context } = await this.__startBrowserPromise;
       this.browser = browser;

packages/test-runner-chrome/src/ChromeLauncher.ts

@@ -48,10 +48,10 @@
   "dependencies": {
     "@web/test-runner-chrome": "^1.0.0",
     "@web/test-runner-core": "^1.0.0",
-    "puppeteer": "^24.0.0"
+    "puppeteer": "^25.0.0"
   },
   "devDependencies": {
     "@web/test-runner-mocha": "^1.0.0",
-    "puppeteer-core": "^24.0.0"
+    "puppeteer-core": "^25.0.0"
   }
 }