remove logging

microsoft · Sep 12, 2023 · 7bfbf45 · 7bfbf45
1 parent 1ddf542
commit 7bfbf45
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 13 deletions.
diff --git a/src/client/testing/testController/common/resultResolver.ts b/src/client/testing/testController/common/resultResolver.ts
@@ -157,9 +157,6 @@ export class PythonResultResolver implements ITestResultResolver {
                 ) {
                     const grabTestItem = this.runIdToTestItem.get(keyTemp);
                     const grabVSid = this.runIdToVSid.get(keyTemp);
-                    console.log('grabTestItem', grabTestItem);
-                    console.log('grabVSid', grabVSid);
-                    console.log('key', keyTemp);
                     if (grabTestItem !== undefined) {
                         testCases.forEach((indiItem) => {
                             if (indiItem.id === grabVSid) {

diff --git a/src/client/testing/testController/pytest/pytestExecutionAdapter.ts b/src/client/testing/testController/pytest/pytestExecutionAdapter.ts
@@ -130,7 +130,6 @@ export class PytestTestExecutionAdapter implements ITestExecutionAdapter {
             if (debugBool && !testArgs.some((a) => a.startsWith('--capture') || a === '-s')) {
                 testArgs.push('--capture', 'no');
             }
-            console.log(`Running PYTEST execution for the following test ids: ${testIds}`);
 
             const pytestRunTestIdsPort = await utils.startTestIdServer(testIds);
             if (spawnOptions.extraVariables)
@@ -170,11 +169,9 @@ export class PytestTestExecutionAdapter implements ITestExecutionAdapter {
                 // Displays output to user and ensure the subprocess doesn't run into buffer overflow.
                 result?.proc?.stdout?.on('data', (data) => {
                     this.outputChannel?.append(data.toString());
-                    console.log(data.toString());
                 });
                 result?.proc?.stderr?.on('data', (data) => {
                     this.outputChannel?.append(data.toString());
-                    console.log(data.toString());
                 });
 
                 result?.proc?.on('exit', (code, signal) => {

diff --git a/src/test/testing/common/testingAdapter.test.ts b/src/test/testing/common/testingAdapter.test.ts
@@ -113,7 +113,6 @@ suite('End to End Tests: test adapters', () => {
             // 2. Confirm no errors
             assert.strictEqual(actualData.error, undefined, "Expected no errors in 'error' field");
             // 3. Confirm tests are found
-            console.log('actual data', actualData.tests);
             assert.ok(actualData.tests, 'Expected tests to be present');
         });
     });
@@ -249,7 +248,6 @@ suite('End to End Tests: test adapters', () => {
         resultResolver
             .setup((x) => x.resolveExecution(typeMoq.It.isAny(), typeMoq.It.isAny()))
             .returns((data) => {
-                console.log(`resolveExecution ${JSON.stringify(data)}`);
                 actualData = data;
                 return Promise.resolve();
             });
@@ -293,7 +291,6 @@ suite('End to End Tests: test adapters', () => {
         resultResolver
             .setup((x) => x.resolveExecution(typeMoq.It.isAny(), typeMoq.It.isAny()))
             .returns((data) => {
-                console.log(`resolveExecution ${JSON.stringify(data)}`);
                 traceLog(`resolveExecution ${data}`);
                 // do the following asserts for each time resolveExecution is called, should be called once per test.
                 // 1. Check the status, can be subtest success or failure
@@ -346,7 +343,6 @@ suite('End to End Tests: test adapters', () => {
         resultResolver
             .setup((x) => x.resolveExecution(typeMoq.It.isAny(), typeMoq.It.isAny()))
             .returns((data) => {
-                console.log(`resolveExecution ${JSON.stringify(data)}`);
                 actualData = data;
                 return Promise.resolve();
             });
@@ -397,7 +393,6 @@ suite('End to End Tests: test adapters', () => {
         resultResolver
             .setup((x) => x.resolveExecution(typeMoq.It.isAny(), typeMoq.It.isAny()))
             .returns((data) => {
-                console.log(`resolveExecution ${JSON.stringify(data)}`);
                 // do the following asserts for each time resolveExecution is called, should be called once per test.
                 // 1. Check the status is "success"
                 assert.strictEqual(data.status, 'success', "Expected status to be 'success'");
@@ -448,7 +443,6 @@ suite('End to End Tests: test adapters', () => {
         resultResolver
             .setup((x) => x.resolveExecution(typeMoq.It.isAny(), typeMoq.It.isAny()))
             .returns((data) => {
-                console.log(`resolveExecution ${JSON.stringify(data)}`);
                 // do the following asserts for each time resolveExecution is called, should be called once per test.
                 // 1. Check the status is "success"
                 assert.strictEqual(data.status, 'error', "Expected status to be 'error'");
@@ -497,7 +491,6 @@ suite('End to End Tests: test adapters', () => {
         resultResolver
             .setup((x) => x.resolveExecution(typeMoq.It.isAny(), typeMoq.It.isAny()))
             .returns((data) => {
-                console.log(`resolveExecution ${JSON.stringify(data)}`);
                 // do the following asserts for each time resolveExecution is called, should be called once per test.
                 // 1. Check the status is "success"
                 assert.strictEqual(data.status, 'error', "Expected status to be 'error'");