test: remove package manager tests from refactor

Package manager coverage is being developed separately in PR #1704.

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

Copilot-Session: 6b2fe9b5-38ea-442f-b07a-b6c71134d480

Author: edvilme

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

@@ -606,4 +606,3 @@ envConfig.inspect
 - **Never skip tests to hide infrastructure problems**: If tests require native binaries (like `pet`), the CI workflow must build/download them. Skipping tests when infrastructure is missing gives false confidence. Build from source (like vscode-python does) rather than skipping. Tests should fail clearly when something is wrong (2)
 - **No retries for masking flakiness**: Mocha `retries` should not be used to mask test flakiness. If a test is flaky, fix the root cause. Retries hide real issues and slow down CI (1)
 - **pet binary is required for environment manager registration**: The smoke/E2E/integration tests require the `pet` binary from `microsoft/python-environment-tools` to be built and placed in `python-env-tools/bin/`. Without it, `waitForApiReady()` will timeout because managers never register. CI must build pet from source using `cargo build --release --package pet` (2)
-- **Package lifecycle integration tests require package-index network access**: Venv quick creation seeds Pip through `uv`, and package roundtrip tests download packages. TLS or connectivity failures to PyPI can make `createEnvironment()` return `undefined` before package assertions run (1)

src/managers/builtin/venvUtils.ts

@@ -19,7 +19,6 @@ import { getWorkspacePersistentState } from '../../common/persistentState';
 import { EventNames } from '../../common/telemetry/constants';
 import { sendTelemetryEvent } from '../../common/telemetry/sender';
 import { normalizePath } from '../../common/utils/pathUtils';
-import { isTestExecution } from '../../common/utils/testing';
 import { getVenvPythonPath } from '../../common/utils/virtualEnvironment';
 import {
     showErrorMessage,
@@ -569,16 +568,14 @@ export async function removeVenv(environment: PythonEnvironment, log: LogOutputC
     // Normalize path for UI display - ensure forward slashes on Windows
     const displayPath = normalizePath(envPath);
 
-    const confirm = isTestExecution()
-        ? { title: Common.yes }
-        : await showWarningMessage(
-              l10n.t('Are you sure you want to remove {0}?', displayPath),
-              {
-                  modal: true,
-              },
-              { title: Common.yes },
-              { title: Common.no, isCloseAffordance: true },
-          );
+    const confirm = await showWarningMessage(
+        l10n.t('Are you sure you want to remove {0}?', displayPath),
+        {
+            modal: true,
+        },
+        { title: Common.yes },
+        { title: Common.no, isCloseAffordance: true },
+    );
     if (confirm?.title === Common.yes) {
         const result = await withProgress(
             {