test(@angular/cli): remove unscoped authentication test cases from registry tests

Author: clydin

tests/legacy-cli/e2e/tests/commands/add/secure-registry.ts

@@ -7,19 +7,14 @@ import { expectToFail } from '../../../utils/utils';
 export default async function () {
   // The environment variable has priority over the .npmrc
   delete process.env['NPM_CONFIG_REGISTRY'];
+  delete process.env['YARN_REGISTRY'];
+  delete process.env['NPM_CONFIG__AUTH'];
+  delete process.env['NPM_CONFIG_ALWAYS_AUTH'];
   const isNpm = getActivePackageManager() === 'npm';
 
   const command = ['add', '@angular/pwa', '--skip-confirmation'];
   await expectFileNotToExist('public/manifest.webmanifest');
 
-  // Works with unscoped registry authentication details
-  if (!isNpm) {
-    // NPM no longer support unscoped.
-    await createNpmConfigForAuthentication(false);
-    await ng(...command);
-    await expectFileToExist('public/manifest.webmanifest');
-    await git('clean', '-dxf');
-  }
   // Works with scoped registry authentication details
   await expectFileNotToExist('public/manifest.webmanifest');
 
@@ -28,12 +23,6 @@ export default async function () {
   await expectFileToExist('public/manifest.webmanifest');
 
   // Invalid authentication token
-  if (isNpm) {
-    // NPM no longer support unscoped.
-    await createNpmConfigForAuthentication(false, true);
-    await expectToFail(() => ng(...command));
-  }
-
   await createNpmConfigForAuthentication(true, true);
   await expectToFail(() => ng(...command));
 

tests/legacy-cli/e2e/tests/commands/add/yarn-env-vars.ts

@@ -1,10 +1,10 @@
+import { readFile, writeFile } from 'node:fs/promises';
+import { join } from 'node:path';
+import { getGlobalVariable } from '../../../utils/env';
 import { expectFileNotToExist, expectFileToExist } from '../../../utils/fs';
 import { getActivePackageManager } from '../../../utils/packages';
 import { git, ng } from '../../../utils/process';
-import {
-  createNpmConfigForAuthentication,
-  setNpmEnvVarsForAuthentication,
-} from '../../../utils/registry';
+import { VALID_TOKEN } from '../../../utils/registry';
 
 export default async function () {
   // Yarn specific test that tests YARN_ env variables.
@@ -14,16 +14,39 @@ export default async function () {
   }
   const command = ['add', '@angular/pwa', '--skip-confirmation'];
 
-  // Environment variables only
-  await expectFileNotToExist('public/manifest.webmanifest');
-  setNpmEnvVarsForAuthentication(false, true);
-  await ng(...command);
-  await expectFileToExist('public/manifest.webmanifest');
-  await git('clean', '-dxf');
+  // Clean up any potential env vars first
+  delete process.env['NPM_CONFIG_REGISTRY'];
+  delete process.env['YARN_REGISTRY'];
+  delete process.env['NPM_CONFIG__AUTH'];
+  delete process.env['NPM_CONFIG_ALWAYS_AUTH'];
 
-  // Mix of config file and env vars works
   await expectFileNotToExist('public/manifest.webmanifest');
-  await createNpmConfigForAuthentication(false, true);
-  await ng(...command);
-  await expectFileToExist('public/manifest.webmanifest');
+
+  // Set the registry via YARN_REGISTRY environment variable
+  const registryUrl = getGlobalVariable('package-secure-registry') as string;
+  process.env['YARN_REGISTRY'] = registryUrl;
+
+  // Read the original user config to restore later
+  const tempRoot = getGlobalVariable('tmp-root') as string;
+  const userNpmrcPath = join(tempRoot, '.npmrc');
+  const originalUserNpmrc = await readFile(userNpmrcPath, 'utf8');
+
+  // Write the scoped auth credentials to the user config .npmrc
+  const registryHost = registryUrl.replace(/^\w+:/, '');
+  await writeFile(
+    userNpmrcPath,
+    originalUserNpmrc +
+      `\n${registryHost}/:_auth="${VALID_TOKEN}"` +
+      `\n${registryHost}/:always-auth=true` +
+      `\nalways-auth=true\n`,
+  );
+
+  try {
+    await ng(...command);
+    await expectFileToExist('public/manifest.webmanifest');
+  } finally {
+    // Clean up and restore
+    delete process.env['YARN_REGISTRY'];
+    await writeFile(userNpmrcPath, originalUserNpmrc);
+  }
 }

tests/legacy-cli/e2e/tests/update/update-secure-registry.ts

@@ -8,6 +8,9 @@ import assert from 'node:assert';
 export default async function () {
   // The environment variable has priority over the .npmrc
   delete process.env['NPM_CONFIG_REGISTRY'];
+  delete process.env['YARN_REGISTRY'];
+  delete process.env['NPM_CONFIG__AUTH'];
+  delete process.env['NPM_CONFIG_ALWAYS_AUTH'];
   const worksMessage = 'We analyzed your package.json';
 
   const extraArgs: string[] = [];
@@ -16,22 +19,13 @@ export default async function () {
   }
 
   // Valid authentication token
-  await createNpmConfigForAuthentication(false);
-  const { stdout: stdout1 } = await ng('update', ...extraArgs);
-  if (!stdout1.includes(worksMessage)) {
-    throw new Error(`Expected stdout to contain "${worksMessage}"`);
-  }
-
   await createNpmConfigForAuthentication(true);
   const { stdout: stdout2 } = await ng('update', ...extraArgs);
   if (!stdout2.includes(worksMessage)) {
     throw new Error(`Expected stdout to contain "${worksMessage}"`);
   }
 
   // Invalid authentication token
-  await createNpmConfigForAuthentication(false, true);
-  await expectToFail(() => ng('update', ...extraArgs));
-
   await createNpmConfigForAuthentication(true, true);
   await expectToFail(() => ng('update', ...extraArgs));
 

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

@@ -42,7 +42,7 @@ export async function createNpmRegistry(
 }
 
 // Token was generated using `echo -n 'testing:s3cret' | openssl base64`.
-const VALID_TOKEN = `dGVzdGluZzpzM2NyZXQ=`;
+export const VALID_TOKEN = `dGVzdGluZzpzM2NyZXQ=`;
 
 export function createNpmConfigForAuthentication(
   /**
@@ -69,7 +69,9 @@ export function createNpmConfigForAuthentication(
     '.npmrc',
     scopedAuthentication
       ? `
-        ${registry}:_auth="${token}"
+        ${registry}/:_auth="${token}"
+        ${registry}/:always-auth=true
+        always-auth=true
         registry=http:${registry}
       `
       : `