Change worker to return interpreter information instead

Author: Kartik Raj

src/client/pythonEnvironments/info/environmentInfoService.ts

@@ -16,7 +16,7 @@ export enum EnvironmentInfoServiceQueuePriority {
 export const IEnvironmentInfoService = Symbol('IEnvironmentInfoService');
 export interface IEnvironmentInfoService {
     getEnvironmentInfo(
-        environment: PythonEnvInfo,
+        interpreterPath: string,
         priority?: EnvironmentInfoServiceQueuePriority
     ): Promise<InterpreterInformation | undefined>;
 }
@@ -49,7 +49,7 @@ export class EnvironmentInfoService implements IEnvironmentInfoService {
     }
 
     public async getEnvironmentInfo(
-        environment: PythonEnvInfo,
+        interpreterPath: string,
         priority?: EnvironmentInfoServiceQueuePriority,
     ): Promise<InterpreterInformation | undefined> {
         const result = this.cache.get(interpreterPath);
@@ -60,8 +60,8 @@ export class EnvironmentInfoService implements IEnvironmentInfoService {
         const deferred = createDeferred<InterpreterInformation>();
         this.cache.set(interpreterPath, deferred);
         return (priority === EnvironmentInfoServiceQueuePriority.High
-            ? this.workerPool.addToQueue(environment, QueuePosition.Front)
-            : this.workerPool.addToQueue(environment, QueuePosition.Back)
+            ? this.workerPool.addToQueue(interpreterPath, QueuePosition.Front)
+            : this.workerPool.addToQueue(interpreterPath, QueuePosition.Back)
         ).then((r) => {
             deferred.resolve(r);
             if (r === undefined) {

src/test/pythonEnvironments/info/environmentInfoService.functional.test.ts

@@ -54,11 +54,9 @@ suite('Environment Info Service', () => {
         for (let i = 0; i < 10; i = i + 1) {
             const path = `any-path${i}`;
             if (i < 5) {
-                promises.push(envService.getEnvironmentInfo(createEnvInfo(path)));
+                promises.push(envService.getEnvironmentInfo(path));
             } else {
-                promises.push(
-                    envService.getEnvironmentInfo(createEnvInfo(path), EnvironmentInfoServiceQueuePriority.High),
-                );
+                promises.push(envService.getEnvironmentInfo(path, EnvironmentInfoServiceQueuePriority.High));
             }
             expected.push(createExpectedEnvInfo(path));
         }
@@ -81,10 +79,10 @@ suite('Environment Info Service', () => {
         // Clear call counts
         stubShellExec.resetHistory();
         // Evaluate once so the result is cached.
-        await envService.getEnvironmentInfo(createEnvInfo(path));
+        await envService.getEnvironmentInfo(path);
 
         for (let i = 0; i < 10; i = i + 1) {
-            promises.push(envService.getEnvironmentInfo(createEnvInfo(path)));
+            promises.push(envService.getEnvironmentInfo(path));
             expected.push(createExpectedEnvInfo(path));
         }