fix: import and throw DOMException correctly

The `node-domexception` module polyfills the global environment with
the DOMExecption constructor but it doesn't export anything, so use
a bare import to perform the polyfill then use the global constructor.

Adds a test to prevent regressions.

Author: achingbrain

polyfill/RTCPeerConnection.js

@@ -4,11 +4,11 @@ import RTCDataChannel from './RTCDataChannel.js';
 import RTCIceCandidate from './RTCIceCandidate.js';
 import { RTCDataChannelEvent, RTCPeerConnectionIceEvent } from './Events.js';
 import RTCSctpTransport from './RTCSctpTransport.js';
-import DOMException from 'node-domexception';
+import 'node-domexception';
 
 export default class _RTCPeerConnection extends EventTarget {
     static async generateCertificate() {
-        throw new Error('Not implemented');
+        throw new DOMException('Not implemented');
     }
 
     #peerConnection;
@@ -192,10 +192,6 @@ export default class _RTCPeerConnection extends EventTarget {
         return this.#peerConnection.signalingState();
     }
 
-    static generateCertificate(keygenAlgorithm) {
-        throw new DOMException('Not implemented');
-    }
-
     async addIceCandidate(candidate) {
         if (candidate == null || candidate.candidate == null) {
             throw new DOMException('Candidate invalid');

test/jest-tests/polyfill.test.js

@@ -0,0 +1,10 @@
+import { expect } from '@jest/globals';
+import polyfill from '../../polyfill/index.js';
+
+describe('polyfill', () => {
+    test('generateCertificate should throw', () => {
+        expect(async () => {
+            await polyfill.RTCPeerConnection.generateCertificate({});
+        }).rejects.toEqual(new DOMException('Not implemented'));
+    });
+});