pass in correct interpreter for pytest

Author: anthonykim1

src/client/testing/testController/pytest/pytestDiscoveryAdapter.ts

@@ -24,6 +24,7 @@ import {
     hasSymlinkParent,
 } from '../common/utils';
 import { IEnvironmentVariablesProvider } from '../../../common/variables/types';
+import { IInterpreterService } from '../../../interpreter/contracts';
 
 /**
  * Wrapper class for unittest test discovery. This is where we call `runTestCommand`. #this seems incorrectly copied
@@ -34,6 +35,7 @@ export class PytestTestDiscoveryAdapter implements ITestDiscoveryAdapter {
         private readonly outputChannel: ITestOutputChannel,
         private readonly resultResolver?: ITestResultResolver,
         private readonly envVarsService?: IEnvironmentVariablesProvider,
+        private readonly interpreterService?: IInterpreterService,
     ) {}
 
     async discoverTests(uri: Uri, executionFactory?: IPythonExecutionFactory): Promise<DiscoveredTestPayload> {
@@ -102,11 +104,15 @@ export class PytestTestDiscoveryAdapter implements ITestDiscoveryAdapter {
             env: mutableEnv,
         };
 
+        const interpreter = await this.interpreterService?.getActiveInterpreter();
+
         // Create the Python environment in which to execute the command.
         const creationOptions: ExecutionFactoryCreateWithEnvironmentOptions = {
             allowEnvironmentFetchExceptions: false,
             resource: uri,
+            interpreter,
         };
+
         const execService = await executionFactory?.createActivatedEnvironment(creationOptions);
         // delete UUID following entire discovery finishing.
         const execArgs = ['-m', 'pytest', '-p', 'vscode_pytest', '--collect-only'].concat(pytestArgs);

src/client/testing/testController/pytest/pytestExecutionAdapter.ts

@@ -25,13 +25,15 @@ import { PYTEST_PROVIDER } from '../../common/constants';
 import { EXTENSION_ROOT_DIR } from '../../../common/constants';
 import * as utils from '../common/utils';
 import { IEnvironmentVariablesProvider } from '../../../common/variables/types';
+import { IInterpreterService } from '../../../interpreter/contracts';
 
 export class PytestTestExecutionAdapter implements ITestExecutionAdapter {
     constructor(
         public configSettings: IConfigurationService,
         private readonly outputChannel: ITestOutputChannel,
         private readonly resultResolver?: ITestResultResolver,
         private readonly envVarsService?: IEnvironmentVariablesProvider,
+        private readonly interpreterService?: IInterpreterService,
     ) {}
 
     async runTests(
@@ -131,10 +133,13 @@ export class PytestTestExecutionAdapter implements ITestExecutionAdapter {
         }
         const debugBool = profileKind && profileKind === TestRunProfileKind.Debug;
 
+        const interpreter = await this.interpreterService?.getActiveInterpreter();
+
         // Create the Python environment in which to execute the command.
         const creationOptions: ExecutionFactoryCreateWithEnvironmentOptions = {
             allowEnvironmentFetchExceptions: false,
             resource: uri,
+            interpreter,
         };
         // need to check what will happen in the exec service is NOT defined and is null
         const execService = await executionFactory?.createActivatedEnvironment(creationOptions);

src/test/testing/common/testingAdapter.test.ts

@@ -22,8 +22,10 @@ import { TestProvider } from '../../../client/testing/types';
 import { PYTEST_PROVIDER, UNITTEST_PROVIDER } from '../../../client/testing/common/constants';
 import { IEnvironmentVariablesProvider } from '../../../client/common/variables/types';
 import { createTypeMoq } from '../../mocks/helper';
+import { IInterpreterService } from '../../../client/interpreter/contracts';
 
 suite('End to End Tests: test adapters', () => {
+    let interpreterService: IInterpreterService;
     let resultResolver: ITestResultResolver;
     let pythonExecFactory: IPythonExecutionFactory;
     let configService: IConfigurationService;
@@ -109,7 +111,7 @@ suite('End to End Tests: test adapters', () => {
         pythonExecFactory = serviceContainer.get<IPythonExecutionFactory>(IPythonExecutionFactory);
         testController = serviceContainer.get<TestController>(ITestController);
         envVarsService = serviceContainer.get<IEnvironmentVariablesProvider>(IEnvironmentVariablesProvider);
-
+        interpreterService = serviceContainer.get<IInterpreterService>(IInterpreterService);
         // create objects that were not injected
 
         testOutputChannel = createTypeMoq<ITestOutputChannel>();
@@ -271,6 +273,7 @@ suite('End to End Tests: test adapters', () => {
             testOutputChannel.object,
             resultResolver,
             envVarsService,
+            interpreterService,
         );
 
         await discoveryAdapter.discoverTests(workspaceUri, pythonExecFactory).finally(() => {
@@ -326,6 +329,7 @@ suite('End to End Tests: test adapters', () => {
             testOutputChannel.object,
             resultResolver,
             envVarsService,
+            interpreterService,
         );
         configService.getSettings(workspaceUri).testing.pytestArgs = [];
 
@@ -415,6 +419,7 @@ suite('End to End Tests: test adapters', () => {
             testOutputChannel.object,
             resultResolver,
             envVarsService,
+            interpreterService,
         );
         configService.getSettings(workspaceUri).testing.pytestArgs = [];
 
@@ -491,6 +496,7 @@ suite('End to End Tests: test adapters', () => {
             testOutputChannel.object,
             resultResolver,
             envVarsService,
+            interpreterService,
         );
 
         // set workspace to test workspace folder
@@ -1038,6 +1044,7 @@ suite('End to End Tests: test adapters', () => {
             testOutputChannel.object,
             resultResolver,
             envVarsService,
+            interpreterService,
         );
 
         // set workspace to test workspace folder

src/test/testing/testController/pytest/pytestDiscoveryAdapter.unit.test.ts

@@ -20,6 +20,7 @@ import { EXTENSION_ROOT_DIR } from '../../../../client/constants';
 import { MockChildProcess } from '../../../mocks/mockChildProcess';
 import { Deferred, createDeferred } from '../../../../client/common/utils/async';
 import * as util from '../../../../client/testing/testController/common/utils';
+import { IInterpreterService } from '../../../../client/interpreter/contracts';
 
 suite('pytest test discovery adapter', () => {
     let configService: IConfigurationService;
@@ -71,7 +72,7 @@ suite('pytest test discovery adapter', () => {
         execService = typeMoq.Mock.ofType<IPythonExecutionService>();
         execService.setup((p) => ((p as unknown) as any).then).returns(() => undefined);
         outputChannel = typeMoq.Mock.ofType<ITestOutputChannel>();
-
+        interpreterService = typeMoq.Mock.ofType<IInterpreterService>();
         const output = new Observable<Output<string>>(() => {
             /* no op */
         });