fix unit tests with linters

Author: DonJayamanne

src/client/common/installer.ts

@@ -228,7 +228,7 @@ export function disableLinter(product: Product) {
 
 function isProductInstalled(product: Product): Promise<boolean> {
     const prodExec = ProductExecutableAndArgs.get(product);
-    return execPythonFile(prodExec.executable, prodExec.args.concat(['--version']), vscode.workspace.rootPath, false, undefined, undefined, true)
+    return execPythonFile(prodExec.executable, prodExec.args.concat(['--version']), vscode.workspace.rootPath, false)
         .then(() => {
             return true;
         }).catch(reason => {
@@ -238,5 +238,5 @@ function isProductInstalled(product: Product): Promise<boolean> {
 
 function uninstallproduct(product: Product): Promise<any> {
     const uninstallArgs = ProductUninstallScripts.get(product);
-    return execPythonFile('python', uninstallArgs, vscode.workspace.rootPath, false, undefined, undefined, true);
+    return execPythonFile('python', uninstallArgs, vscode.workspace.rootPath, false);
 }

src/client/common/utils.ts

@@ -96,7 +96,9 @@ export function getPathFromPythonCommand(args: string[]): Promise<string> {
     });
 }
 
-export function execPythonFile(file: string, args: string[], cwd: string, includeErrorAsResponse: boolean = false, stdOut: (line: string) => void = null, token?: CancellationToken, execAsModule: boolean = false): Promise<string> {
+export function execPythonFile(file: string, args: string[], cwd: string, includeErrorAsResponse: boolean = false, stdOut: (line: string) => void = null, token?: CancellationToken): Promise<string> {
+    const execAsModule = file.toUpperCase() === 'PYTHON' && args.length > 0 && args[0] === '-m';
+
     // If running the python file, then always revert to execFileInternal
     // Cuz python interpreter is always a file and we can and will always run it using child_process.execFile()
     if (file === settings.PythonSettings.getInstance().pythonPath) {