IPyKernel install issue with windows paths (microsoft#13667)

* Both changes * remove args change * Remove logging message * Add news * remove logMessage imports * Update python.ts * Fix unit tests * Actually fix the unit tests Co-authored-by: Rich Chiodo <rchiodo@users.noreply.github.com>

Author: IanMatthewHuff

news/2 Fixes/13493.md

@@ -0,0 +1 @@
+Fix path to isolated script on Windows shell_exec.

src/client/common/process/internal/python.ts

@@ -28,7 +28,7 @@ export function execCode(code: string, isolated = true): string[] {
 export function execModule(name: string, moduleArgs: string[], isolated = true): string[] {
  const args = ['-m', name, ...moduleArgs];
  if (isolated) {
- args[0] = ISOLATED; // replace
+ args[0] = ISOLATED.fileToCommandArgument();
  }
  // "code" isn't specific enough to know how to parse it,
  // so we only return the args.

src/client/common/process/internal/scripts/index.ts

@@ -306,7 +306,7 @@ export function shell_exec(command: string, lockfile: string, shellArgs: string[
  // We don't bother with a "parse" function since the output
  // could be anything.
  return [
- ISOLATED,
+ ISOLATED.fileToCommandArgument(),
  script,
  command.fileToCommandArgument(),
  // The shell args must come after the command

src/test/common/moduleInstaller.test.ts

@@ -140,7 +140,7 @@ import { closeActiveWindows, initializeTest } from './../initialize';
 
 chai_use(chaiAsPromised);
 
-const isolated = path.join(EXTENSION_ROOT_DIR_FOR_TESTS, 'pythonFiles', 'pyvsc-run-isolated.py');
+const isolated = path.join(EXTENSION_ROOT_DIR_FOR_TESTS, 'pythonFiles', 'pyvsc-run-isolated.py').replace(/\\/g, '/');
 
 const info: PythonEnvironment = {
  architecture: Architecture.Unknown,