Include profiles with account_id in listProfiles results

Previously, `listProfiles` filtered out any profile that had `account_id` set to differentiate workspace-level from account-level profiles.   This logic is no longer valid, so all profiles are displayed now

Author: misha-db

packages/databricks-vscode/src/cli/CliWrapper.test.ts

@@ -169,6 +169,34 @@ nothing = true
         });
     });
 
+    it("should include profiles with account id", async () => {
+        const logFilePath = getTempLogFilePath();
+        const cli = createCliWrapper(logFilePath);
+
+        await withFile(async ({path}) => {
+            writeFileSync(
+                path,
+                `[regular-profile]
+host = https://cloud.databricks.com/
+token = dapitest1234
+
+[profile-with-accunt-id]
+host = https://accounts.cloud.databricks.com/
+account_id = 1234567890
+token = dapitest5678
+`,
+                "utf-8"
+            );
+
+            const profiles = await cli.listProfiles(path);
+
+            assert.equal(profiles.length, 2);
+            assert.equal(profiles[0].name, "regular-profile");
+            assert.equal(profiles[1].name, "profile-with-accunt-id");
+            assert.equal(profiles[1].accountId, "1234567890");
+        });
+    });
+
     it("should show error for corrupted config file and return empty profile list", async () => {
         const logFilePath = getTempLogFilePath();
         const cli = createCliWrapper(logFilePath);

packages/databricks-vscode/src/cli/CliWrapper.ts

@@ -404,10 +404,7 @@ export class CliWrapper {
             return [];
         }
 
-        let profiles = JSON.parse(res.stdout).profiles || [];
-
-        // filter out account profiles
-        profiles = profiles.filter((p: any) => !p.account_id);
+        const profiles = JSON.parse(res.stdout).profiles || [];
 
         const result = [];
         let hasError = false;