Merge pull request #2579 from rchande/handleEmptyTests

Handle empty tests

Author: Ravi Chande

src/observers/DotnetTestLoggerObserver.ts

@@ -72,7 +72,9 @@ export default class DotNetTestLoggerObserver extends BaseLoggerObserver {
     private handleReportDotnetTestResults(event: ReportDotNetTestResults) {
         this.logger.appendLine("----- Test Execution Summary -----");
         this.logger.appendLine('');
-        const results = event.results;
+        
+        // Omnisharp returns null results if there are build failures
+        const results = event.results || [];
         const totalTests = results.length;
 
         let totalPassed = 0, totalFailed = 0, totalSkipped = 0;

test/unitTests/logging/DotnetTestLoggerObserver.test.ts

@@ -107,6 +107,11 @@ suite(`${DotNetTestLoggerObserver.name}`, () => {
                 result.StandardError.forEach(message => expect(appendedMessage).to.contain(message));
             });
         });
+          
+        test(`Can handle malformed results`, () => {
+            observer.post(new ReportDotNetTestResults([]));
+            expect(appendedMessage).to.contain("----- Test Execution Summary -----\n\nTotal tests: 0. Passed: 0. Failed: 0. Skipped: 0");
+        });
     });
 });