lib: add cause to DOMException

PR-URL: nodejs#44703 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]>
panva · Oct 4, 2022 · d360f30 · d360f30
1 parent 897fc3d
commit d360f30
Show file tree

Hide file tree

Showing 2 changed files with 57 additions and 5 deletions.
diff --git a/lib/internal/per_context/domexception.js b/lib/internal/per_context/domexception.js
@@ -49,12 +49,31 @@ const disusedNamesSet = new SafeSet()
   .add('ValidationError');
 
 class DOMException {
-  constructor(message = '', name = 'Error') {
+  constructor(message = '', options = 'Error') {
     ErrorCaptureStackTrace(this);
-    internalsMap.set(this, {
-      message: `${message}`,
-      name: `${name}`
-    });
+
+    if (options && typeof options === 'object') {
+      const { name } = options;
+      internalsMap.set(this, {
+        message: `${message}`,
+        name: `${name}`
+      });
+
+      if ('cause' in options) {
+        ObjectDefineProperty(this, 'cause', {
+          __proto__: null,
+          value: options.cause,
+          configurable: true,
+          writable: true,
+          enumerable: false,
+        });
+      }
+    } else {
+      internalsMap.set(this, {
+        message: `${message}`,
+        name: `${options}`
+      });
+    }
   }
 
   get name() {

diff --git a/test/parallel/test-domexception-cause.js b/test/parallel/test-domexception-cause.js
@@ -0,0 +1,33 @@
+'use strict';
+
+require('../common');
+const { strictEqual, deepStrictEqual } = require('assert');
+
+{
+  const domException = new DOMException('no cause', 'abc');
+  strictEqual(domException.name, 'abc');
+  strictEqual('cause' in domException, false);
+  strictEqual(domException.cause, undefined);
+}
+
+{
+  const domException = new DOMException('with undefined cause', { name: 'abc', cause: undefined });
+  strictEqual(domException.name, 'abc');
+  strictEqual('cause' in domException, true);
+  strictEqual(domException.cause, undefined);
+}
+
+{
+  const domException = new DOMException('with string cause', { name: 'abc', cause: 'foo' });
+  strictEqual(domException.name, 'abc');
+  strictEqual('cause' in domException, true);
+  strictEqual(domException.cause, 'foo');
+}
+
+{
+  const object = { reason: 'foo' };
+  const domException = new DOMException('with object cause', { name: 'abc', cause: object });
+  strictEqual(domException.name, 'abc');
+  strictEqual('cause' in domException, true);
+  deepStrictEqual(domException.cause, object);
+}