Move extension options config test into it's own folder

Author: bjornstar

lib/get-source-loader.mjs

@@ -1,4 +1,4 @@
-import { send } from './ipc.js';
+import { send } from './ipc.mjs';
 
 export async function getSource(url, context, defaultGetSource) {
   send({ required: new URL(url).pathname });

lib/index.js

@@ -101,6 +101,9 @@ module.exports = function (
         const resolveLoader = resolveMain(localPath('resolve-loader.mjs'));
         args.push('--experimental-modules', `--loader=${resolveLoader}`);
       } else if (semver.satisfies(process.version, '>=12.11.1')) {
+        if (semver.satisfies(process.version, '<12.17.0')) {
+          args.push('--experimental-modules');
+        }
         const loaderPath = semver.satisfies(process.version, '>=16.12.0')
           ? 'load-loader.mjs'
           : 'get-source-loader.mjs';

lib/ipc.mjs

@@ -0,0 +1,17 @@
+const cmd = 'NODE_DEV';
+
+export const send = m => {
+  if (process.connected) process.send({ ...m, cmd });
+};
+
+export const on = (src, prop, cb) => {
+  src.on('internalMessage', m => {
+    if (m.cmd === cmd && prop in m) cb(m);
+  });
+};
+
+export const relay = src => {
+  src.on('internalMessage', m => {
+    if (process.connected && m.cmd === cmd) process.send(m);
+  });
+};

test/fixture/.node-dev.json

@@ -2,12 +2,6 @@
   "notify": false,
   "ignore": ["./ignored-module.js"],
   "extensions": {
-    "js": {
-      "name": "./extension",
-      "options": {
-        "test": true
-      }
-    },
     "ts": "ts-node/register"
   }
 }

test/fixture/ecma-script-module-package/.node-dev.json

@@ -0,0 +1,10 @@
+{
+  "extensions": {
+    "js": {
+      "name": "./extension.js",
+      "options": {
+        "test": true
+      }
+    }
+  }
+}

test/fixture/extension-options/.node-dev.json

@@ -6,4 +6,5 @@ module.exports = function (options) {
   if (options.test !== true) {
     throw new Error('Expected options.test to be true');
   }
+  console.log(options);
 };

test/fixture/extension.js …t/fixture/extension-options/extension.jstest/fixture/extension.js renamed to test/fixture/extension-options/extension.js test/fixture/extension.js renamed to test/fixture/extension-options/extension.js

@@ -0,0 +1,6 @@
+const message = require('../message.js');
+
+console.log(message);
+
+// So it doesn't immediately exit.
+setTimeout(() => {}, 10000);

test/fixture/extension-options/index.js

@@ -0,0 +1,9 @@
+const tap = require('tap');
+
+const { spawn } = require('../utils');
+
+tap.test('should pass options to extensions according to .node-dev.json', t => {
+  spawn('extension-options', out => {
+    if (out.match(/\{ test: true \}/)) return { exit: t.end.bind(t) };
+  });
+});