fixup! lib: add abortSignal.throwIfAborted()

nodejs · Nov 24, 2021 · f909d80 · f909d80
1 parent 02cc0ee
commit f909d80
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/internal/abort_controller.js b/lib/internal/abort_controller.js
@@ -90,7 +90,7 @@ class AbortSignal extends EventTarget {
    * @returns {AbortSignal}
    */
   static abort(
-      reason = new DOMException('This operation was aborted', 'AbortError')) {
+    reason = new DOMException('This operation was aborted', 'AbortError')) {
     return createAbortSignal(true, reason);
   }
 }

diff --git a/test/parallel/test-abortcontroller.js b/test/parallel/test-abortcontroller.js
@@ -169,4 +169,8 @@ const { ok, strictEqual, throws } = require('assert');
 {
   // Test abortSignal.throwIfAborted()
   throws(() => AbortSignal.abort().throwIfAborted(), { code: 20 });
+
+  // Does not throw because it's not aborted.
+  const ac = new AbortController();
+  ac.signal.throwIfAborted();
 }