Remove time from all nodes except the tests (#5890)

Kartik Raj · web-flow · commit 737ff6e6a89b · 2019-06-03T17:03:44.000-07:00
* Remove time for non-function nodes

* News entry
diff --git a/news/1 Enhancements/5120.md b/news/1 Enhancements/5120.md
@@ -0,0 +1 @@
+Test Explorer : Remove time from all nodes except the tests
diff --git a/src/client/testing/common/services/testResultsService.ts b/src/client/testing/common/services/testResultsService.ts
@@ -38,13 +38,10 @@ export class TestResultsService implements ITestResultsService {
         }
         let allChildrenPassed = true;
         let noChildrenRan = true;
-        test.time = test.functionsPassed = test.functionsFailed = test.functionsDidNotRun = 0;
+        test.functionsPassed = test.functionsFailed = test.functionsDidNotRun = 0;
 
         const children = getChildren(test);
         children.forEach(child => {
-            if (child.time) {
-                test.time! += child.time;
-            }
             if (getTestType(child) === TestType.testFunction) {
                 if (typeof child.passed === 'boolean') {
                     noChildrenRan = false;
diff --git a/src/client/testing/explorer/testTreeViewItem.ts b/src/client/testing/explorer/testTreeViewItem.ts
@@ -82,9 +82,9 @@ export class TestTreeItem extends TreeItem {
                 return '';
             }
             if (result.functionsDidNotRun) {
-                return `${result.functionsFailed} failed, ${result.functionsDidNotRun} not run and ${result.functionsPassed} passed in ${+result.time.toFixed(3)} seconds`;
+                return `${result.functionsFailed} failed, ${result.functionsDidNotRun} not run and ${result.functionsPassed} passed`;
             }
-            return `${result.functionsFailed} failed, ${result.functionsPassed} passed in ${+result.time.toFixed(3)} seconds`;
+            return `${result.functionsFailed} failed, ${result.functionsPassed} passed`;
         }
         switch (this.data.status) {
             case TestStatus.Error:

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+Test Explorer : Remove time from all nodes except the tests`
Original file line number	Diff line number	Diff line change
`@@ -82,9 +82,9 @@ export class TestTreeItem extends TreeItem {`
`82`	`82`	`return '';`
`83`	`83`	`}`
`84`	`84`	`if (result.functionsDidNotRun) {`
`85`		- return `${result.functionsFailed} failed, ${result.functionsDidNotRun} not run and ${result.functionsPassed} passed in ${+result.time.toFixed(3)} seconds`;
	`85`	+ return `${result.functionsFailed} failed, ${result.functionsDidNotRun} not run and ${result.functionsPassed} passed`;
`86`	`86`	`}`
`87`		- return `${result.functionsFailed} failed, ${result.functionsPassed} passed in ${+result.time.toFixed(3)} seconds`;
	`87`	+ return `${result.functionsFailed} failed, ${result.functionsPassed} passed`;
`88`	`88`	`}`
`89`	`89`	`switch (this.data.status) {`
`90`	`90`	`case TestStatus.Error:`