help mock

Author: eleanorjboyd

src/managers/builtin/helpers.ts

@@ -16,7 +16,9 @@ export function resetUvInstallationCache(): void {
 }
 
 export async function isUvInstalled(log?: LogOutputChannel): Promise<boolean> {
+    console.log('into isUvInstalled function');
     if (available.completed) {
+        console.log('UV installation status already determined');
         return available.promise;
     }
     log?.info(`Running: uv --version`);
@@ -61,6 +63,7 @@ export async function shouldUseUv(log?: LogOutputChannel, envPath?: string): Pro
     console.log(config.inspect<boolean>('alwaysUseUv')?.globalValue);
 
     if (alwaysUseUv) {
+        console.log(`alwaysUseUv is true, checking if UV is installed`);
         return await isUvInstalled(log);
     }
     return false;

src/test/managers/builtin/helpers.shouldUseUv.unit.test.ts

@@ -4,7 +4,6 @@ import { LogOutputChannel } from 'vscode';
 import * as persistentState from '../../../common/persistentState';
 import * as workspaceApis from '../../../common/workspace.apis';
 import * as helpers from '../../../managers/builtin/helpers';
-import { shouldUseUv } from '../../../managers/builtin/helpers';
 import * as uvEnvironments from '../../../managers/builtin/uvEnvironments';
 
 interface MockWorkspaceConfig {
@@ -79,15 +78,15 @@ suite('Helpers - shouldUseUv', () => {
             globalLocalValue: true,
             globalValue: true,
         };
-        
+
         mockConfig.get.withArgs('alwaysUseUv', true).returns(true);
         mockConfig.get.withArgs('alwaysUseUv').returns(true);
         mockConfig.inspect.withArgs('alwaysUseUv').returns(mockInspectResult);
         isUvInstalledStub.resolves(true);
         getUvEnvironmentsStub.resolves([]);
 
         // Run
-        const result = await shouldUseUv(mockLog);
+        const result = await helpers.shouldUseUv(mockLog);
 
         // Assert - Should return true when setting is true and UV is installed
         assert.strictEqual(result, true);
@@ -99,7 +98,7 @@ suite('Helpers - shouldUseUv', () => {
         getUvEnvironmentsStub.resolves([]);
 
         // Run
-        const result = await shouldUseUv(mockLog);
+        const result = await helpers.shouldUseUv(mockLog);
 
         // Assert - Should not use UV when setting is false
         assert.strictEqual(result, false);
@@ -113,7 +112,7 @@ suite('Helpers - shouldUseUv', () => {
         mockConfig.get.withArgs('alwaysUseUv', true).returns(false);
 
         // Run
-        const result = await shouldUseUv(mockLog, uvEnvPath);
+        const result = await helpers.shouldUseUv(mockLog, uvEnvPath);
 
         // Assert - Should return true for UV environments when UV is installed
         assert.strictEqual(result, true);
@@ -126,7 +125,7 @@ suite('Helpers - shouldUseUv', () => {
         getUvEnvironmentsStub.resolves([]);
 
         // Run
-        const result = await shouldUseUv(mockLog, nonUvEnvPath);
+        const result = await helpers.shouldUseUv(mockLog, nonUvEnvPath);
 
         // Assert - Should not use UV for non-UV environments when setting is false
         assert.strictEqual(result, false);
@@ -140,7 +139,7 @@ suite('Helpers - shouldUseUv', () => {
         getUvEnvironmentsStub.resolves([]);
 
         // Run
-        const result = await shouldUseUv(mockLog, nonUvEnvPath);
+        const result = await helpers.shouldUseUv(mockLog, nonUvEnvPath);
 
         // Assert - Should return true when alwaysUseUv is true and UV is installed
         assert.strictEqual(result, true);