feat(Playwright): Add browser restart strategy support

- Add 'browser' as valid restart option in PlaywrightRestartOpts
- Add restartsBrowser() function to check browser restart strategy
- Update Playwright helper to restart browser between tests when BROWSER_RESTART=browser
- Add timeout protection (5s) to prevent hanging during browser stop
- Close all pages before stopping browser for cleaner shutdown
- Update _before(), _after(), _afterSuite(), and _finishTest() methods

Fixes support for BROWSER_RESTART=browser environment variable to completely
restart the browser between each test, providing maximum isolation.

Author: kobenguyent

lib/helper/Playwright.js

@@ -43,7 +43,7 @@ const popupStore = new Popup()
 const consoleLogStore = new Console()
 const availableBrowsers = ['chromium', 'webkit', 'firefox', 'electron']
 
-import { setRestartStrategy, restartsSession, restartsContext } from './extras/PlaywrightRestartOpts.js'
+import { setRestartStrategy, restartsSession, restartsContext, restartsBrowser } from './extras/PlaywrightRestartOpts.js'
 import { createValueEngine, createDisabledEngine } from './extras/PlaywrightPropEngine.js'
 import { seeElementError, dontSeeElementError, dontSeeElementInDOMError, seeElementInDOMError } from './errors/ElementAssertion.js'
 import { dontSeeTraffic, seeTraffic, grabRecordedNetworkTraffics, stopRecordingTraffic, flushNetworkTraffics } from './network/actions.js'
@@ -600,7 +600,12 @@ class Playwright extends Helper {
       },
     })
 
+    // Start browser if needed (initial start or browser restart strategy)
     if (!this.isRunning && !this.options.manualStart) await this._startBrowser()
+    else if (restartsBrowser() && !this.options.manualStart) {
+      // Browser restart strategy: start browser for each test
+      await this._startBrowser()
+    }
 
     this.isAuthenticated = false
     if (this.isElectron) {
@@ -704,6 +709,31 @@ class Playwright extends Helper {
       return refreshContextSession.bind(this)()
     }
 
+    if (restartsBrowser()) {
+      // Close browser completely for restart strategy
+      if (this.isRunning) {
+        try {
+          // Close all pages first to release resources
+          if (this.browserContext) {
+            const pages = await this.browserContext.pages()
+            await Promise.allSettled(pages.map(p => p.close().catch(() => {})))
+          }
+          // Use timeout to prevent hanging
+          await Promise.race([
+            this._stopBrowser(),
+            new Promise((_, reject) => setTimeout(() => reject(new Error('Browser stop timeout')), 5000)),
+          ])
+        } catch (e) {
+          console.warn('Warning during browser restart in _after:', e.message)
+          // Force cleanup even on timeout
+          this.browser = null
+          this.browserContext = null
+          this.isRunning = false
+        }
+      }
+      return
+    }
+
     // close other sessions with timeout protection, but only if restartsContext() is true
     if (restartsContext()) {
       try {
@@ -730,7 +760,7 @@ class Playwright extends Helper {
 
   async _afterSuite() {
     // Only stop browser if restart strategy requires it
-    if ((restartsSession() || restartsContext()) && this.isRunning) {
+    if ((restartsSession() || restartsContext() || restartsBrowser()) && this.isRunning) {
       try {
         await this._stopBrowser()
       } catch (e) {
@@ -813,7 +843,7 @@ class Playwright extends Helper {
   }
 
   async _finishTest() {
-    if ((restartsSession() || restartsContext()) && this.isRunning) {
+    if ((restartsSession() || restartsContext() || restartsBrowser()) && this.isRunning) {
       try {
         await Promise.race([this._stopBrowser(), new Promise((_, reject) => setTimeout(() => reject(new Error('Test finish timeout')), 10000))])
       } catch (e) {

lib/helper/extras/PlaywrightRestartOpts.js

@@ -1,6 +1,7 @@
 const RESTART_OPTS = {
   session: 'keep',
   context: false,
+  browser: true,
 }
 
 let restarts = null
@@ -19,9 +20,15 @@ export function setRestartStrategy(options) {
     return
   }
 
+  // When restart is true, map to 'browser' restart
+  if (restart === true) {
+    restarts = 'browser'
+    return
+  }
+
   restarts = Object.keys(RESTART_OPTS).find(key => RESTART_OPTS[key] === restart)
 
-  if (restarts === null || restarts === undefined) throw new Error('No restart strategy set, use the following values for restart: session, context')
+  if (restarts === null || restarts === undefined) throw new Error('No restart strategy set, use the following values for restart: session, context, browser')
 }
 
 export function restartsSession() {
@@ -31,3 +38,7 @@ export function restartsSession() {
 export function restartsContext() {
   return restarts === 'context'
 }
+
+export function restartsBrowser() {
+  return restarts === 'browser'
+}