fixup! fixup! module: run require.resolve through module.registerHooks()

Author: joyeecheung

test/module-hooks/test-module-hooks-resolve-require-resolve-consistency.js

@@ -7,6 +7,7 @@ require('../common');
 const assert = require('assert');
 const { registerHooks } = require('module');
 const fixtures = require('../common/fixtures');
+const { pathToFileURL } = require('url');
 
 const redirectedPath = fixtures.path('module-hooks', 'redirected-assert.js');
 
@@ -16,7 +17,7 @@ const hook = registerHooks({
     if (specifier === 'test-consistency-target') {
       resolvedSpecifiers.push(specifier);
       return {
-        url: `file://${redirectedPath}`,
+        url: pathToFileURL(redirectedPath).href,
         shortCircuit: true,
       };
     }

test/module-hooks/test-module-hooks-resolve-require-resolve-create-require.js

@@ -8,14 +8,15 @@ require('../common');
 const assert = require('assert');
 const { registerHooks, createRequire } = require('module');
 const fixtures = require('../common/fixtures');
+const { pathToFileURL } = require('url');
 
 const redirectedPath = fixtures.path('module-hooks', 'redirected-assert.js');
 
 const hook = registerHooks({
   resolve(specifier, context, nextResolve) {
     if (specifier === 'test-create-require-resolve-target') {
       return {
-        url: `file://${redirectedPath}`,
+        url: pathToFileURL(redirectedPath).href,
         shortCircuit: true,
       };
     }

test/module-hooks/test-module-hooks-resolve-require-resolve-imported-cjs.js

@@ -7,14 +7,15 @@ const common = require('../common');
 const assert = require('assert');
 const { registerHooks } = require('module');
 const fixtures = require('../common/fixtures');
+const { pathToFileURL } = require('url');
 
 const redirectedPath = fixtures.path('module-hooks', 'redirected-assert.js');
 
 const hook = registerHooks({
   resolve: common.mustCall((specifier, context, nextResolve) => {
     if (specifier === 'test-require-resolve-hook-target') {
       return {
-        url: `file://${redirectedPath}`,
+        url: pathToFileURL(redirectedPath).href,
         shortCircuit: true,
       };
     }

test/module-hooks/test-module-hooks-resolve-require-resolve-loaded-with-source.js

@@ -9,6 +9,7 @@ const common = require('../common');
 const assert = require('assert');
 const { register, registerHooks } = require('module');
 const fixtures = require('../common/fixtures');
+const { pathToFileURL } = require('url');
 
 const redirectedPath = fixtures.path('module-hooks', 'redirected-assert.js');
 
@@ -20,7 +21,7 @@ const hook = registerHooks({
   resolve: common.mustCall((specifier, context, nextResolve) => {
     if (specifier === 'test-require-resolve-hook-target') {
       return {
-        url: `file://${redirectedPath}`,
+        url: pathToFileURL(redirectedPath).href,
         shortCircuit: true,
       };
     }

test/module-hooks/test-module-hooks-resolve-require-resolve-redirect.js

@@ -7,14 +7,15 @@ require('../common');
 const assert = require('assert');
 const { registerHooks } = require('module');
 const fixtures = require('../common/fixtures');
+const { pathToFileURL } = require('url');
 
 const redirectedPath = fixtures.path('module-hooks', 'redirected-assert.js');
 
 const hook = registerHooks({
   resolve(specifier, context, nextResolve) {
     if (specifier === 'test-resolve-target') {
       return {
-        url: `file://${redirectedPath}`,
+        url: pathToFileURL(redirectedPath).href,
         shortCircuit: true,
       };
     }