test_runner: fix --require with --experimental-loader

Author: MoLow

src/api/environment.cc

@@ -529,6 +529,9 @@ NODE_EXTERN std::unique_ptr<InspectorParentHandle> GetInspectorParentHandle(
   CHECK_NOT_NULL(env);
   if (name == nullptr) name = "";
   CHECK_NE(thread_id.id, static_cast<uint64_t>(-1));
+  if (!env->should_create_inspector()) {
+    return nullptr;
+  }
 #if HAVE_INSPECTOR
   return std::make_unique<InspectorParentHandleImpl>(
       env->inspector_agent()->GetParentHandle(thread_id.id, url, name));

test/fixtures/test-runner/esm/bootloader.js

@@ -0,0 +1,7 @@
+export function resolve(specifier, context, next) {
+  return next(specifier);
+}
+
+export function load(href, context, next) {
+  return next(href);
+}

test/fixtures/test-runner/esm/loader.mjs

@@ -185,3 +185,20 @@ const testFixtures = fixtures.path('test-runner');
   assert.match(stdout, /# tests 1/);
   assert.match(stdout, /# pass 1/);
 }
+
+{
+  // Use test with --loader and --require.
+  // This case is common since vscode uses --require to load the debugger.
+  const args = ['--no-warnings',
+                '--loader', join(testFixtures, 'esm', 'loader.mjs'),
+                '--test', join(testFixtures, 'index.test.js')];
+  const child = spawnSync(process.execPath, args, {
+    env: { NODE_OPTIONS: `--require="${join(testFixtures, 'esm', 'bootloader.js')}"` }
+  });
+
+  assert.strictEqual(child.stderr.toString(), '');
+  assert.strictEqual(child.status, 0);
+  assert.strictEqual(child.signal, null);
+  const stdout = child.stdout.toString();
+  assert.match(stdout, /ok 1 - this should pass/);
+}