feat: no more provider require function by default.

src/evaluate.ts

@@ -1422,21 +1422,27 @@ const visitors: EvaluateMap = {
 
     const requireVar = scope.hasBinding("require");
 
-    if (requireVar) {
-      const requireFunc = requireVar.value;
+    if (requireVar === undefined) {
+      throw ErrNotDefined("require");
+    }
 
-      const targetModule: any = requireFunc(moduleName) || {};
+    const requireFunc = requireVar.value;
 
-      if (defaultImport) {
-        scope.const(
-          defaultImport,
-          targetModule.default ? targetModule.default : targetModule
-        );
-      }
+    if (!isFunction(requireFunc)) {
+      throw ErrIsNotFunction("require");
+    }
 
-      otherImport.forEach((varName: string) => {
-        scope.const(varName, targetModule[varName]);
-      });
+    const targetModule: any = requireFunc(moduleName) || {};
+
+    if (defaultImport) {
+      scope.const(
+        defaultImport,
+        targetModule.default ? targetModule.default : targetModule
+      );
+    }
+
+    for (const varName of otherImport) {
+      scope.const(varName, targetModule[varName]);
     }
   },
   ImportDefaultSpecifier(path) {

src/vm.ts

@@ -24,21 +24,6 @@ export function runInContext(code: string, context: Context) {
   scope.const("module", $module);
   scope.var("exports", $exports);
 
-  // require can be cover
-  if (!scope.hasBinding("require")) {
-    const requireFunc =
-      // tslint:disable-next-line
-      typeof context["require"] === "function"
-        ? // tslint:disable-next-line
-          context["require"]
-        : typeof require === "function"
-          ? require
-          : function _require(id: string) {
-              return {};
-            };
-    scope.var("require", requireFunc);
-  }
-
   const ast = parse(code, {
     sourceType: "module",
     plugins: [
@@ -52,12 +37,11 @@ export function runInContext(code: string, context: Context) {
     ]
   });
 
-  const path = new Path(ast, null, scope, {});
-  evaluate(path);
+  evaluate(new Path(ast, null, scope, {}));
 
   // exports
   const moduleVar = scope.hasBinding("module");
-  return moduleVar ? moduleVar.value.exports : null;
+  return moduleVar ? moduleVar.value.exports : undefined;
 }
 
 /**

test/es2015/es-module/ImportDeclaration.test.ts

@@ -2,9 +2,10 @@ import test from "ava";
 import { isImportDefaultSpecifier } from "babel-types";
 import { parse } from "babylon";
 import vm from "../../../src/vm";
+import { ErrNotDefined, ErrIsNotFunction } from "../../../src/error";
 
 test("ImportDeclaration-1", t => {
-  const sandbox: any = vm.createContext({});
+  const sandbox: any = vm.createContext({ require });
 
   const obj: any = vm.runInContext(
     `
@@ -22,7 +23,7 @@ module.exports = {b, c, d, isImportDefaultSpecifier};
 });
 
 test("ImportDeclaration-2", t => {
-  const sandbox: any = vm.createContext({});
+  const sandbox: any = vm.createContext({ require });
 
   const outputParser: any = vm.runInContext(
     `
@@ -34,3 +35,35 @@ module.exports = parse;
   );
   t.true(outputParser === parse);
 });
+
+test("ImportDeclaration without require", t => {
+  // we didnot defined require function in the context
+  const sandbox: any = vm.createContext({});
+
+  t.throws(function() {
+    vm.runInContext(
+      `
+import {parse} from "babylon";
+
+module.exports = parse;
+    `,
+      sandbox
+    );
+  }, ErrNotDefined("require").message);
+});
+
+test("ImportDeclaration with null require", t => {
+  // we didnot defined require function in the context
+  const sandbox: any = vm.createContext({ require: null });
+
+  t.throws(function() {
+    vm.runInContext(
+      `
+import {parse} from "babylon";
+
+module.exports = parse;
+    `,
+      sandbox
+    );
+  }, ErrIsNotFunction("require").message);
+});