Merge pull request #1436 from DustinCampbell/improve-test-output

Add summary line after test run

Author: DustinCampbell

src/features/dotnetTest.ts

@@ -55,13 +55,27 @@ export function runDotnetTest(testMethod: string, fileName: string, testFramewor
 
     serverUtils.runTest(server, request)
         .then(response => {
-            if (response.Pass) {
-                output.appendLine('Test passed \n');
-            }
-            else {
-                output.appendLine('Test failed \n');
+            const totalTests = response.Results.length;
+
+            let totalPassed = 0, totalFailed = 0, totalSkipped = 0;
+            for (let result of response.Results) {
+                switch (result.Outcome) {
+                    case protocol.V2.TestOutcomes.Failed:
+                        totalFailed += 1;
+                        break;
+                    case protocol.V2.TestOutcomes.Passed:
+                        totalPassed += 1;
+                        break;
+                    case protocol.V2.TestOutcomes.Skipped:
+                        totalSkipped += 1;
+                        break;
+                }
             }
 
+            output.appendLine('');
+            output.appendLine(`Total tests: ${totalTests}. Passed: ${totalPassed}. Failed: ${totalFailed}. Skipped: ${totalSkipped}`);
+            output.appendLine('');
+
             disposable.dispose();
         },
         reason => {

src/omnisharp/protocol.ts

@@ -538,7 +538,15 @@ export namespace V2 {
         TestFrameworkName: string;
     }
 
-    export interface DotNetResult {
+    export module TestOutcomes {
+        export const None = 'none';
+        export const Passed = 'passed';
+        export const Failed = 'failed';
+        export const Skipped = 'skipped';
+        export const NotFound = 'notfound';
+    }
+
+    export interface DotNetTestResult {
         MethodName: string;
         Outcome: string;
         ErrorMessage: string;
@@ -548,6 +556,7 @@ export namespace V2 {
     export interface RunTestResponse {
         Failure: string;
         Pass: boolean;
+        Results: DotNetTestResult[];
     }
 
     export interface TestMessageEvent {