Separated telemetry metrics for debugger (#214)

microsoft · Feb 21, 2020 · 97cb358 · 97cb358
1 parent 532d196
commit 97cb358
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 8 deletions.
diff --git a/src/constants.ts b/src/constants.ts
@@ -285,8 +285,12 @@ export enum CONFIG_KEYS {
 
 export enum TelemetryEventName {
     FAILED_TO_OPEN_SIMULATOR = "SIMULATOR.FAILED_TO_OPEN",
-    DEBUGGER_INIT_SUCCESS = "DEBUGGER.INIT.SUCCESS",
-    DEBUGGER_INIT_FAIL = "DEBUGGER.INIT.FAIL",
+
+    // Debugger
+    CPX_DEBUGGER_INIT_SUCCESS = "CPX.DEBUGGER.INIT.SUCCESS",
+    CPX_DEBUGGER_INIT_FAIL = "CPX.DEBUGGER.INIT.FAIL",
+    MICROBIT_DEBUGGER_INIT_SUCCESS = "MICROBIT.DEBUGGER.INIT.SUCCESS",
+    MICROBIT_DEBUGGER_INIT_FAIL = "MICROBIT.DEBUGGER.INIT.FAIL",
 
     // Extension commands
     COMMAND_RUN_SIMULATOR_BUTTON = "COMMAND.RUN.SIMULATOR_BUTTON",

diff --git a/src/extension.ts b/src/extension.ts
@@ -934,9 +934,7 @@ export async function activate(context: vscode.ExtensionContext) {
                     utils.getServerPortConfig()
                 );
 
-                telemetryAI.trackFeatureUsage(
-                    TelemetryEventName.DEBUGGER_INIT_SUCCESS
-                );
+                handleDebuggerTelemetry();
 
                 openWebview();
                 if (currentPanel) {
@@ -952,9 +950,7 @@ export async function activate(context: vscode.ExtensionContext) {
                         `Error trying to init the server on port ${utils.getServerPortConfig()}`
                     );
 
-                    telemetryAI.trackFeatureUsage(
-                        TelemetryEventName.DEBUGGER_INIT_FAIL
-                    );
+                    handleDebuggerFailTelemetry();
 
                     vscode.window.showErrorMessage(
                         CONSTANTS.ERROR.DEBUGGER_SERVER_INIT_FAILED(
@@ -1039,6 +1035,40 @@ const updateCurrentFileIfPython = async (
     }
 };
 
+const handleDebuggerTelemetry = () => {
+    switch (currentActiveDevice) {
+        case CONSTANTS.DEVICE_NAME.CPX:
+            telemetryAI.trackFeatureUsage(
+                TelemetryEventName.CPX_DEBUGGER_INIT_SUCCESS
+            );
+            break;
+        case CONSTANTS.DEVICE_NAME.MICROBIT:
+            telemetryAI.trackFeatureUsage(
+                TelemetryEventName.MICROBIT_DEBUGGER_INIT_SUCCESS
+            );
+            break;
+        default:
+            break;
+    }
+};
+
+const handleDebuggerFailTelemetry = () => {
+    switch (currentActiveDevice) {
+        case CONSTANTS.DEVICE_NAME.CPX:
+            telemetryAI.trackFeatureUsage(
+                TelemetryEventName.CPX_DEBUGGER_INIT_FAIL
+            );
+            break;
+        case CONSTANTS.DEVICE_NAME.MICROBIT:
+            telemetryAI.trackFeatureUsage(
+                TelemetryEventName.MICROBIT_DEBUGGER_INIT_FAIL
+            );
+            break;
+        default:
+            break;
+    }
+};
+
 const handleButtonPressTelemetry = (buttonState: any) => {
     switch (currentActiveDevice) {
         case CONSTANTS.DEVICE_NAME.CPX: