corrected tooltip

Author: Kartik Raj

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

@@ -24,7 +24,7 @@ import { EventName } from '../../../telemetry/constants';
 import { sendTelemetryEvent } from '../../../telemetry/index';
 import { TestDiscoverytTelemetry, TestRunTelemetry } from '../../../telemetry/types';
 import { IPythonUnitTestMessage, IUnitTestDiagnosticService, WorkspaceTestStatus } from '../../types';
-import { copyTestResults } from '../testUtils';
+import { copyDesiredTestResults } from '../testUtils';
 import { CANCELLATION_REASON, CommandSource, TEST_OUTPUT_CHANNEL } from './../constants';
 import {
     ITestCollectionStorageService,
@@ -175,7 +175,7 @@ export abstract class BaseTestManager implements ITestManager {
                 if (clearTestStatus) {
                     this.resetTestResults();
                 } else if (existingTests) {
-                    copyTestResults(existingTests, tests);
+                    copyDesiredTestResults(existingTests, tests);
                     this._testResultsService.updateResults(tests);
                 }
                 this.testCollectionStorage.storeTests(wkspace, tests);

src/client/unittests/common/services/testResultsService.ts

@@ -38,6 +38,7 @@ export class TestResultsService implements ITestResultsService {
         let totalTime = 0;
         let allFilesPassed = true;
         let noFilesRan = true;
+        testFolder.functionsPassed = testFolder.functionsFailed = testFolder.functionsDidNotRun = 0;
 
         testFolder.testFiles.forEach(fl => {
             totalTime += fl.time;
@@ -50,6 +51,7 @@ export class TestResultsService implements ITestResultsService {
 
             testFolder.functionsFailed! += fl.functionsFailed!;
             testFolder.functionsPassed! += fl.functionsPassed!;
+            testFolder.functionsDidNotRun! += fl.functionsDidNotRun!;
         });
 
         let allFoldersPassed = true;
@@ -67,6 +69,7 @@ export class TestResultsService implements ITestResultsService {
 
             testFolder.functionsFailed! += folder.functionsFailed!;
             testFolder.functionsPassed! += folder.functionsPassed!;
+            testFolder.functionsDidNotRun! += folder.functionsDidNotRun!;
         });
 
         testFolder.time = totalTime;
@@ -82,6 +85,7 @@ export class TestResultsService implements ITestResultsService {
         let totalTime = 0;
         let allFunctionsPassed = true;
         let noFunctionsRan = true;
+        test.functionsPassed = test.functionsFailed = test.functionsDidNotRun = 0;
 
         test.functions.forEach(fn => {
             totalTime += fn.time;
@@ -93,6 +97,8 @@ export class TestResultsService implements ITestResultsService {
                     test.functionsFailed! += 1;
                     allFunctionsPassed = false;
                 }
+            } else {
+                test.functionsDidNotRun! += 1;
             }
         });
 
@@ -111,6 +117,7 @@ export class TestResultsService implements ITestResultsService {
 
             test.functionsFailed! += suite.functionsFailed!;
             test.functionsPassed! += suite.functionsPassed!;
+            test.functionsDidNotRun! += suite.functionsDidNotRun!;
         });
 
         test.time = totalTime;

src/client/unittests/common/testUtils.ts

@@ -454,7 +454,7 @@ export function getChildren(item: TestDataItem): TestDataItem[] {
     }
 }
 
-export function copyTestResults(source: Tests, target: Tests): void {
+export function copyDesiredTestResults(source: Tests, target: Tests): void {
     copyResultsForFolders(source.testFolders, target.testFolders);
 }
 
@@ -466,6 +466,8 @@ function copyResultsForFolders(source: TestFolder[], target: TestFolder[]): void
         }
         copyValueTypes<TestFolder>(sourceFolder, targetFolder);
         copyResultsForFiles(sourceFolder.testFiles, targetFolder.testFiles);
+        // These should be reinitialized
+        targetFolder.functionsPassed = targetFolder.functionsDidNotRun = targetFolder.functionsFailed = 0;
     });
 }
 function copyResultsForFiles(source: TestFile[], target: TestFile[]): void {
@@ -477,6 +479,8 @@ function copyResultsForFiles(source: TestFile[], target: TestFile[]): void {
         copyValueTypes<TestFile>(sourceFile, targetFile);
         copyResultsForFunctions(sourceFile.functions, targetFile.functions);
         copyResultsForSuites(sourceFile.suites, targetFile.suites);
+        // These should be reinitialized
+        targetFile.functionsPassed = targetFile.functionsDidNotRun = targetFile.functionsFailed = 0;
     });
 }
 
@@ -501,6 +505,8 @@ function copyResultsForSuites(source: TestSuite[], target: TestSuite[]): void {
         copyValueTypes<TestSuite>(sourceSuite, targetSuite);
         copyResultsForFunctions(sourceSuite.functions, targetSuite.functions);
         copyResultsForSuites(sourceSuite.suites, targetSuite.suites);
+        // These should be reinitialized
+        targetSuite.functionsPassed = targetSuite.functionsDidNotRun = targetSuite.functionsFailed = 0;
     });
 }
 

src/client/unittests/explorer/testTreeViewItem.ts

@@ -72,7 +72,7 @@ export class TestTreeItem extends TreeItem {
             return '';
         }
         const result = this.data as TestResult;
-        if (!result.status || result.status === TestStatus.Idle || result.status === TestStatus.Unknown || result.status === TestStatus.Skipped){
+        if (!result.status || result.status === TestStatus.Idle || result.status === TestStatus.Unknown || result.status === TestStatus.Skipped) {
             return '';
         }
         if (this.testType !== TestType.testFunction) {

src/test/unittests/common/services/storageService.unit.test.ts

@@ -4,7 +4,7 @@
 'use strict';
 
 import * as assert from 'assert';
-import { copyTestResults } from '../../../../client/unittests/common/testUtils';
+import { copyDesiredTestResults } from '../../../../client/unittests/common/testUtils';
 import { FlattenedTestFunction, FlattenedTestSuite, TestFile, TestFolder, TestFunction, Tests, TestStatus, TestSuite, TestType } from '../../../../client/unittests/common/types';
 import { createMockTestDataItem } from '../testUtils.unit.test';
 
@@ -114,7 +114,7 @@ suite('Unit Tests - Storage Service', () => {
         assert.notDeepEqual(testData1.testFunctions[1].testFunction, testData2.testFunctions[1].testFunction);
         assert.notDeepEqual(testData1.testFunctions[2].testFunction, testData2.testFunctions[2].testFunction);
 
-        copyTestResults(testData1, testData2);
+        copyDesiredTestResults(testData1, testData2);
 
         // Function 1 is in a different suite now, hence should not get updated.
         assert.notDeepEqual(testData1.testFunctions[0].testFunction, testData2.testFunctions[0].testFunction);