Fix Windows test fails

Author: jdalton

src/shadow/npm-paths.ts

@@ -1,3 +1,5 @@
+import { existsSync } from 'node:fs'
+import Module from 'node:module'
 import path from 'node:path'
 import process from 'node:process'
 
@@ -81,18 +83,27 @@ export function getNpmPath() {
   return _npmPath
 }
 
-let _npmNmPath: string | undefined
-export function getNpmNodeModulesPath() {
-  if (_npmNmPath === undefined) {
-    _npmNmPath = path.join(getNpmPath(), NODE_MODULES)
+let _npmRequire: NodeJS.Require | undefined
+export function getNpmRequire(): NodeJS.Require {
+  if (_npmRequire === undefined) {
+    const npmPath = getNpmPath()
+    const npmNmPath = path.join(npmPath, NODE_MODULES, NPM)
+    _npmRequire = Module.createRequire(
+      path.join(existsSync(npmNmPath) ? npmNmPath : npmPath, '<dummy-basename>')
+    )
   }
-  return _npmNmPath
+  return _npmRequire
 }
 
 let _arboristPkgPath: string | undefined
 export function getArboristPackagePath() {
   if (_arboristPkgPath === undefined) {
-    _arboristPkgPath = path.join(getNpmNodeModulesPath(), '@npmcli/arborist')
+    const pkgName = '@npmcli/arborist'
+    const mainPath = getNpmRequire().resolve(pkgName)
+    _arboristPkgPath = mainPath.slice(
+      0,
+      mainPath.indexOf(pkgName) + pkgName.length
+    )
   }
   return _arboristPkgPath
 }

src/shadow/proc-log.ts

@@ -1,6 +1,4 @@
-import path from 'node:path'
-
-import { getNpmNodeModulesPath } from './npm-paths'
+import { getNpmRequire } from './npm-paths'
 import constants from '../constants'
 
 const { UNDEFINED_TOKEN } = constants
@@ -17,6 +15,7 @@ type RequireTransformer<T extends keyof RequireKnownModules> = (
 ) => RequireKnownModules[T]
 
 function tryRequire<T extends keyof RequireKnownModules>(
+  req: NodeJS.Require,
   ...ids: (T | [T, RequireTransformer<T>])[]
 ): RequireKnownModules[T] | undefined {
   for (const data of ids) {
@@ -32,7 +31,7 @@ function tryRequire<T extends keyof RequireKnownModules>(
     try {
       // Check that the transformed value isn't `undefined` because older
       // versions of packages like 'proc-log' may not export a `log` method.
-      const exported = transformer(require(id))
+      const exported = transformer(req(id))
       if (exported !== undefined) {
         return exported
       }
@@ -49,15 +48,15 @@ export type Logger =
 let _log: Logger | {} | undefined = UNDEFINED_TOKEN
 export function getLogger(): Logger {
   if (_log === UNDEFINED_TOKEN) {
-    const npmNmPath = getNpmNodeModulesPath()
     _log = tryRequire(
+      getNpmRequire(),
       [
-        <'proc-log'>path.join(npmNmPath, 'proc-log/lib/index.js'),
+        <'proc-log'>'proc-log/lib/index.js',
         // The proc-log DefinitelyTyped definition is incorrect. The type definition
         // is really that of its export log.
         mod => <RequireKnownModules['proc-log']>(mod as any).log
       ],
-      <'npmlog'>path.join(npmNmPath, 'npmlog/lib/log.js')
+      <'npmlog'>'npmlog/lib/log.js'
     )
   }
   return <Logger | undefined>_log