Enables successful running of python file even when parentheses are p…

…resent in the path (microsoft#20414)

src/client/common/extensions.ts

@@ -73,7 +73,9 @@ String.prototype.toCommandArgumentForPythonExt = function (this: string): string
     if (!this) {
         return this;
     }
-    return (this.indexOf(' ') >= 0 || this.indexOf('&') >= 0) && !this.startsWith('"') && !this.endsWith('"')
+    return (this.indexOf(' ') >= 0 || this.indexOf('&') >= 0 || this.indexOf('(') >= 0 || this.indexOf(')') >= 0) &&
+        !this.startsWith('"') &&
+        !this.endsWith('"')
         ? `"${this}"`
         : this.toString();
 };

src/test/common/extensions.unit.test.ts

@@ -20,6 +20,21 @@ suite('String Extensions', () => {
         const argTotest = 'one two three';
         expect(argTotest.toCommandArgumentForPythonExt()).to.be.equal(`"${argTotest}"`);
     });
+    test('Should quote file paths containing one of the parentheses: ( ', () => {
+        const fileToTest = 'user/code(1.py';
+        expect(fileToTest.fileToCommandArgumentForPythonExt()).to.be.equal(`"${fileToTest}"`);
+    });
+
+    test('Should quote file paths containing one of the parentheses: ) ', () => {
+        const fileToTest = 'user)/code1.py';
+        expect(fileToTest.fileToCommandArgumentForPythonExt()).to.be.equal(`"${fileToTest}"`);
+    });
+
+    test('Should quote file paths containing both of the parentheses: () ', () => {
+        const fileToTest = '(user)/code1.py';
+        expect(fileToTest.fileToCommandArgumentForPythonExt()).to.be.equal(`"${fileToTest}"`);
+    });
+
     test('Should quote command arguments containing ampersand', () => {
         const argTotest = 'one&twothree';
         expect(argTotest.toCommandArgumentForPythonExt()).to.be.equal(`"${argTotest}"`);