Always change drive letter if a file outside the workspace has been run on Windows (#18146)

Author: Kartik Raj

src/client/terminals/codeExecution/terminalCodeExecution.ts

@@ -16,6 +16,7 @@ import { ICodeExecutionService } from '../../terminals/types';
 
 @injectable()
 export class TerminalCodeExecutionProvider implements ICodeExecutionService {
+    private hasRanOutsideCurrentDrive = false;
     protected terminalTitle!: string;
     private _terminalService!: ITerminalService;
     private replActive?: Promise<boolean>;
@@ -98,7 +99,8 @@ export class TerminalCodeExecutionProvider implements ICodeExecutionService {
                         ? this.workspace.rootPath.replace(/\:.*/g, '')
                         : undefined;
                 const fileDrive = fileDirPath.replace(/\:.*/g, '');
-                if (fileDrive !== currentDrive) {
+                if (fileDrive !== currentDrive || this.hasRanOutsideCurrentDrive) {
+                    this.hasRanOutsideCurrentDrive = true;
                     await this.getTerminalService(file).sendText(`${fileDrive}:`);
                 }
             }

src/test/terminals/codeExecution/terminalCodeExec.unit.test.ts

@@ -179,6 +179,13 @@ suite('Terminal - Code Execution', () => {
                 await ensureWeSetCurrentDriveBeforeChangingDirectory(true);
             });
 
+            test('Ensure once set current drive before, we always send command to change the drive letter for subsequent executions', async () => {
+                await ensureWeSetCurrentDriveBeforeChangingDirectory(true);
+                const file = Uri.file(path.join('c:', 'path', 'to', 'file', 'one.py'));
+                await executor.executeFile(file);
+                terminalService.verify(async (t) => t.sendText(TypeMoq.It.isValue('c:')), TypeMoq.Times.once());
+            });
+
             async function ensureWeDoNotChangeDriveIfDriveLetterSameAsFileDriveLetter(
                 _isWindows: boolean,
             ): Promise<void> {