ci: add bun package manager to e2e tests

alan-agius4 · alan-agius4 · commit 44c18a2ddc38 · 2025-11-19T10:24:32.000Z
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.
diff --git a/.github/workflows/ci.yml b/.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:
diff --git a/.github/workflows/pr.yml b/.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:
diff --git a/packages/angular/cli/src/utilities/package-manager.ts b/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 {
diff --git a/tests/legacy-cli/e2e/assets/BUILD.bazel b/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"],
 )
diff --git a/tests/legacy-cli/e2e/assets/add-collection-dir/collection.json b/tests/legacy-cli/e2e/assets/add-collection-dir/collection.json
diff --git a/tests/legacy-cli/e2e/assets/add-collection-dir/index.js b/tests/legacy-cli/e2e/assets/add-collection-dir/index.js
diff --git a/tests/legacy-cli/e2e/assets/add-collection-dir/package.json b/tests/legacy-cli/e2e/assets/add-collection-dir/package.json
diff --git a/tests/legacy-cli/e2e/setup/100-global-cli.ts b/tests/legacy-cli/e2e/setup/100-global-cli.ts
@@ -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 () {
diff --git a/tests/legacy-cli/e2e/tests/commands/add/dir.ts b/tests/legacy-cli/e2e/tests/commands/add/dir.ts
@@ -1,8 +1,25 @@
+import { cp } from 'node:fs/promises';
+import { resolve } from 'node:path';
 import { assetDir } from '../../../utils/assets';
 import { expectFileToExist } from '../../../utils/fs';
 import { ng } from '../../../utils/process';
+import { expectToFail } from '../../../utils/utils';
 
 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');
+
+  await ng('add', dirCollectionPath);
+  await expectFileToExist('empty-file');
+
+  await expectToFail(() => ng('add', dirCollectionPath)); // File already exists.
 }
diff --git a/tests/legacy-cli/e2e/tests/commands/add/peer.ts b/tests/legacy-cli/e2e/tests/commands/add/peer.ts
@@ -1,24 +1,45 @@
 import assert from 'node:assert/strict';
 import { assetDir } from '../../../utils/assets';
 import { ng } from '../../../utils/process';
+import { resolve } from 'node:path';
+import { cp } from 'node:fs/promises';
 
 const warning = 'Adding the package may not succeed.';
 
 export default async function () {
-  const { stdout: bad } = await ng(
-    'add',
-    assetDir('add-collection-peer-bad'),
-    '--skip-confirmation',
-  );
+  const peerBadCollectionName = 'add-collection-peer-bad';
+  const peerBadDirCollectionPath = resolve(peerBadCollectionName);
+
+  // Copy locally as bun doesn't install the dependency correctly if it has symlinks.
+  await cp(assetDir(peerBadCollectionName), peerBadDirCollectionPath, {
+    recursive: true,
+    dereference: true,
+  });
+
+  const { stdout: bad } = await ng('add', peerBadDirCollectionPath, '--skip-confirmation');
   assert.match(bad, new RegExp(warning), 'peer warning not shown on bad package');
 
-  const { stdout: base } = await ng('add', assetDir('add-collection'), '--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,
+  });
+
+  const { stdout: base } = await ng('add', dirCollectionPath, '--skip-confirmation');
   assert.doesNotMatch(base, new RegExp(warning), 'peer warning shown on base package');
 
-  const { stdout: good } = await ng(
-    'add',
-    assetDir('add-collection-peer-good'),
-    '--skip-confirmation',
-  );
+  const peerGoodCollectionName = 'add-collection-peer-good';
+  const peerGoodDirCollectionPath = resolve(peerGoodCollectionName);
+
+  // Copy locally as bun doesn't install the dependency correctly if it has symlinks.
+  await cp(assetDir(peerGoodCollectionName), peerGoodDirCollectionPath, {
+    recursive: true,
+    dereference: true,
+  });
+
+  const { stdout: good } = await ng('add', peerGoodDirCollectionPath, '--skip-confirmation');
   assert.doesNotMatch(good, new RegExp(warning), 'peer warning shown on good package');
 }
diff --git a/tests/legacy-cli/e2e/tests/commands/add/secure-registry.ts b/tests/legacy-cli/e2e/tests/commands/add/secure-registry.ts
@@ -1,25 +1,33 @@
 import { expectFileNotToExist, expectFileToExist } from '../../../utils/fs';
-import { getActivePackageManager, installWorkspacePackages } from '../../../utils/packages';
+import {
+  getActivePackageManager,
+  installWorkspacePackages,
+  uninstallPackage,
+} from '../../../utils/packages';
 import { git, ng } from '../../../utils/process';
 import { createNpmConfigForAuthentication } from '../../../utils/registry';
 import { expectToFail } from '../../../utils/utils';
 
 export default async function () {
+  await uninstallPackage('@angular/pwa');
+
   // 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 +36,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));
   }
