test: refactor test-runner-module-mocking

PR-URL: #54233
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

Author: aduh95

test/parallel/test-runner-module-mocking.js

@@ -10,7 +10,7 @@ const fixtures = require('../common/fixtures');
 constassert=require('node:assert');
 const{ relative }=require('node:path');
 const{ test }=require('node:test');
-const{ pathToFileURL }=require('node:url');
+const{fileURLToPath,pathToFileURL }=require('node:url');
 
 test('input validation',async(t)=>{
 awaitt.test('throws if specifier is not a string',(t)=>{
@@ -514,41 +514,21 @@ test('CJS mocks can be used by both module systems', async (t) => {
 constcjsMock=t.mock.module(cjsFixture,{
 namedExports: {fn(){return42;}},
 });
-letesmImpl=awaitimport(cjsFixture);
+letesmImpl=awaitimport(pathToFileURL(cjsFixture));
 letcjsImpl=require(cjsFixture);
 
 assert.strictEqual(esmImpl.fn(),42);
 assert.strictEqual(cjsImpl.fn(),42);
 
 cjsMock.restore();
 
-esmImpl=awaitimport(cjsFixture);
+esmImpl=awaitimport(pathToFileURL(cjsFixture));
 cjsImpl=require(cjsFixture);
 
 assert.strictEqual(esmImpl.default.string,'original cjs string');
 assert.strictEqual(cjsImpl.string,'original cjs string');
 });
 
-test('ESM mocks can be used by both module systems',async(t)=>{
-constesmFixture=fixtures.path('module-mocking','basic-esm.mjs');
-constesmMock=t.mock.module(esmFixture,{
-namedExports: {fn(){return42;}},
-});
-
-letcjsImpl=require(esmFixture);
-letesmImpl=awaitimport(esmFixture);
-
-assert.strictEqual(cjsImpl.fn(),42);
-assert.strictEqual(esmImpl.fn(),42);
-
-esmMock.restore();
-cjsImpl=require(esmFixture);
-esmImpl=awaitimport(esmFixture);
-
-assert.strictEqual(esmImpl.string,'original esm string');
-assert.strictEqual(cjsImpl.string,'original esm string');
-});
-
 test('relative paths can be used by both module systems',async(t)=>{
 constfixture=relative(
 __dirname,fixtures.path('module-mocking','basic-esm.mjs')
@@ -586,9 +566,7 @@ test('node_modules can be used by both module systems', async (t) => {
 });
 
 test('file:// imports are supported in ESM only',async(t)=>{
-constfixture=pathToFileURL(
-fixtures.path('module-mocking','basic-esm.mjs')
-).href;
+constfixture=fixtures.fileURL('module-mocking','basic-esm.mjs').href;
 constmock=t.mock.module(fixture,{
 namedExports: {fn(){return42;}},
 });
@@ -604,9 +582,9 @@ test('file:// imports are supported in ESM only', async (t) => {
 });
 
 test('mocked modules do not impact unmocked modules',async(t)=>{
-constmockedFixture=fixtures.path('module-mocking','basic-cjs.js');
-constunmockedFixture=fixtures.path('module-mocking','basic-esm.mjs');
-t.mock.module(mockedFixture,{
+constmockedFixture=fixtures.fileURL('module-mocking','basic-cjs.js');
+constunmockedFixture=fixtures.fileURL('module-mocking','basic-esm.mjs');
+t.mock.module(`${mockedFixture}`,{
 namedExports: {fn(){return42;}},
 });
 constmockedImpl=awaitimport(mockedFixture);
@@ -625,18 +603,18 @@ test('defaultExports work with CJS mocks in both module systems', async (t) => {
 assert.strictEqual(original.string,'original cjs string');
 t.mock.module(fixture,{ defaultExport });
 assert.strictEqual(require(fixture),defaultExport);
-assert.strictEqual((awaitimport(fixture)).default,defaultExport);
+assert.strictEqual((awaitimport(pathToFileURL(fixture))).default,defaultExport);
 });
 
 test('defaultExports work with ESM mocks in both module systems',async(t)=>{
-constfixture=fixtures.path('module-mocking','basic-esm.mjs');
+constfixture=fixtures.fileURL('module-mocking','basic-esm.mjs');
 constoriginal=awaitimport(fixture);
 constdefaultExport=Symbol('default');
 
 assert.strictEqual(original.string,'original esm string');
-t.mock.module(fixture,{ defaultExport });
+t.mock.module(`${fixture}`,{ defaultExport });
 assert.strictEqual((awaitimport(fixture)).default,defaultExport);
-assert.strictEqual(require(fixture),defaultExport);
+assert.strictEqual(require(fileURLToPath(fixture)),defaultExport);
 });
 
 test('wrong import syntax should throw error after module mocking.',async()=>{