test: remove use of global npm cache and config

Author: jbedard

tests/legacy-cli/e2e/setup/200-create-tmp-dir.ts renamed to tests/legacy-cli/e2e/setup/001-create-tmp-dir.ts

@@ -17,5 +17,4 @@ export default function () {
   }
   console.log(`  Using "${tempRoot}" as temporary directory for a new project.`);
   setGlobalVariable('tmp-root', tempRoot);
-  process.chdir(tempRoot);
 }

tests/legacy-cli/e2e/setup/002-npm-sandbox.ts

@@ -0,0 +1,26 @@
+import { delimiter, join } from 'path';
+import { getGlobalVariable } from '../utils/env';
+
+/**
+ * Configure npm to use a unique sandboxed environment.
+ */
+export default async function () {
+  const tempRoot: string = getGlobalVariable('tmp-root');
+  const npmModulesPrefix = join(tempRoot, 'npm-global');
+  const npmCache = join(tempRoot, 'npm-cache');
+  const npmrc = join(tempRoot, '.npmrc');
+
+  // Configure npm to use the sandboxed npm globals, cache and rc file
+  process.env.NPM_CONFIG_USERCONFIG = npmrc;
+  process.env.NPM_CONFIG_CACHE = npmCache;
+  process.env.NPM_CONFIG_PREFIX = npmModulesPrefix;
+
+  // Ensure the custom npm global bin is first on the PATH
+  // https://docs.npmjs.com/cli/v8/configuring-npm/folders#executables
+  process.env.PATH = join(npmModulesPrefix, 'bin') + delimiter + process.env.PATH;
+
+  // The windows bin is the same as the `prefix`
+  process.env.PATH = npmModulesPrefix + delimiter + process.env;
+
+  console.log(`  Using "${npmModulesPrefix}" as an isolated global npm cache.`);
+}

tests/legacy-cli/e2e/setup/200-create-project-dir.ts

@@ -0,0 +1,15 @@
+import { mkdir } from 'fs/promises';
+import { join } from 'path';
+import { getGlobalVariable, setGlobalVariable } from '../utils/env';
+
+export default async function () {
+  const tempRoot: string = getGlobalVariable('tmp-root');
+  const projectsRoot = join(tempRoot, 'e2e-test');
+
+  setGlobalVariable('projects-root', projectsRoot);
+
+  await mkdir(projectsRoot);
+
+  console.log(`  Using "${projectsRoot}" as temporary directory for a new project.`);
+  process.chdir(projectsRoot);
+}

tests/legacy-cli/e2e/setup/500-create-project.ts

@@ -1,9 +1,9 @@
 import { join } from 'path';
 import { getGlobalVariable } from '../utils/env';
-import { expectFileToExist, writeFile } from '../utils/fs';
+import { expectFileToExist } from '../utils/fs';
 import { gitClean } from '../utils/git';
-import { setRegistry as setNPMConfigRegistry } from '../utils/packages';
-import { ng, npm } from '../utils/process';
+import { setRegistry as setNPMConfigRegistry, setRegistry } from '../utils/packages';
+import { ng } from '../utils/process';
 import { prepareProjectForE2e, updateJsonFile } from '../utils/project';
 
 export default async function () {
@@ -18,8 +18,6 @@ export default async function () {
     await gitClean();
   } else {
     const extraArgs = [];
-    const testRegistry = getGlobalVariable('package-registry');
-    const isCI = getGlobalVariable('ci');
 
     // Ensure local test registry is used when outside a project
     await setNPMConfigRegistry(true);
@@ -28,11 +26,7 @@ export default async function () {
     await expectFileToExist(join(process.cwd(), 'test-project'));
     process.chdir('./test-project');
 
-    // If on CI, the user configuration set above will handle project usage
-    if (!isCI) {
-      // Ensure local test registry is used inside a project
-      await writeFile('.npmrc', `registry=${testRegistry}`);
-    }
+    setRegistry(true);
 
     // Setup esbuild builder if requested on the commandline
     const useEsbuildBuilder = !!getGlobalVariable('argv')['esbuild'];

tests/legacy-cli/e2e/tests/misc/update-git-clean-subdirectory.ts

@@ -4,7 +4,7 @@ import { ng, silentGit } from '../../utils/process';
 import { prepareProjectForE2e } from '../../utils/project';
 
 export default async function () {
-  process.chdir(getGlobalVariable('tmp-root'));
+  process.chdir(getGlobalVariable('projects-root'));
 
   await createDir('./subdirectory');
   process.chdir('./subdirectory');

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

@@ -11,7 +11,7 @@ export function assetDir(assetName: string) {
 }
 
 export function copyProjectAsset(assetName: string, to?: string) {
-  const tempRoot = join(getGlobalVariable('tmp-root'), 'test-project');
+  const tempRoot = join(getGlobalVariable('projects-root'), 'test-project');
   const sourcePath = assetDir(assetName);
   const targetPath = join(tempRoot, to || assetName);
 
@@ -20,7 +20,7 @@ export function copyProjectAsset(assetName: string, to?: string) {
 
 export function copyAssets(assetName: string, to?: string) {
   const seed = +Date.now();
-  const tempRoot = join(getGlobalVariable('tmp-root'), 'assets', assetName + '-' + seed);
+  const tempRoot = join(getGlobalVariable('projects-root'), 'assets', assetName + '-' + seed);
   const root = assetDir(assetName);
 
   return Promise.resolve()
@@ -30,7 +30,7 @@ export function copyAssets(assetName: string, to?: string) {
       return allFiles.reduce((promise, filePath) => {
         const toPath =
           to !== undefined
-            ? resolve(getGlobalVariable('tmp-root'), 'test-project', to, filePath)
+            ? resolve(getGlobalVariable('projects-root'), 'test-project', to, filePath)
             : join(tempRoot, filePath);
 
         return promise.then(() => copyFile(join(root, filePath), toPath));

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

@@ -10,11 +10,27 @@ const treeKill = require('tree-kill');
 interface ExecOptions {
   silent?: boolean;
   waitForMatch?: RegExp;
-  env?: { [varname: string]: string };
+  env?: { [varname: string]: string } | 'npm';
   stdin?: string;
   cwd?: string;
 }
 
+const NPM_CONFIG_RE = /^npm_config_/i;
+
+function extractNpmEnv() {
+  return Object.keys(process.env)
+    .filter((v) => NPM_CONFIG_RE.test(v))
+    .reduce(
+      (vars, n) => {
+        vars[n] = process.env[n];
+        return vars;
+      },
+      {
+        PATH: process.env.PATH,
+      },
+    );
+}
+
 let _processes: child_process.ChildProcess[] = [];
 
 export type ProcessOutput = {
@@ -30,7 +46,7 @@ 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 === 'npm' ? extractNpmEnv() : options.env;
   console.log(
     `==========================================================================================`,
   );
@@ -269,21 +285,22 @@ export function silentNpm(
       {
         silent: true,
         cwd: (options as { cwd?: string } | undefined)?.cwd,
+        env: 'npm',
       },
       'npm',
       params,
     );
   } else {
-    return _exec({ silent: true }, 'npm', args as string[]);
+    return _exec({ silent: true, env: 'npm' }, 'npm', args as string[]);
   }
 }
 
 export function silentYarn(...args: string[]) {
-  return _exec({ silent: true }, 'yarn', args);
+  return _exec({ silent: true, env: 'npm' }, 'yarn', args);
 }
 
 export function npm(...args: string[]) {
-  return _exec({}, 'npm', args);
+  return _exec({ env: 'npm' }, 'npm', args);
 }
 
 export function node(...args: string[]) {