Merge pull request #9 from microsoft/udpate-telemetry

Updated telemetry

Author: pkpio

.gitignore

@@ -6,4 +6,5 @@ dist
 .DS_Store
 **/.DS_Store
 test/assets/extracted/
+dist_gh_action/package.json
 

ado-extension.json

@@ -2,7 +2,7 @@
     "manifestVersion": 1,
     "id": "android-app-size-diff-utils",
     "name": "Android app size changes",
-    "version": "0.0.28",
+    "version": "0.0.29",
     "publisher": "PraveenPendyala",
     "targets": [
         {

dist_gh_action/index.js

@@ -1,6 +1,6 @@
 {
     "name": "android-app-size-diff",
-    "version": "0.0.28",
+    "version": "0.0.29",
     "description": "Azure DevOps task to measure the size in Android app size by looking at 2 given APKs and AABs",
     "main": "index.js",
     "scripts": {

package.json

@@ -63,34 +63,40 @@ export default class CiRunner {
             ciName: this.ciCore.getCiName()
         }
         this.telemetryClient.trackEvent({
-            name: 'AppStart',
+            name: 'RunStarted',
             properties: telemetryProperties
         });
 
         var result: any;
         try {
             result = await this.run();
+
+            // Send success telemetry
+            const endTime = new Date().getTime();
+            const elapsedTime = endTime - startTime;
+            this.telemetryClient.trackEvent({
+                name: 'RunSuccess',
+                measurements: {
+                    duration: elapsedTime
+                },
+                properties: telemetryProperties
+            });
         } catch (err) {
             // Send error telemetry
+            this.telemetryClient.trackEvent({
+                name: 'RunFailed',
+                properties: telemetryProperties
+            });
             this.telemetryClient.trackException({
                 exception: err,
                 properties: telemetryProperties
             });
-            this.telemetryClient.flush()
 
             throw err;
+        } finally {
+            this.telemetryClient.flush();
         }
 
-        // Send performance telemetry
-        const endTime = new Date().getTime();
-        const elapsedTime = endTime - startTime;
-        this.telemetryClient.trackMetric({
-            name: 'RunPerformance',
-            value: elapsedTime,
-            properties: telemetryProperties
-        });
-        this.telemetryClient.flush()
-
         return result;
     }
 

src/apkAnalyzer/CiRunner.ts

@@ -10,7 +10,7 @@
     "version": {
         "Major": 0,
         "Minor": 0,
-        "Patch": 28
+        "Patch": 29
     },
     "instanceNameFormat": "Android App size change - $(baseAppPath) vs $(targetAppPath)",
     "inputs": [