Fix location for native environments (microsoft#23851)

Author: karthiknadig

src/client/pythonEnvironments/nativeAPI.ts

@@ -48,15 +48,17 @@ function toArch(a: string | undefined): Architecture {
     }
 }
 
-function getLocation(nativeEnv: NativeEnvInfo): string {
-    if (nativeEnv.prefix) {
-        return nativeEnv.prefix;
-    }
+function getLocation(nativeEnv: NativeEnvInfo, executable: string): string {
     if (nativeEnv.executable) {
         return nativeEnv.executable;
     }
-    // We should not get here: either prefix or executable should always be available
-    return '';
+
+    if (nativeEnv.prefix) {
+        return nativeEnv.prefix;
+    }
+
+    // This is a path to a generated executable. Needed for backwards compatibility.
+    return executable;
 }
 
 function kindToShortString(kind: PythonEnvKind): string | undefined {
@@ -172,7 +174,7 @@ function toPythonEnvInfo(nativeEnv: NativeEnvInfo): PythonEnvInfo | undefined {
     const executable = nativeEnv.executable ?? makeExecutablePath(nativeEnv.prefix);
     return {
         name,
-        location: getLocation(nativeEnv),
+        location: getLocation(nativeEnv, executable),
         kind,
         id: executable,
         executable: {

src/test/pythonEnvironments/nativeAPI.unit.test.ts

@@ -52,7 +52,7 @@ suite('Native Python API', () => {
         distro: { org: '' },
         executable: { filename: '/usr/bin/python', sysPrefix: '/usr/bin', ctime: -1, mtime: -1 },
         kind: PythonEnvKind.System,
-        location: '/usr/bin',
+        location: '/usr/bin/python',
         source: [],
         name: 'basic_python',
         type: undefined,
@@ -103,7 +103,7 @@ suite('Native Python API', () => {
             mtime: -1,
         },
         kind: PythonEnvKind.Conda,
-        location: '/home/user/.conda/envs/conda_python',
+        location: '/home/user/.conda/envs/conda_python/python',
         source: [],
         name: 'conda_python',
         type: PythonEnvType.Conda,