fix error events not working

Author: robin-drexler

.changeset/nasty-spoons-wonder.md

@@ -0,0 +1,5 @@
+---
+'@remote-dom/polyfill': patch
+---
+
+fix error events not working

packages/polyfill/source/ErrorEvent.ts

@@ -0,0 +1,20 @@
+import {Event} from './Event.ts';
+
+// https://html.spec.whatwg.org/multipage/webappapis.html#errorevent
+export class ErrorEvent extends Event {
+  readonly message: ErrorEventInit['message'];
+  readonly filename: ErrorEventInit['filename'];
+  readonly lineno: ErrorEventInit['lineno'];
+  readonly colno: ErrorEventInit['colno'];
+  readonly error: ErrorEventInit['error'];
+
+  constructor(type: string, eventInitDict: ErrorEventInit) {
+    super(type, eventInitDict);
+
+    this.message = eventInitDict.message;
+    this.filename = eventInitDict.filename;
+    this.lineno = eventInitDict.lineno;
+    this.colno = eventInitDict.colno;
+    this.error = eventInitDict.error;
+  }
+}

packages/polyfill/source/PromiseRejectionEvent.ts

@@ -0,0 +1,14 @@
+import {Event} from './Event.ts';
+
+//https://html.spec.whatwg.org/multipage/webappapis.html#promiserejectionevent
+export class PromiseRejectionEvent extends Event {
+  readonly promise: PromiseRejectionEventInit['promise'];
+  readonly reason: PromiseRejectionEventInit['reason'];
+
+  constructor(type: string, eventInitDict: PromiseRejectionEventInit) {
+    super(type, eventInitDict);
+
+    this.promise = eventInitDict.promise;
+    this.reason = eventInitDict.reason;
+  }
+}

packages/polyfill/source/Window.ts

@@ -2,6 +2,8 @@ import {Document} from './Document.ts';
 import {Event} from './Event.ts';
 import {EventTarget} from './EventTarget.ts';
 import {CustomEvent} from './CustomEvent.ts';
+import {ErrorEvent} from './ErrorEvent.ts';
+import {PromiseRejectionEvent} from './PromiseRejectionEvent.ts';
 import {Node} from './Node.ts';
 import {ParentNode} from './ParentNode.ts';
 import {ChildNode} from './ChildNode.ts';
@@ -30,6 +32,8 @@ export class Window extends EventTarget {
   location = globalThis.location;
   navigator = globalThis.navigator;
   Event = Event;
+  ErrorEvent = ErrorEvent;
+  PromiseRejectionEvent = PromiseRejectionEvent;
   EventTarget = EventTarget;
   CustomEvent = CustomEvent;
   Node = Node;
@@ -46,6 +50,20 @@ export class Window extends EventTarget {
   HTMLTemplateElement = HTMLTemplateElement;
   MutationObserver = MutationObserver;
 
+  get onerror() {
+    return globalThis.onerror;
+  }
+  set onerror(handler: any) {
+    globalThis.onerror = handler;
+  }
+
+  get onunhandledrejection() {
+    return globalThis.onunhandledrejection;
+  }
+  set onunhandledrejection(handler: any) {
+    globalThis.onunhandledrejection = handler;
+  }
+
   static setGlobal(window: Window) {
     const properties = Object.getOwnPropertyDescriptors(window);
 

packages/polyfill/source/tests/global-errors.test.ts

@@ -0,0 +1,104 @@
+import {Window} from '../Window.ts';
+import {describe, it, expect, beforeEach, afterEach, vi} from 'vitest';
+
+describe('global errors', () => {
+  describe('onerror', () => {
+    let originalOnerror: any;
+
+    beforeEach(() => {
+      originalOnerror = globalThis.onerror;
+    });
+
+    afterEach(() => {
+      globalThis.onerror = originalOnerror;
+    });
+
+    it('sets onerror handler on window instance', () => {
+      const handler = vi.fn();
+      const window = new Window();
+
+      window.onerror = handler;
+      expect(window.onerror).toBe(handler);
+    });
+
+    it('sets onerror handler on globalThis when assigned to window', () => {
+      const handler = vi.fn();
+      const window = new Window();
+
+      window.onerror = handler;
+      expect(globalThis.onerror).toBe(handler);
+    });
+
+    it('allows null assignment', () => {
+      const handler = vi.fn();
+      const window = new Window();
+      window.onerror = handler;
+      expect(window.onerror).toBe(handler);
+      expect(globalThis.onerror).toBe(handler);
+
+      window.onerror = null;
+      expect(window.onerror).toBe(null);
+      expect(globalThis.onerror).toBe(null);
+    });
+
+    it('works when windowis is set as global', () => {
+      const window = new Window();
+      Window.setGlobalThis(window);
+      const handler = vi.fn();
+      window.onerror = handler;
+
+      expect(window.onerror).toBe(handler);
+      expect(globalThis.onerror).toBe(handler);
+    });
+  });
+
+  describe('onunhandledrejection', () => {
+    let originalOnunhandledrejection: any;
+
+    beforeEach(() => {
+      originalOnunhandledrejection = globalThis.onunhandledrejection;
+    });
+
+    afterEach(() => {
+      globalThis.onunhandledrejection = originalOnunhandledrejection;
+    });
+
+    it('sets onunhandledrejection handler on window instance', () => {
+      const handler = vi.fn();
+      const window = new Window();
+
+      window.onunhandledrejection = handler;
+      expect(window.onunhandledrejection).toBe(handler);
+    });
+
+    it('sets onunhandledrejection handler on globalThis when assigned to window', () => {
+      const handler = vi.fn();
+      const window = new Window();
+
+      window.onunhandledrejection = handler;
+      expect(globalThis.onunhandledrejection).toBe(handler);
+    });
+
+    it('allows null assignment', () => {
+      const handler = vi.fn();
+      const window = new Window();
+      window.onunhandledrejection = handler;
+      expect(window.onunhandledrejection).toBe(handler);
+      expect(globalThis.onunhandledrejection).toBe(handler);
+
+      window.onunhandledrejection = null;
+      expect(window.onunhandledrejection).toBe(null);
+      expect(globalThis.onunhandledrejection).toBe(null);
+    });
+
+    it('works when windowis is set as global', () => {
+      const window = new Window();
+      Window.setGlobalThis(window);
+      const handler = vi.fn();
+      window.onunhandledrejection = handler;
+
+      expect(window.onunhandledrejection).toBe(handler);
+      expect(globalThis.onunhandledrejection).toBe(handler);
+    });
+  });
+});