test: ensure private npm is always on PATH

Author: jbedard

tests/legacy-cli/e2e/utils/process.ts

@@ -33,11 +33,19 @@ function _exec(options: ExecOptions, cmd: string, args: string[]): Promise<Proce
   let stdout = '';
   let stderr = '';
   const cwd = options.cwd ?? process.cwd();
-  const env = options.env;
+  const env = options.env ?? process.env;
   console.log(
     `==========================================================================================`,
   );
 
+  // Ensure the custom npm and yarn global bin is on the PATH
+  // https://docs.npmjs.com/cli/v8/configuring-npm/folders#executables
+  const paths = [
+    join(getGlobalVariable('yarn-global'), 'bin'),
+    join(getGlobalVariable('npm-global'), process.platform.startsWith('win') ? 'bin' : ''),
+    env.PATH || process.env['PATH'],
+  ].join(delimiter);
+
   args = args.filter((x) => x !== undefined);
   const flags = [
     options.silent && 'silent',
@@ -52,7 +60,7 @@ function _exec(options: ExecOptions, cmd: string, args: string[]): Promise<Proce
 
   const spawnOptions: SpawnOptions = {
     cwd,
-    ...(env ? { env } : {}),
+    env: { ...env, PATH: paths },
   };
 
   if (process.platform.startsWith('win')) {
@@ -147,15 +155,10 @@ function _exec(options: ExecOptions, cmd: string, args: string[]): Promise<Proce
 export function extractNpmEnv() {
   return Object.keys(process.env)
     .filter((v) => NPM_CONFIG_RE.test(v))
-    .reduce<NodeJS.ProcessEnv>(
-      (vars, n) => {
-        vars[n] = process.env[n];
-        return vars;
-      },
-      {
-        PATH: process.env.PATH,
-      },
-    );
+    .reduce<NodeJS.ProcessEnv>((vars, n) => {
+      vars[n] = process.env[n];
+      return vars;
+    }, {});
 }
 
 export function waitForAnyProcessOutputToMatch(
@@ -355,20 +358,12 @@ export async function launchTestProcess(entry: string, ...args: any[]) {
   };
 
   // Modify the PATH environment variable...
-  let paths = process.env.PATH!.split(delimiter);
+  let paths = (env.PATH || process.env.PATH)!.split(delimiter);
 
   // Only include paths within the sandboxed test environment or external
   // non angular-cli paths such as /usr/bin for generic commands.
   paths = paths.filter((p) => p.startsWith(tempRoot) || !p.includes('angular-cli'));
 
-  // Ensure the custom npm global bin is on the PATH
-  // https://docs.npmjs.com/cli/v8/configuring-npm/folders#executables
-  if (process.platform.startsWith('win')) {
-    paths.unshift(env.NPM_CONFIG_PREFIX!);
-  } else {
-    paths.unshift(join(env.NPM_CONFIG_PREFIX!, 'bin'));
-  }
-
   env.PATH = paths.join(delimiter);
 
   return _exec({ env }, process.execPath, [resolve(__dirname, 'run_test_process'), entry, ...args]);