Updates

Author: DonJayamanne

src/client/chat/listPackagesTool.ts

@@ -96,26 +96,30 @@ export class ListPythonPackagesTool implements LanguageModelTool<IResourceRefere
         const resourcePath = resolveFilePath(options.input.resourcePath);
         const envName = await raceCancellationError(getEnvDisplayName(this.discovery, resourcePath, this.api), token);
         return {
-            invocationMessage: envName ?  l10n.t('Listing packages in {0}', envName) : l10n.t('Fetching Python environment information'),
+            invocationMessage: envName
+                ? l10n.t('Listing packages in {0}', envName)
+                : l10n.t('Fetching Python environment information'),
         };
     }
 }
 
-
-async function listPipPackages(execFactory: IPythonExecutionFactory, resource: Uri | undefined): Promise<[packageName:string, version:string][]> {
+async function listPipPackages(
+    execFactory: IPythonExecutionFactory,
+    resource: Uri | undefined,
+): Promise<[string, string][]> {
     // Add option --format to subcommand list of pip  cache, with abspath choice to output the full path of a wheel file. (#8355)
     // Added in 202. Thats almost 5 years ago. When Python 3.8 was released.
     const exec = await execFactory.createActivatedEnvironment({ allowEnvironmentFetchExceptions: true, resource });
     const output = await exec.execModule('pip', ['list'], { throwOnStdErr: false, encoding: 'utf8' });
-    return parsePipList(output.stdout).map((pkg) => [pkg.name, pkg.version] );
+    return parsePipList(output.stdout).map((pkg) => [pkg.name, pkg.version]);
 }
 
 async function listCondaPackages(
     execFactory: IPythonExecutionFactory,
     env: ResolvedEnvironment,
     resource: Uri | undefined,
     processService: IProcessService,
-): Promise<[packageName:string, version:string][]> {
+): Promise<[string, string][]> {
     const conda = await Conda.getConda();
     if (!conda) {
         traceError('Conda is not installed, falling back to pip packages');

src/test/common.ts

@@ -62,7 +62,7 @@ export async function updateSetting(
     configTarget: ConfigurationTarget,
 ) {
     const vscode = require('vscode') as typeof import('vscode');
-    const settings = vscode.workspace.getConfiguration('python', { uri: resource, languageId: 'python' } || null);
+    const settings = vscode.workspace.getConfiguration('python', { uri: resource, languageId: 'python' });
     const currentValue = settings.inspect(setting);
     if (
         currentValue !== undefined &&

src/test/interpreters/pythonPathUpdaterFactory.unit.test.ts

@@ -17,6 +17,7 @@ suite('Python Path Settings Updater', () => {
         serviceContainer = TypeMoq.Mock.ofType<IServiceContainer>();
         workspaceService = TypeMoq.Mock.ofType<IWorkspaceService>();
         interpreterPathService = TypeMoq.Mock.ofType<IInterpreterPathService>();
+        experimentsManager = TypeMoq.Mock.ofType<IExperimentService>();
         serviceContainer
             .setup((c) => c.get(TypeMoq.It.isValue(IWorkspaceService)))
             .returns(() => workspaceService.object);

src/test/testing/testController/testCancellationRunAdapters.unit.test.ts

@@ -36,6 +36,7 @@ suite('Execution Flow Run Adapters', () => {
     let useEnvExtensionStub: sinon.SinonStub;
 
     setup(() => {
+        mockProc = {} as MockChildProcess;
         useEnvExtensionStub = sinon.stub(extapi, 'useEnvExtension');
         useEnvExtensionStub.returns(false);
         // general vars