test: expand package manager integration coverage (#1717)

Follow-up to #1704. Adds shared conformance coverage ensuring Pip,
Conda, and Poetry do not prompt or show error UI during headless package
operations, while failures still reject.

---------

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3fd1a810-6840-4ac9-ac33-c8a9fda4bfc4
Copilot-Session: edcaaa35-9a42-4351-98fa-55bcf8f1968e

Author: 3 people

.github/instructions/testing-workflow.instructions.md

@@ -16,6 +16,10 @@ This guide covers the full testing lifecycle:
 4. **🛠️ Fixing Problems** - Resolve compilation and runtime issues
 5. **✅ Validation** - Ensure coverage and resilience
 
+## Learnings
+
+- Pip commands that return JSON must pass `--disable-pip-version-check`; the process helper combines stderr with stdout, so update notices can otherwise make valid JSON unparseable (1).
+
 ### When to Use This Guide
 
 **User Requests Testing:**

.github/workflows/pr-check.yml

@@ -252,6 +252,9 @@ jobs:
     name: Integration Tests
     runs-on: ${{ matrix.os }}
     needs: [smoke-tests]
+    env:
+      CONDA_PLUGINS_AUTO_ACCEPT_TOS: 'true'
+      VSC_PYTHON_PACKAGE_NETWORK_TEST: '1'
     strategy:
       fail-fast: false
       matrix:
@@ -325,6 +328,12 @@ jobs:
       - name: Compile Tests
         run: npm run compile-tests
 
+      - name: Set up Conda
+        uses: conda-incubator/setup-miniconda@fc2d68f6413eb2d87b895e92f8584b5b94a10167 # v3
+        with:
+          activate-environment: ''
+          auto-activate: false
+
       - name: Run Integration Tests (Linux)
         if: runner.os == 'Linux'
         uses: GabrielBB/xvfb-action@86d97bde4a65fe9b290c0b3fb92c2c4ed0e5302d # v1.6
@@ -335,14 +344,6 @@ jobs:
         if: runner.os != 'Linux'
         run: npm run integration-test
 
-      - name: Run Package Manager Network Integration Tests
-        if: runner.os == 'Linux' && matrix.python-version == '3.12'
-        uses: GabrielBB/xvfb-action@86d97bde4a65fe9b290c0b3fb92c2c4ed0e5302d # v1.6
-        env:
-          VSC_PYTHON_PACKAGE_NETWORK_TEST: '1'
-        with:
-          run: npm run integration-test -- --grep "Package Manager"
-
   integration-tests-multiroot:
     name: Integration Tests (Multi-Root)
     runs-on: ${{ matrix.os }}

.github/workflows/push-check.yml

@@ -253,6 +253,9 @@ jobs:
     name: Integration Tests
     runs-on: ${{ matrix.os }}
     needs: [smoke-tests]
+    env:
+      CONDA_PLUGINS_AUTO_ACCEPT_TOS: 'true'
+      VSC_PYTHON_PACKAGE_NETWORK_TEST: '1'
     strategy:
       fail-fast: false
       matrix:
@@ -326,6 +329,12 @@ jobs:
       - name: Compile Tests
         run: npm run compile-tests
 
+      - name: Set up Conda
+        uses: conda-incubator/setup-miniconda@fc2d68f6413eb2d87b895e92f8584b5b94a10167 # v3
+        with:
+          activate-environment: ''
+          auto-activate: false
+
       - name: Run Integration Tests (Linux)
         if: runner.os == 'Linux'
         uses: GabrielBB/xvfb-action@86d97bde4a65fe9b290c0b3fb92c2c4ed0e5302d # v1.6
@@ -335,11 +344,3 @@ jobs:
       - name: Run Integration Tests (non-Linux)
         if: runner.os != 'Linux'
         run: npm run integration-test
-
-      - name: Run Package Manager Network Integration Tests
-        if: runner.os == 'Linux' && matrix.python-version == '3.12'
-        uses: GabrielBB/xvfb-action@86d97bde4a65fe9b290c0b3fb92c2c4ed0e5302d # v1.6
-        env:
-          VSC_PYTHON_PACKAGE_NETWORK_TEST: '1'
-        with:
-          run: npm run integration-test -- --grep "Package Manager"

src/extension.ts

@@ -276,6 +276,16 @@ export async function activate(context: ExtensionContext): Promise<PythonEnviron
                   commands.registerCommand('python-envs.test.getPackageManagerIds', () =>
                       envManagers.packageManagers.map((manager) => manager.id),
                   ),
+                  commands.registerCommand(
+                      'python-envs.test.resolveEnvironmentWithManager',
+                      async (managerId: string, environmentUri: Uri) => {
+                          const manager = envManagers.getEnvironmentManager(managerId);
+                          if (!manager) {
+                              throw new Error(`Environment manager not found: ${managerId}`);
+                          }
+                          return manager.resolve(environmentUri);
+                      },
+                  ),
                   commands.registerCommand(
                       'python-envs.test.getDirectPackageNames',
                       async (environment: PythonEnvironment) => {

src/managers/builtin/pipPackageManager.ts

@@ -2,14 +2,14 @@ import type { Pep440Version } from '@renovatebot/pep440';
 import { compare, explain as parse, rcompare } from '@renovatebot/pep440';
 import {
     CancellationError,
+    CancellationToken,
     Disposable,
     Event,
     EventEmitter,
     LogOutputChannel,
     MarkdownString,
     ProgressLocation,
     ThemeIcon,
-    window,
 } from 'vscode';
 import {
     DidChangePackagesEventArgs,
@@ -22,6 +22,7 @@ import {
     PythonEnvironment,
     PythonEnvironmentApi,
 } from '../../api';
+import { showErrorMessage, withProgress } from '../../common/window.apis';
 import { updatePackagesAndNotify } from '../common/packageChanges';
 import { runPython, runUV, shouldUseUv } from './helpers';
 import { getWorkspacePackagesToInstall } from './pipUtils';
@@ -75,45 +76,52 @@ export class PipPackageManager implements PackageManager, Disposable {
             install: toInstall,
             uninstall: toUninstall,
         };
-        await window.withProgress(
+        const execute = async (token?: CancellationToken): Promise<void> => {
+            try {
+                await managePackages(environment, manageOptions, this, token);
+                await updatePackagesAndNotify(
+                    this,
+                    environment,
+                    this.packages.get(environment.envId.id),
+                    (changes) => {
+                        this._onDidChangePackages.fire({ environment, manager: this, changes });
+                    },
+                    () => this.fetchPackages(environment, !manageOptions.runHeadless),
+                );
+            } catch (e) {
+                if (e instanceof CancellationError) {
+                    throw e;
+                }
+                this.log.error('Error managing packages', e);
+                if (!manageOptions.runHeadless) {
+                    setImmediate(async () => {
+                        const result = await showErrorMessage('Error managing packages', 'View Output');
+                        if (result === 'View Output') {
+                            this.log.show();
+                        }
+                    });
+                }
+                throw e;
+            }
+        };
+
+        if (manageOptions.runHeadless) {
+            await execute();
+            return;
+        }
+
+        await withProgress(
             {
                 location: ProgressLocation.Notification,
                 title: 'Installing packages',
                 cancellable: true,
             },
-            async (_progress, token) => {
-                try {
-                    await managePackages(environment, manageOptions, this, token);
-                    await updatePackagesAndNotify(
-                        this,
-                        environment,
-                        this.packages.get(environment.envId.id),
-                        (changes) => {
-                            this._onDidChangePackages.fire({ environment, manager: this, changes });
-                        },
-                        () => this.fetchPackages(environment, !manageOptions.runHeadless),
-                    );
-                } catch (e) {
-                    if (e instanceof CancellationError) {
-                        throw e;
-                    }
-                    this.log.error('Error managing packages', e);
-                    if (!manageOptions.runHeadless) {
-                        setImmediate(async () => {
-                            const result = await window.showErrorMessage('Error managing packages', 'View Output');
-                            if (result === 'View Output') {
-                                this.log.show();
-                            }
-                        });
-                    }
-                    throw e;
-                }
-            },
+            async (_progress, token) => execute(token),
         );
     }
 
     async refresh(environment: PythonEnvironment): Promise<void> {
-        await window.withProgress(
+        await withProgress(
             {
                 location: ProgressLocation.Window,
                 title: 'Refreshing packages',

src/managers/builtin/utils.ts

@@ -200,7 +200,7 @@ async function execPipList(environment: PythonEnvironment, log?: LogOutputChanne
     try {
         return await runPython(
             environment.execInfo.run.executable,
-            ['-m', 'pip', 'list', '--format=json', ...(args ?? [])],
+            ['-m', 'pip', 'list', '--format=json', '--disable-pip-version-check', ...(args ?? [])],
             undefined,
             log,
             undefined,

src/managers/conda/condaPackageManager.ts

@@ -3,6 +3,7 @@ import { explain as parse, rcompare } from '@renovatebot/pep440';
 import * as path from 'path';
 import {
     CancellationError,
+    CancellationToken,
     Disposable,
     Event,
     EventEmitter,
@@ -72,37 +73,44 @@ export class CondaPackageManager implements PackageManager, Disposable {
             install: toInstall,
             uninstall: toUninstall,
         };
+        const execute = async (token?: CancellationToken): Promise<void> => {
+            try {
+                await managePackages(environment, manageOptions, token, this.log);
+                await updatePackagesAndNotify(
+                    this,
+                    environment,
+                    this.packages.get(environment.envId.id),
+                    (changes) => {
+                        this._onDidChangePackages.fire({ environment, manager: this, changes });
+                    },
+                );
+            } catch (e) {
+                if (e instanceof CancellationError) {
+                    throw e;
+                }
+
+                this.log.error('Error installing packages', e);
+                if (!manageOptions.runHeadless) {
+                    setImmediate(async () => {
+                        await showErrorMessageWithLogs(CondaStrings.condaInstallError, this.log);
+                    });
+                }
+                throw e;
+            }
+        };
+
+        if (manageOptions.runHeadless) {
+            await execute();
+            return;
+        }
+
         await withProgress(
             {
                 location: ProgressLocation.Notification,
                 title: CondaStrings.condaInstallingPackages,
                 cancellable: true,
             },
-            async (_progress, token) => {
-                try {
-                    await managePackages(environment, manageOptions, token, this.log);
-                    await updatePackagesAndNotify(
-                        this,
-                        environment,
-                        this.packages.get(environment.envId.id),
-                        (changes) => {
-                            this._onDidChangePackages.fire({ environment, manager: this, changes });
-                        },
-                    );
-                } catch (e) {
-                    if (e instanceof CancellationError) {
-                        throw e;
-                    }
-
-                    this.log.error('Error installing packages', e);
-                    if (!manageOptions.runHeadless) {
-                        setImmediate(async () => {
-                            await showErrorMessageWithLogs(CondaStrings.condaInstallError, this.log);
-                        });
-                    }
-                    throw e;
-                }
-            },
+            async (_progress, token) => execute(token),
         );
     }
 
@@ -178,16 +186,6 @@ export class CondaPackageManager implements PackageManager, Disposable {
         }
     }
 
-    /**
-     * Lists available versions for a package via `conda search --json`, newest first.
-     *
-     * Conda always supports version lookup, so operational failures (command, network, or
-     * malformed/unparseable output) propagate instead of being swallowed into `undefined`.
-     *
-     * @param _environment - Unused; conda resolves versions from its configured channels.
-     * @param packageName - The package whose versions should be listed.
-     * @returns A promise that resolves to an array of {@link Pep440Version} objects (newest first).
-     */
     async getPackageAvailableVersions(
         _environment: PythonEnvironment,
         packageName: string,
@@ -197,7 +195,6 @@ export class CondaPackageManager implements PackageManager, Disposable {
         if (!parsed || typeof parsed !== 'object' || !Array.isArray(parsed[packageName])) {
             throw new Error(`Conda returned unexpected package version data for: ${packageName}`);
         }
-
         const uniqueVersions = new Map<string, Pep440Version>();
         parsed[packageName]
             .filter((entry: { version?: string }) => !!entry.version?.trim())

src/managers/conda/condaUtils.ts

@@ -1253,7 +1253,7 @@ export async function deleteCondaEnvironment(environment: PythonEnvironment, log
 export async function managePackages(
     environment: PythonEnvironment,
     options: PackageManagementOptions,
-    token: CancellationToken,
+    token: CancellationToken | undefined,
     log: LogOutputChannel,
 ): Promise<void> {
     if (options.uninstall && options.uninstall.length > 0) {