Changed property name used to capture the trigger source of Unit Tests

Fixes #1213

Author: DonJayamanne

news/3 Code Health/1213.md

@@ -0,0 +1 @@
+Changed property name used to capture the trigger source of Unit Tests.

src/client/telemetry/types.ts

@@ -44,7 +44,7 @@ export type TestRunTelemetry = {
     tool: 'nosetest' | 'pytest' | 'unittest';
     scope: 'currentFile' | 'all' | 'file' | 'class' | 'function' | 'failed';
     debugging: boolean;
-    trigger: 'ui' | 'codelens' | 'commandpalette' | 'auto';
+    triggeredBy: 'ui' | 'codelens' | 'commandpalette' | 'auto';
     failed: boolean;
 };
 export type TestDiscoverytTelemetry = {

src/client/unittests/common/managers/baseTestManager.ts

@@ -177,7 +177,7 @@ export abstract class BaseTestManager implements ITestManager {
             tool: this.testProvider,
             scope: 'all',
             debugging: debug === true,
-            trigger: cmdSource,
+            triggeredBy: cmdSource,
             failed: false
         };
         if (runFailedTests === true) {