vm: migrate isContext to internal/errors

PR-URL: https://github.com/nodejs/node/pull/19268
Refs: https://github.com/nodejs/node/issues/18106
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@hiroppy.me>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

1 files changed, 18 insertions, 2 deletions

diff --git a/lib/vm.js b/lib/vm.js
index 5266dbd29f..ca7b6f3396 100644
--- a/lib/vm.js
+++ b/lib/vm.js
@@ -26,10 +26,13 @@ const {
   kParsingContext,
 
   makeContext,
-  isContext,
+  isContext: _isContext,
 } = process.binding('contextify');
 
-const { ERR_INVALID_ARG_TYPE } = require('internal/errors').codes;
+const {
+  ERR_INVALID_ARG_TYPE,
+  ERR_MISSING_ARGS
+} = require('internal/errors').codes;
 
 // The binding provides a few useful primitives:
 // - Script(code, { filename = "evalmachine.anonymous",
@@ -119,6 +122,19 @@ function getContextOptions(options) {
   return {};
 }
 
+function isContext(sandbox) {
+  if (arguments.length < 1) {
+    throw new ERR_MISSING_ARGS('sandbox');
+  }
+
+  if (typeof sandbox !== 'object' && typeof sandbox !== 'function' ||
+      sandbox === null) {
+    throw new ERR_INVALID_ARG_TYPE('sandbox', 'object', sandbox);
+  }
+
+  return _isContext(sandbox);
+}
+
 let defaultContextNameIndex = 1;
 function createContext(sandbox, options) {
   if (sandbox === undefined) {