fix: wire up exitOnPageError config option (#108)

Author: Steve Davis

README.md

@@ -80,7 +80,7 @@ You can specify a `jest-playwright.config.js` at the root of the project or defi
   - `firefox` Each test runs Firefox.
   - `webkit` Each test runs Webkit.
 - `device` <[string]>. Define a [device](https://github.com/microsoft/playwright/blob/master/docs/api.md#browsertypedevices) to run tests into. Actual list of devices can be found [here](https://github.com/Microsoft/playwright/blob/master/src/deviceDescriptors.ts)
-- `exitOnPageError` <[boolean]> Exits page on any global error message thrown. Defaults to `true`.
+- `exitOnPageError` <[boolean]> Exits process on any page error. Defaults to `true`.
 - `server` <[object]> [All `jest-dev-server` options](https://github.com/smooth-code/jest-puppeteer/tree/master/packages/jest-dev-server#options)
 - `selectors` <[array]>. Define [selector](https://github.com/microsoft/playwright/blob/v0.11.1/docs/api.md#class-selectors). Each selector must be an object with name and script properties.
 

src/PlaywrightEnvironment.ts

@@ -92,7 +92,7 @@ class PlaywrightEnvironment extends NodeEnvironment {
     const config = await readConfig(this._config.rootDir)
     const browserType = getBrowserType(config)
     checkBrowserEnv(browserType)
-    const { context, server, selectors } = config
+    const { context, exitOnPageError, server, selectors } = config
     const device = getDeviceType(config)
     const playwrightInstance = await getPlaywrightInstance(
       browserType,
@@ -106,7 +106,7 @@ class PlaywrightEnvironment extends NodeEnvironment {
       const { setup, ERROR_TIMEOUT, ERROR_NO_COMMAND } = devServer
       teardownServer = devServer.teardown
       try {
-        await setup(config.server)
+        await setup(server)
       } catch (error) {
         if (error.code === ERROR_TIMEOUT) {
           logMessage({
@@ -133,7 +133,9 @@ class PlaywrightEnvironment extends NodeEnvironment {
     this.global.browser = await getBrowserPerProcess(playwrightInstance, config)
     this.global.context = await this.global.browser.newContext(contextOptions)
     this.global.page = await this.global.context.newPage()
-    this.global.page.on('pageerror', handleError)
+    if (exitOnPageError) {
+      this.global.page.on('pageerror', handleError)
+    }
     this.global.jestPlaywright = {
       debug: async (): Promise<void> => {
         // Run a debugger (in case Playwright has been launched with `{ devtools: true }`)