Fix tests that did not exercise their intended behavior

- managerCapabilities.integration.test.ts: the delegation test queried
  packages.manage.upgrade (undeclared) instead of packages.manage.install
  (the key that actually delegates to packages.manage), so it silently
  passed by coincidence rather than exercising cross-module delegation.
  The cycle test declared packages.manage -> packages.manage.install but
  never wired the reverse edge, so no cycle existed to detect.
- api/test/consumer.ts: removed a duplicate @ts-expect-error assertion
  for 'environments.create.named', which is rejected the same way as any
  other non-catalog string literal and is already covered by the generic
  arbitraryKey assertion immediately below it.
- packageManagerHeadlessConformance.unit.test.ts: fixed two import
  statements that had been merged onto a single line.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: edvilme

api/test/consumer.ts

@@ -123,9 +123,6 @@ if (!support.supported) {
 ({ 'packages.unknown': async () => ({ supported: true }) } satisfies Capabilities<PackageManagerCapability>);
 // @ts-expect-error Operation options are not capability query context.
 ({ createOptions: { quickCreate: true } } satisfies CapabilityContext);
-// @ts-expect-error Named creation is deferred rather than an enabled capability.
-resolveEnvironmentManagerCapability(legacyEnvironmentManager, 'environments.create.named');
-
 declare const arbitraryKey: string;
 // @ts-expect-error A general string is not a declared environment capability.
 resolveEnvironmentManagerCapability(legacyEnvironmentManager, arbitraryKey);

src/test/integration/managerCapabilities.integration.test.ts

@@ -160,14 +160,16 @@ suite('Manager capabilities integration', function () {
             'packages.manage.install': (context) =>
                 resolvePackageManagerCapability(packages, 'packages.manage', context),
         };
-        assert.deepStrictEqual(await api.getPackageManagerCapability(environment, 'packages.manage.upgrade'), denied);
+        assert.deepStrictEqual(await api.getPackageManagerCapability(environment, 'packages.manage.install'), denied);
     });
 
     test('prerequisite cycles are detected across separately loaded API modules', async () => {
         packageCapabilities = {
             ...noPackageCapabilities,
             'packages.manage': (context) =>
                 resolvePackageManagerCapability(packages, 'packages.manage.install', context),
+            'packages.manage.install': (context) =>
+                resolvePackageManagerCapability(packages, 'packages.manage', context),
         };
         await assert.rejects(
             () => api.getPackageManagerCapability(environment, 'packages.manage'),

src/test/managers/common/packageManagerHeadlessConformance.unit.test.ts

@@ -16,7 +16,8 @@ import * as windowApis from '../../../common/window.apis';
 import * as workspaceFs from '../../../common/workspace.fs.apis';
 import * as workspaceApis from '../../../common/workspace.apis';
 import { InternalPackageManager } from '../../../managers/common/registeredManagers';
-import { PipInstallCommand } from '../../../managers/builtin/commands/install';import { PipListCommand } from '../../../managers/builtin/commands/list';
+import { PipInstallCommand } from '../../../managers/builtin/commands/install';
+import { PipListCommand } from '../../../managers/builtin/commands/list';
 import * as helpers from '../../../managers/builtin/helpers';
 import { PipPackageManager } from '../../../managers/builtin/pipPackageManager';
 import * as pipUtils from '../../../managers/builtin/pipUtils';