v5

Author: eleanorjboyd

.vscode-test.mjs

@@ -48,6 +48,7 @@ export default defineConfig([
     {
         label: 'integrationTests',
         files: 'out/test/integration/**/*.integration.test.js',
+        workspaceFolder: 'src/test/integration/test-workspace/integration-tests.code-workspace',
         mocha: {
             ui: 'tdd',
             timeout: 60000,

.vscode/settings.json

@@ -29,5 +29,8 @@
     "python-envs.pythonProjects": [],
     "git.branchRandomName.enable": true,
     "git.branchProtection": ["main"],
-    "git.branchProtectionPrompt": "alwaysCommitToNewBranch"
+    "git.branchProtectionPrompt": "alwaysCommitToNewBranch",
+    "chat.tools.terminal.autoApprove": {
+        "npx tsc": true
+    }
 }

src/test/integration/interpreterSelection.integration.test.ts

@@ -255,6 +255,14 @@ suite('Integration: Interpreter Selection Priority', function () {
         // Set environment first time
         await api.setEnvironment(undefined, env);
 
+        // Wait for any async config changes to settle before testing idempotency
+        await new Promise((resolve) => setTimeout(resolve, 500));
+
+        // Verify the environment was actually set
+        const currentEnv = await api.getEnvironment(undefined);
+        assert.ok(currentEnv, 'Environment should be set before idempotency test');
+        assert.strictEqual(currentEnv.envId.id, env.envId.id, 'Environment should match what we just set');
+
         const handler = new TestEventHandler<DidChangeEnvironmentEventArgs>(
             api.onDidChangeEnvironment,
             'onDidChangeEnvironment',
@@ -284,7 +292,7 @@ suite('Integration: Interpreter Selection Priority', function () {
                 }
             } else {
                 // No event fired - this is the ideal idempotent behavior
-                console.log('No event fired for same env selection (idempotent behavior)');
+                assert.ok(!handler.fired, 'No event should fire when setting same environment');
             }
         } finally {
             handler.dispose();
@@ -353,7 +361,7 @@ suite('Integration: Interpreter Selection Priority', function () {
             // but the explicit selection should be cleared
         } else {
             // Undefined is a valid result - no auto-discovery available
-            console.log('Selection cleared, no auto-discovery fallback');
+            assert.strictEqual(autoEnv, undefined, 'Cleared selection with no auto-discovery should return undefined');
         }
     });
 });

src/test/integration/packageManagement.integration.test.ts

@@ -84,7 +84,7 @@ suite('Integration: Package Management', function () {
 
         // May be undefined if package manager not available
         if (packages === undefined) {
-            console.log('Package manager not available for:', targetEnv.displayName);
+            this.skip();
             return;
         }
 
@@ -224,7 +224,7 @@ suite('Integration: Package Management', function () {
 
         // Both should return valid results (or undefined for same reason)
         if (packages1 === undefined || packages2 === undefined) {
-            console.log('Package manager not available for one or both environments');
+            this.skip();
             return;
         }
 

src/test/integration/pythonProjects.integration.test.ts

@@ -254,10 +254,13 @@ suite('Integration: Python Projects', function () {
         if (afterClear) {
             assert.ok(afterClear.envId, 'If environment returned, it must have valid envId');
             assert.ok(afterClear.envId.id, 'If environment returned, envId must have id');
+        } else {
+            assert.strictEqual(
+                afterClear,
+                undefined,
+                'Cleared association should return undefined when no auto-discovery',
+            );
         }
-        // If undefined, that's also valid - explicit selection was cleared
-
-        console.log('After clearing:', afterClear?.displayName ?? 'undefined (no auto-discovery)');
     });
 
     /**

src/test/integration/test-workspace/.gitkeep

@@ -0,0 +1,3 @@
+{
+    "python-envs.defaultEnvManager": "ms-python.python:system"
+}

src/test/integration/test-workspace/.vscode/settings.json

@@ -0,0 +1,11 @@
+{
+    "folders": [
+        {
+            "path": "project-a",
+        },
+        {
+            "path": "project-b",
+        },
+    ],
+    "settings": {},
+}

src/test/integration/test-workspace/integration-tests.code-workspace

@@ -0,0 +1,9 @@
+{
+    "python-envs.pythonProjects": [
+        {
+            "path": ".",
+            "envManager": "ms-python.python:system",
+            "packageManager": "ms-python.python:pip"
+        }
+    ]
+}