From 15f0f77322bb5291d6611ebfb6e7f3327f7782ae Mon Sep 17 00:00:00 2001
From: Charles Lyding <19598772+clydin@users.noreply.github.com>
Date: Fri, 10 Jul 2026 14:00:10 -0400
Subject: [PATCH] test(@angular/cli): verify package group locking under peer
 updates

Asserts that the update resolver loops correctly prioritize package group expansions before evaluating peer dependencies.

This prevents regression of issue #22654, where specifying a package with a wide pre-release peer range (like @angular/cdk) before a package defining a group (like @angular/core) would resolve group dependencies (like @angular/common) to next-major pre-releases instead of locking them to target group versions.
---
 .../commands/update/update-resolver_spec.ts   | 83 +++++++++++++++++++
 1 file changed, 83 insertions(+)

diff --git a/packages/angular/cli/src/commands/update/update-resolver_spec.ts b/packages/angular/cli/src/commands/update/update-resolver_spec.ts
index a309830156cf..9cf4a4d826e1 100644
--- a/packages/angular/cli/src/commands/update/update-resolver_spec.ts
+++ b/packages/angular/cli/src/commands/update/update-resolver_spec.ts
@@ -172,6 +172,62 @@ describe('UpdateResolver', () => {
         '1.2.0': { name: '@angular-devkit-tests/update-release-age', version: '1.2.0' },
       },
     },
+    '@angular-devkit-tests/core-bug': {
+      metadata: {
+        name: '@angular-devkit-tests/core-bug',
+        'dist-tags': { latest: '14.0.0-next.0' },
+        versions: ['12.2.0', '13.2.0', '14.0.0-next.0'],
+      },
+      manifests: {
+        '12.2.0': { name: '@angular-devkit-tests/core-bug', version: '12.2.0' },
+        '13.2.0': {
+          name: '@angular-devkit-tests/core-bug',
+          version: '13.2.0',
+          'ng-update': {
+            packageGroup: ['@angular-devkit-tests/core-bug', '@angular-devkit-tests/common-bug'],
+          },
+        },
+        '14.0.0-next.0': { name: '@angular-devkit-tests/core-bug', version: '14.0.0-next.0' },
+      },
+    },
+    '@angular-devkit-tests/common-bug': {
+      metadata: {
+        name: '@angular-devkit-tests/common-bug',
+        'dist-tags': { latest: '14.0.0-next.0' },
+        versions: ['12.2.0', '13.2.0', '14.0.0-next.0'],
+      },
+      manifests: {
+        '12.2.0': { name: '@angular-devkit-tests/common-bug', version: '12.2.0' },
+        '13.2.0': { name: '@angular-devkit-tests/common-bug', version: '13.2.0' },
+        '14.0.0-next.0': { name: '@angular-devkit-tests/common-bug', version: '14.0.0-next.0' },
+      },
+    },
+    '@angular-devkit-tests/cdk-bug': {
+      metadata: {
+        name: '@angular-devkit-tests/cdk-bug',
+        'dist-tags': { latest: '14.0.0-next.0' },
+        versions: ['12.2.0', '13.2.0', '14.0.0-next.0'],
+      },
+      manifests: {
+        '12.2.0': {
+          name: '@angular-devkit-tests/cdk-bug',
+          version: '12.2.0',
+          peerDependencies: {
+            '@angular-devkit-tests/core-bug': '^12.0.0',
+            '@angular-devkit-tests/common-bug': '^12.0.0',
+          },
+        },
+        '13.2.0': {
+          name: '@angular-devkit-tests/cdk-bug',
+          version: '13.2.0',
+          peerDependencies: {
+            '@angular-devkit-tests/core-bug': '^13.0.0 || ^14.0.0-0',
+            '@angular-devkit-tests/common-bug': '^13.0.0 || ^14.0.0-0',
+          },
+        },
+        '14.0.0-next.0': { name: '@angular-devkit-tests/cdk-bug', version: '14.0.0-next.0' },
+      },
+    },
   };
 
   async function resolvePlan(options: UpdateResolverOptions, minReleaseAge = 0) {
@@ -391,6 +447,33 @@ describe('UpdateResolver', () => {
       '1.1.0',
     );
   });
+
+  it('correctly keeps packageGroup version when updating alongside peer dependencies', async () => {
+    createMockWorkspace(
+      {
+        name: 'blah',
+        dependencies: {
+          '@angular-devkit-tests/core-bug': '12.2.0',
+          '@angular-devkit-tests/common-bug': '12.2.0',
+          '@angular-devkit-tests/cdk-bug': '12.2.0',
+        },
+      },
+      {
+        '@angular-devkit-tests/core-bug': { version: '12.2.0' },
+        '@angular-devkit-tests/common-bug': { version: '12.2.0' },
+        '@angular-devkit-tests/cdk-bug': { version: '12.2.0' },
+      },
+    );
+
+    const plan = await resolvePlan({
+      packages: ['@angular-devkit-tests/cdk-bug@13', '@angular-devkit-tests/core-bug@13'],
+      workspaceRoot: tempRoot,
+    });
+
+    expect(plan.packagesToUpdate.get('@angular-devkit-tests/core-bug')).toBe('13.2.0');
+    expect(plan.packagesToUpdate.get('@angular-devkit-tests/common-bug')).toBe('13.2.0');
+    expect(plan.packagesToUpdate.get('@angular-devkit-tests/cdk-bug')).toBe('13.2.0');
+  });
 });
 
 describe('RegistryClient', () => {