Tests

dotnet · May 24, 2018 · a0eeb96 · a0eeb96
1 parent 6d69083
commit a0eeb96
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 8 deletions.
diff --git a/src/observers/DotnetTestLoggerObserver.ts b/src/observers/DotnetTestLoggerObserver.ts
@@ -80,7 +80,7 @@ export default class DotNetTestLoggerObserver extends BaseLoggerObserver {
     }
 
     private logTestResult(result: protocol.V2.DotNetTestResult) {
-        this.logger.appendLine(`${result.MethodName}: ${processOutcome(result.Outcome)}`);
+        this.logger.appendLine(`${result.MethodName}:`);
         this.logger.increaseIndent();
         this.logger.appendLine(`Outcome: ${processOutcome(result.Outcome)}`);
         if (result.ErrorMessage) {
@@ -91,6 +91,6 @@ export default class DotNetTestLoggerObserver extends BaseLoggerObserver {
     }
 }
 
-function processOutcome(input: string) {
+export function processOutcome(input: string) {
     return input.charAt(0).toUpperCase() + input.slice(1);
 }
diff --git a/test/unitTests/logging/DotnetTestLoggerObserver.test.ts b/test/unitTests/logging/DotnetTestLoggerObserver.test.ts
@@ -3,12 +3,15 @@
  *  Licensed under the MIT License. See License.txt in the project root for license information.
  *--------------------------------------------------------------------------------------------*/
 
-import { expect } from 'chai';
+import * as chai from 'chai';
 import { getNullChannel } from '../testAssets/Fakes';
 import { EventWithMessage, DotNetTestDebugWarning, DotNetTestDebugStart, BaseEvent, DotNetTestRunStart, DotNetTestDebugProcessStart, DotNetTestMessage, DotNetTestDebugComplete, ReportDotNetTestResults } from '../../../src/omnisharp/loggingEvents';
 import DotNetTestLoggerObserver from '../../../src/observers/DotnetTestLoggerObserver';
 import * as protocol from '../../../src/omnisharp/protocol';
 
+const expect = chai.expect;
+chai.use(require('chai-string'));
+
 suite(`${DotNetTestLoggerObserver.name}`, () => {
     let appendedMessage: string;
     let observer = new DotNetTestLoggerObserver({
@@ -65,16 +68,19 @@ suite(`${DotNetTestLoggerObserver.name}`, () => {
         test(`Displays the outcome of each test`, () => {
             observer.post(event);
             event.results.forEach(result => {
-                expect(appendedMessage).to.contain(`${result.MethodName}: ${result.Outcome}`);
+                expect(appendedMessage).to.containIgnoreCase(`${result.MethodName}:\n    Outcome: ${result.Outcome}`);
             });
         });
 
         test(`Displays the total outcome`, () => {
             observer.post(event);
-            expect(appendedMessage).to.contain(`Total tests: ${event.results.length}. Passed: 1. Failed: 2. Skipped: 1`);
-            event.results.forEach(result => {
-                expect(appendedMessage).to.contain(`${result.MethodName}: ${result.Outcome}`);
-            });
+            expect(appendedMessage).to.contain(`Total tests: 4. Passed: 1. Failed: 2. Skipped: 1`);
+        });
+
+        test('Displays the error message if any is present', () => {
+            observer.post(event);
+            expect(appendedMessage).to.contain("foo:\n    Outcome: Failed\n    Error Message: assertion failed");
+            expect(appendedMessage).to.contain("failinator:\n    Outcome: Failed\n    Error Message: error occured");
         });
     });
 });