Fixing tests so they can run in both NodeJS and browsers

Author: moufmouf

test/basic.ts

@@ -2,13 +2,6 @@ import common from './common.js'
 import Peer from '../index.js'
 import { test, expect } from 'vitest'
 
-// Handle uncaught exceptions in both Node.js and browser
-if (typeof process !== 'undefined' && process.on) {
-  process.on('uncaughtException', console.error) // User-Initiated Abort, reason=Close called
-} else if (typeof window !== 'undefined') {
-  window.addEventListener('error', (event) => console.error(event.error))
-}
-
 test('detect WebRTC support', function () {
   expect(Peer.WEBRTC_SUPPORT).toBe(true)
 })

test/stream.ts

@@ -2,13 +2,6 @@ import Peer from '../index.js'
 import str from 'string-to-stream'
 import { test, expect } from 'vitest'
 
-// Handle uncaught exceptions in both Node.js and browser
-if (typeof process !== 'undefined' && process.on) {
-  process.on('uncaughtException', console.error) // User-Initiated Abort, reason=Close called
-} else if (typeof window !== 'undefined') {
-  window.addEventListener('error', (event) => console.error(event.error))
-}
-
 test('duplex stream: send data one-way', { timeout: 20000 }, function () {
   return new Promise<void>((resolve) => {
     const peer1 = new Peer({ initiator: true })
@@ -17,6 +10,8 @@ test('duplex stream: send data one-way', { timeout: 20000 }, function () {
     peer2.on('signal', function (data) { peer1.signal(data) })
     peer1.on('connect', tryTest)
     peer2.on('connect', tryTest)
+    peer1.on('error', function (err) { console.error(err) })
+    peer2.on('error', function (err) { console.error(err) })
 
     function tryTest () {
       if (!peer1.connected || !peer2.connected) return

vitest.browser.config.ts

@@ -31,7 +31,9 @@ export default defineConfig({
       instances: [
         // Default to chromium, but can be overridden with --browser.name flag
         // Supported browsers: chromium, firefox, webkit
-        { browser: 'chromium', provider: playwright() }
+        { browser: 'chromium', provider: playwright() },
+        { browser: 'firefox', provider: playwright() },
+        { browser: 'webkit', provider: playwright() },
       ],
       headless: true
     }