Adopt new APIs from vscode-tas-client (#974)

* Adopt new APIs from vscode-tas-client

Signed-off-by: Sheng Chen <[email protected]>

Author: jdneo

package-lock.json

@@ -787,6 +787,6 @@
     "uuid": "^8.3.1",
     "vscode-extension-telemetry": "^0.1.6",
     "vscode-extension-telemetry-wrapper": "^0.9.0",
-    "vscode-tas-client": "^0.1.17"
+    "vscode-tas-client": "^0.1.22"
   }
 }

package.json

@@ -3,7 +3,7 @@
 
 import * as vscode from "vscode";
 import { addContextProperty, sendInfo } from "vscode-extension-telemetry-wrapper";
-import { getExperimentationService, IExperimentationService, IExperimentationTelemetry, TargetPopulation } from "vscode-tas-client";
+import { getExperimentationServiceAsync, IExperimentationService, IExperimentationTelemetry, TargetPopulation } from "vscode-tas-client";
 
 class ExperimentationTelemetry implements IExperimentationTelemetry {
 
@@ -27,12 +27,14 @@ export function getExpService() {
     return expService;
 }
 
-export function initExpService(context: vscode.ExtensionContext): void {
+export async function initExpService(context: vscode.ExtensionContext): Promise<void> {
     const packageJson: {[key: string]: any} = require("../package.json");
     // tslint:disable: no-string-literal
     const extensionName = `${packageJson["publisher"]}.${packageJson["name"]}`;
     const extensionVersion = packageJson["version"];
     // tslint:enable: no-string-literal
-    expService = getExperimentationService(extensionName, extensionVersion,
+
+    // The async version will await the initializePromise to make sure shared property is set
+    expService = await getExperimentationServiceAsync(extensionName, extensionVersion,
         TargetPopulation.Public, new ExperimentationTelemetry(), context.globalState);
 }

src/experimentationService.ts

@@ -30,14 +30,14 @@ export async function activate(context: vscode.ExtensionContext): Promise<any> {
     await initializeFromJsonFile(context.asAbsolutePath("./package.json"), {
         firstParty: true,
     });
+    await initExpService(context);
     return instrumentOperation("activation", initializeExtension)(context);
 }
 
 function initializeExtension(_operationId: string, context: vscode.ExtensionContext): any {
     // Deprecated
     logger.initialize(context, true);
 
-    initExpService(context);
     registerDebugEventListener(context);
     context.subscriptions.push(logger);
     context.subscriptions.push(vscode.window.registerTerminalLinkProvider(new JavaTerminalLinkProvder()));