Ability to prevent logging python execution info (microsoft#7464)

Author: DonJayamanne

src/client/common/constants.ts

@@ -80,6 +80,8 @@ export namespace Delays {
 
 export const STANDARD_OUTPUT_CHANNEL = 'STANDARD_OUTPUT_CHANNEL';
 
+export const isCI = process.env.TRAVIS === 'true' || process.env.TF_BUILD !== undefined;
+
 export function isTestExecution(): boolean {
  return process.env.VSC_PYTHON_CI_TEST === '1' || isUnitTestExecution();
 }

src/client/common/process/logger.ts

@@ -1,8 +1,11 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT License.
-import { inject, injectable, named } from 'inversify';
-import { STANDARD_OUTPUT_CHANNEL } from '../constants';
 
+'use strict';
+
+import { inject, injectable, named } from 'inversify';
+import { workspace } from 'vscode';
+import { isCI, isTestExecution, STANDARD_OUTPUT_CHANNEL } from '../constants';
 import { traceInfo } from '../logger';
 import { IOutputChannel, IPathUtils } from '../types';
 import { Logging } from '../utils/localize';
@@ -13,9 +16,18 @@ export class ProcessLogger implements IProcessLogger {
  constructor(
  @inject(IOutputChannel) @named(STANDARD_OUTPUT_CHANNEL) private readonly outputChannel: IOutputChannel,
  @inject(IPathUtils) private readonly pathUtils: IPathUtils
- ) {}
+ ) { }
 
  public logProcess(file: string, args: string[], options?: SpawnOptions) {
+ if (
+ !isTestExecution() &&
+ isCI &&
+ !workspace.getConfiguration('python', null).get<boolean>('enableProcessLogging', true)
+ ) {
+ // Added to disable logging of process execution commands during UI Tests.
+ // Used only during UI Tests (hence this setting need not be exposed as a valid setting).
+ return;
+ }
  const argsList = args.reduce((accumulator, current, index) => {
  let formattedArg = this.pathUtils.getDisplayName(current).toCommandArgument();
  if (current[0] === '\'' || current[0] === '"') {