test_runner: print coverage and

   diagnostics with dot reporter

Author: madkate42

lib/internal/test_runner/reporter/dot.js

@@ -4,12 +4,19 @@ const {
   MathMax,
 } = primordials;
 const colors = require('internal/util/colors');
-const { formatTestReport } = require('internal/test_runner/reporter/utils');
+const { getCoverageReport } = require('internal/test_runner/utils');
+const {
+  formatTestReport,
+  reporterColorMap,
+  reporterUnicodeSymbolMap,
+} = require('internal/test_runner/reporter/utils');
 
 module.exports = async function* dot(source) {
   let count = 0;
   let columns = getLineLength();
   const failedTests = [];
+  const diagnostics = [];
+  let coverage;
   for await (const { type, data } of source) {
     if (type === 'test:pass') {
       yield `${colors.green}.${colors.reset}`;
@@ -25,6 +32,12 @@ module.exports = async function* dot(source) {
       columns = getLineLength();
       count = 0;
     }
+    if (type === 'test:diagnostic') {
+      ArrayPrototypePush(diagnostics, data);
+    }
+    if (type === 'test:coverage') {
+      coverage = data;
+    }
   }
   yield '\n';
   if (failedTests.length > 0) {
@@ -33,6 +46,13 @@ module.exports = async function* dot(source) {
       yield formatTestReport('test:fail', test);
     }
   }
+  for (const diagnostic of diagnostics) {
+    const color = reporterColorMap[diagnostic.level] || reporterColorMap['test:diagnostic'];
+    yield `${color}${reporterUnicodeSymbolMap['test:diagnostic']}${diagnostic.message}${colors.white}\n`;
+  }
+  if (coverage) {
+    yield getCoverageReport('', coverage.summary, reporterUnicodeSymbolMap['test:coverage'], colors.blue, true);
+  }
 };
 
 function getLineLength() {

test/parallel/test-runner-coverage-thresholds.js

@@ -143,6 +143,23 @@ for (const coverage of coverages) {
     assert(!findCoverageFileForPid(result.pid));
   });
 
+  test(`test failing ${coverage.flag} with dot reporter`, () => {
+    const result = spawnSync(process.execPath, [
+      '--test',
+      '--experimental-test-coverage',
+      '--test-coverage-exclude=!test/**',
+      `${coverage.flag}=99`,
+      '--test-reporter', 'dot',
+      fixture,
+    ]);
+
+    const stdout = result.stdout.toString();
+    assert.match(stdout, RegExp(`Error: ${coverage.actual.toFixed(2)}% ${coverage.name} coverage does not meet threshold of 99%`));
+    assert.match(stdout, /start of coverage report/);
+    assert.strictEqual(result.status, 1);
+    assert(!findCoverageFileForPid(result.pid));
+  });
+
   test(`test out-of-range ${coverage.flag} (too high)`, () => {
     const result = spawnSync(process.execPath, [
       '--test',