repl: default useGlobal to true

This is a partial revert of
15157c3. This change lead to a
regression that broke require() in the CLI REPL, as imported
files were evaluated in a different context.

Refs: #5703
Fixes: #7788
PR-URL: #7795
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>

Author: cjihrig

lib/internal/repl.js

@@ -22,7 +22,7 @@ function createRepl(env, opts, cb) {
   opts = opts || {
     ignoreUndefined: false,
     terminal: process.stdout.isTTY,
-    useGlobal: false,
+    useGlobal: true,
     breakEvalOnSigint: true
   };
 

test/parallel/test-repl-require-context.js

@@ -0,0 +1,28 @@
+'use strict';
+const common = require('../common');
+const assert = require('assert');
+const cp = require('child_process');
+const path = require('path');
+const child = cp.spawn(process.execPath, ['--interactive']);
+const fixture = path.join(common.fixturesDir, 'is-object.js').replace(/\\/g,
+                                                                      '/');
+let output = '';
+
+child.stdout.setEncoding('utf8');
+child.stdout.on('data', (data) => {
+  output += data;
+});
+
+child.on('exit', common.mustCall(() => {
+  const results = output.split('\n').map((line) => {
+    return line.replace(/\w*>\w*/, '').trim();
+  });
+
+  assert.deepStrictEqual(results, ['undefined', 'true', 'true', '']);
+}));
+
+child.stdin.write('const isObject = (obj) => obj.constructor === Object;\n');
+child.stdin.write('isObject({});\n');
+child.stdin.write(`require('${fixture}').isObject({});\n`);
+child.stdin.write('.exit');
+child.stdin.end();