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

Fixes #1213
microsoft · Mar 27, 2018 · d1eb8e9 · d1eb8e9
1 parent 346a166
commit d1eb8e9
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/news/3 Code Health/1213.md b/news/3 Code Health/1213.md
@@ -0,0 +1 @@
+Changed property name used to capture the trigger source of Unit Tests.
diff --git a/src/client/telemetry/types.ts b/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 = {

diff --git a/src/client/unittests/common/managers/baseTestManager.ts b/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) {
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Changed property name used to capture the trigger source of Unit Tests.