ci: add bun package manager to e2e tests

This commit introduces support for the `bun` package manager in our end-to-end tests.

The following changes are included:
- The `bun` package manager is now part of the test matrix in the CI configuration.
- The package manager flags for `bun` have been updated to align with the latest version.
- The `ng add` command has been updated to support `bun`, including a workaround for a symlink issue.
- Tests that are not compatible with `bun`, such as those for unscoped authentication, are now skipped when `bun` the active package manager.

Author: alan-agius4

.github/workflows/ci.yml

@@ -164,7 +164,7 @@ jobs:
         # flaky targets are retried. The larger machine type comes with 2x more SSD space.
         os: [ubuntu-latest-4core]
         node: [22]
-        subset: [yarn, pnpm]
+        subset: [yarn, pnpm, bun]
         shard: [0, 1, 2]
     runs-on: ${{ matrix.os }}
     steps:

.github/workflows/pr.yml

@@ -182,7 +182,7 @@ jobs:
         # flaky targets are retried. The larger machine type comes with 2x more SSD space.
         os: [ubuntu-latest-4core]
         node: [22]
-        subset: [yarn, pnpm]
+        subset: [yarn, pnpm, bun]
         shard: [0, 1, 2]
     runs-on: ${{ matrix.os }}
     steps:

packages/angular/cli/src/utilities/package-manager.ts

@@ -158,11 +158,11 @@ export class PackageManagerUtils {
         };
       case PackageManager.Bun:
         return {
-          saveDev: '--development',
+          saveDev: '--dev',
           install: 'add',
           installAll: 'install',
           prefix: '--cwd',
-          noLockfile: '',
+          noLockfile: '--no-save',
         };
       default:
         return {

tests/legacy-cli/e2e/assets/BUILD.bazel

@@ -2,6 +2,9 @@ load("//tools:defaults.bzl", "copy_to_bin")
 
 copy_to_bin(
     name = "assets",
-    srcs = glob(["**"]),
+    srcs = glob(
+        include = ["**"],
+        exclude = ["BUILD.bazel"],
+    ),
     visibility = ["//visibility:public"],
 )

tests/legacy-cli/e2e/assets/add-collection/collection.json renamed to tests/legacy-cli/e2e/assets/add-collection-dir/collection.json

@@ -6,7 +6,7 @@ const PACKAGE_MANAGER_VERSION = {
   'npm': '10.8.1',
   'yarn': '1.22.22',
   'pnpm': '10.17.1',
-  'bun': '1.2.21',
+  'bun': '1.3.2',
 };
 
 export default async function () {

tests/legacy-cli/e2e/assets/add-collection/index.js renamed to tests/legacy-cli/e2e/assets/add-collection-dir/index.js

@@ -1,8 +1,19 @@
+import { cp } from 'fs/promises';
 import { assetDir } from '../../../utils/assets';
 import { expectFileToExist } from '../../../utils/fs';
 import { ng } from '../../../utils/process';
+import { resolve } from 'path';
 
 export default async function () {
-  await ng('add', assetDir('add-collection'), '--name=blah', '--skip-confirmation');
+  const collectionName = 'add-collection-dir';
+  const dirCollectionPath = resolve(collectionName);
+
+  // Copy locally as bun doesn't install the dependency correctly if it has symlinks.
+  await cp(assetDir(collectionName), dirCollectionPath, {
+    recursive: true,
+    dereference: true,
+  });
+
+  await ng('add', dirCollectionPath, '--name=blah', '--skip-confirmation');
   await expectFileToExist('blah');
 }

tests/legacy-cli/e2e/assets/add-collection/package.json renamed to tests/legacy-cli/e2e/assets/add-collection-dir/package.json

@@ -7,19 +7,21 @@ import { expectToFail } from '../../../utils/utils';
 export default async function () {
   // The environment variable has priority over the .npmrc
   delete process.env['NPM_CONFIG_REGISTRY'];
-  const isNpm = getActivePackageManager() === 'npm';
+  const packageManager = getActivePackageManager();
+  const supportsUnscopedAuth = packageManager !== 'bun' && packageManager !== '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.
+  if (supportsUnscopedAuth) {
+    // Some package managers such as Bun and NPM do not support unscoped auth.
     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,8 +30,8 @@ export default async function () {
   await expectFileToExist('public/manifest.webmanifest');
 
   // Invalid authentication token
-  if (isNpm) {
-    // NPM no longer support unscoped.
+  if (!supportsUnscopedAuth) {
+    // Some package managers such as Bun and NPM do not support unscoped auth.
     await createNpmConfigForAuthentication(false, true);
     await expectToFail(() => ng(...command));
   }