Ensure file names are not captured when sending telemetry for unit tests (microsoft#3768)

For microsoft#3767

Author: DonJayamanne

news/2 Fixes/3767.md

@@ -0,0 +1 @@
+Ensure file names are not captured when sending telemetry for unit tests.

src/client/telemetry/types.ts

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

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

@@ -2,6 +2,7 @@ import { CancellationToken, CancellationTokenSource, Disposable, OutputChannel,
 import { IWorkspaceService } from '../../../common/application/types';
 import { isNotInstalledError } from '../../../common/helpers';
 import { IConfigurationService, IDisposableRegistry, IInstaller, IOutputChannel, IPythonSettings, Product } from '../../../common/types';
+import { getNamesAndValues } from '../../../common/utils/enum';
 import { IServiceContainer } from '../../../ioc/types';
 import { UNITTEST_DISCOVER, UNITTEST_RUN } from '../../../telemetry/constants';
 import { sendTelemetryEvent } from '../../../telemetry/index';
@@ -171,11 +172,13 @@ export abstract class BaseTestManager implements ITestManager {
  Run_Specific_Class: 'false',
  Run_Specific_Function: 'false'
  };
+ //Ensure valid values are sent.
+ const validCmdSourceValues = getNamesAndValues<CommandSource>(CommandSource).map(item => item.value);
  const telementryProperties: TestRunTelemetry = {
  tool: this.testProvider,
  scope: 'all',
  debugging: debug === true,
- triggeredBy: cmdSource,
+ triggerSource: validCmdSourceValues.indexOf(cmdSource) === -1 ? 'commandpalette' : cmdSource,
  failed: false
  };
  if (runFailedTests === true) {