Allow Activated Virtual Envs to Be Detected w/ a Workspace File (#25141)

Addresses #25140 (which
I just filed).

1) Removed the check that a workspaceFile isn't used when looking for
Activate Virtual Envs
2) Adds a debug log to help make other forms of early exit.  

Of note, I'm not sure if the existing `traceVerbose('VS Code was not
launched from the command line');` is correct.

As far as I can tell, workspaceFile != multiroot workspaces. I'm not
sure why this check previously existed. I don't know if this might cause
others problems; but right now this check causes a lot of challenge for
our workflow.

---------

Co-authored-by: Anthony Kim <[email protected]>

Author: tgrue-openai

src/client/interpreter/virtualEnvs/activatedEnvLaunch.ts

@@ -85,12 +85,9 @@ export class ActivatedEnvironmentLaunch implements IActivatedEnvironmentLaunch {
 
     @cache(-1, true)
     private async _selectIfLaunchedViaActivatedEnv(doNotBlockOnSelection = false): Promise<string | undefined> {
-        if (this.workspaceService.workspaceFile) {
-            // Assuming multiroot workspaces cannot be directly launched via `code .` command.
-            return undefined;
-        }
         if (process.env.VSCODE_CLI !== '1') {
             // We only want to select the interpreter if VS Code was launched from the command line.
+            traceLog("Skipping ActivatedEnv Detection:  process.env.VSCODE_CLI !== '1'");
             return undefined;
         }
         traceVerbose('VS Code was not launched from the command line');

src/test/interpreters/virtualEnvs/activatedEnvLaunch.unit.test.ts

@@ -263,37 +263,6 @@ suite('Activated Env Launch', async () => {
             expect(_promptIfApplicable.notCalled).to.equal(true, 'Prompt should not be displayed');
         });
 
-        test('Does not update interpreter path if a multiroot workspace is opened', async () => {
-            process.env.VIRTUAL_ENV = virtualEnvPrefix;
-            interpreterService
-                .setup((i) => i.getInterpreterDetails(TypeMoq.It.isAny()))
-                .returns(() => Promise.resolve(({ envName: 'base' } as unknown) as PythonEnvironment));
-            workspaceService.setup((w) => w.workspaceFile).returns(() => uri);
-            const workspaceFolder: WorkspaceFolder = { name: 'one', uri, index: 0 };
-            workspaceService.setup((w) => w.workspaceFolders).returns(() => [workspaceFolder]);
-            pythonPathUpdaterService
-                .setup((p) =>
-                    p.updatePythonPath(
-                        TypeMoq.It.isValue(virtualEnvPrefix),
-                        TypeMoq.It.isValue(ConfigurationTarget.WorkspaceFolder),
-                        TypeMoq.It.isValue('load'),
-                        TypeMoq.It.isValue(uri),
-                    ),
-                )
-                .returns(() => Promise.resolve())
-                .verifiable(TypeMoq.Times.never());
-            activatedEnvLaunch = new ActivatedEnvironmentLaunch(
-                workspaceService.object,
-                appShell.object,
-                pythonPathUpdaterService.object,
-                interpreterService.object,
-                processServiceFactory.object,
-            );
-            const result = await activatedEnvLaunch.selectIfLaunchedViaActivatedEnv();
-            expect(result).to.be.equal(undefined, 'Incorrect value');
-            pythonPathUpdaterService.verifyAll();
-        });
-
         test('Returns `undefined` if env was already selected', async () => {
             activatedEnvLaunch = new ActivatedEnvironmentLaunch(
                 workspaceService.object,