fix(playwright): relaunch browser correctly with restart: 'session' in run-workers --by pool (#5118)

* fix: Reset browser context and state on close

* test: add acceptance tests

Author: Samuel-StO

.github/workflows/playwright.yml

@@ -48,7 +48,9 @@ jobs:
         run: './bin/codecept.js run -c test/acceptance/codecept.Playwright.js --grep @Playwright --debug'
       - name: run chromium with restart==browser tests
         run: 'BROWSER_RESTART=browser ./bin/codecept.js run -c test/acceptance/codecept.Playwright.js --grep @Playwright --debug'
-      - name: run chromium with restart==session tests
+      - name: run chromium with restart==session tests on 2 workers split by pool
+        run: 'BROWSER_RESTART=session ./bin/codecept.js run-workers 2 -c test/acceptance/codecept.Playwright.js --grep @Playwright --debug --by=pool'
+      - name: run chromium with restart==session tests and
         run: 'BROWSER_RESTART=session ./bin/codecept.js run -c test/acceptance/codecept.Playwright.js --grep @Playwright --debug'
       - name: run firefox tests
         run: 'BROWSER=firefox node ./bin/codecept.js run -c test/acceptance/codecept.Playwright.js --grep @Playwright --debug'

lib/helper/Playwright.js

@@ -1031,7 +1031,10 @@ class Playwright extends Helper {
     this.frame = null
     popupStore.clear()
     if (this.options.recordHar) await this.browserContext.close()
+    this.browserContext = null
     await this.browser.close()
+    this.browser = null
+    this.isRunning = false
   }
 
   async _evaluateHandeInContext(...args) {