lib: throw error in structuedClone when no arguments are passed

PR-URL: #41651 Fixes: #41450 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
nodejs · Feb 7, 2022 · 8ab0540 · 8ab0540
1 parent 2bba6cd
commit 8ab0540
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/lib/internal/structured_clone.js b/lib/internal/structured_clone.js
@@ -1,12 +1,20 @@
 'use strict';
 
+const {
+  codes: { ERR_MISSING_ARGS },
+} = require('internal/errors');
+
 const {
   MessageChannel,
   receiveMessageOnPort,
 } = require('internal/worker/io');
 
 let channel;
 function structuredClone(value, options = undefined) {
+  if (arguments.length === 0) {
+    throw new ERR_MISSING_ARGS('value');
+  }
+
   // TODO: Improve this with a more efficient solution that avoids
   // instantiating a MessageChannel
   channel ??= new MessageChannel();
@@ -17,5 +25,5 @@ function structuredClone(value, options = undefined) {
 }
 
 module.exports = {
-  structuredClone
+  structuredClone,
 };
diff --git a/test/parallel/test-structuredClone-global.js b/test/parallel/test-structuredClone-global.js
@@ -5,10 +5,12 @@
 require('../common');
 
 const {
-  structuredClone: _structuredClone
+  structuredClone: _structuredClone,
 } = require('internal/structured_clone');
+
 const {
-  strictEqual
+  strictEqual,
+  throws,
 } = require('assert');
 
 strictEqual(globalThis.structuredClone, _structuredClone);
@@ -17,3 +19,5 @@ strictEqual(globalThis.structuredClone, undefined);
 
 // Restore the value for the known globals check.
 structuredClone = _structuredClone;
+
+throws(() => _structuredClone(), /ERR_MISSING_ARGS/);