Don't break strict mode! Fixes #17.

felixge · Feb 12, 2013 · a543553 · a543553
1 parent e4355ba
commit a543553
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 1 deletion.
diff --git a/lib/sandboxed_module.js b/lib/sandboxed_module.js
@@ -141,8 +141,8 @@ SandboxedModule.prototype._getCompileInfo = function() {
   }
 
   var source =
-    '(function(' + localVariables.join(', ')  + ') { ' +
     'global = GLOBAL = root = (function() { return this; })();' +
+    '(function(' + localVariables.join(', ')  + ') { ' +
     sourceToWrap +
     '\n' +
     'return global;\n' +

diff --git a/test/fixture/stayStrict.js b/test/fixture/stayStrict.js
@@ -0,0 +1,5 @@
+'use strict';
+
+module.exports = function () {
+  return this;
+};
diff --git a/test/integration/test-stay-strict.js b/test/integration/test-stay-strict.js
@@ -0,0 +1,9 @@
+var common = require('../common');
+var assert = common.assert;
+var SandboxedModule = require(common.dir.lib + '/sandboxed_module');
+
+(function testStayStrict() {
+  var exports = SandboxedModule.load(common.dir.fixture + '/stayStrict').exports;
+
+  assert(exports() === undefined);
+})();