Fix requiring JSON arrays or other non-objects.

It was doing element-wise copy, but it should just be assigning the exports.

Author: domenic

lib/sandboxed_module.js

@@ -233,10 +233,7 @@ SandboxedModule.prototype._requireInterceptor = function() {
 
 SandboxedModule.prototype._compile = function() {
   if (/\.json$/.test(this.filename)){
-    var json = require(this.filename);
-    for( var key in json){
-      this.exports[key] = json[key];
-    }
+    this.module.exports = require(this.filename);
     return;
   }
   var compile = this._getCompileInfo();

test/fixture/includeJson.js

@@ -1,3 +1,2 @@
-var json =require('./json');
-
-module.exports.json = json;
+exports.json = require('./json');
+exports.jsonArray = require('./jsonArray.json');

test/fixture/jsonArray.json

@@ -0,0 +1 @@
+["foo", "bar"]

test/integration/test-recursive-json.js

@@ -5,3 +5,6 @@ var fakeBar = 'fakeBar';
 var requireModule = SandboxedModule.load('../fixture/includeJson');
 var recursiveExports = requireModule.exports;
 assert.strictEqual(recursiveExports.json.value, 'value from json');
+
+assert.deepEqual(recursiveExports.jsonArray, ['foo', 'bar']);
+assert.ok(Array.isArray(recursiveExports.jsonArray));