feat(jest-environment-puppeteer): added config option for global err msg

[kaufmann42]

Closes #34

[gregberge]

Thanks, can you default it to true?

[kaufmann42]

Will do.

[kaufmann42]

Went ahead and added.

[gregberge]

Thanks!

[linusrachlis]

Hi @neoziro ,

Just curious what's the thinking behind having this default to true. Should any page-level JS error be considered a test failure? Sadly these days, it's a rare website that has no JS errors, and sometimes they are caused by external libraries out of the developer's control. I would have thought the more common case would be that you don't want the browser to exit on a page-level error, although some people might want the option to do so. What do you think?

Thanks,
Linus

[gregberge]

Hello @linusrachlis, I think it is better to encourage user to not have uncaught errors in console. Anyway now you have the choice but defaulting to true encourages best practice.