errors: add support for cause in aborterror

Signed-off-by: James M Snell <[email protected]> PR-URL: #41008 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Robert Nagy <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
nodejs · Dec 2, 2021 · 36c0ac0 · 36c0ac0
1 parent 1086468
commit 36c0ac0
Show file tree

Hide file tree

Showing 2 changed files with 36 additions and 2 deletions.
diff --git a/lib/internal/errors.js b/lib/internal/errors.js
@@ -821,8 +821,11 @@ function hideInternalStackFrames(error) {
 // to make usage of the error in userland and readable-stream easier.
 // It is a regular error with `.code` and `.name`.
 class AbortError extends Error {
-  constructor() {
-    super('The operation was aborted');
+  constructor(message = 'The operation was aborted', options = undefined) {
+    if (options !== undefined && typeof options !== 'object') {
+      throw new codes.ERR_INVALID_ARG_TYPE('options', 'Object', options);
+    }
+    super(message, options);
     this.code = 'ABORT_ERR';
     this.name = 'AbortError';
   }

diff --git a/test/parallel/test-errors-aborterror.js b/test/parallel/test-errors-aborterror.js
@@ -0,0 +1,31 @@
+// Flags: --expose-internals
+'use strict';
+
+require('../common');
+const {
+  strictEqual,
+  throws,
+} = require('assert');
+const { AbortError } = require('internal/errors');
+
+{
+  const err = new AbortError();
+  strictEqual(err.message, 'The operation was aborted');
+  strictEqual(err.cause, undefined);
+}
+
+{
+  const cause = new Error('boom');
+  const err = new AbortError('bang', { cause });
+  strictEqual(err.message, 'bang');
+  strictEqual(err.cause, cause);
+}
+
+{
+  throws(() => new AbortError('', false), {
+    code: 'ERR_INVALID_ARG_TYPE'
+  });
+  throws(() => new AbortError('', ''), {
+    code: 'ERR_INVALID_ARG_TYPE'
+  });
+}