test_runner: remove console.logs and fix lint errors

Author: philnash

lib/internal/test_runner/coverage.js

@@ -190,7 +190,7 @@ class TestCoverage {
           ArrayPrototypePush(functionReports, {
             __proto__: null,
             name: functions[j].functionName,
-            count: MathMax(...ArrayPrototypeMap(ranges, r => r.count)),
+            count: MathMax(...ArrayPrototypeMap(ranges, (r) => r.count)),
             line: range.lines[0].line,
           });
 
@@ -215,7 +215,7 @@ class TestCoverage {
               __proto__: null,
               line: line.line,
               count: line.count,
-            })
+            });
           }
         } else {
           ArrayPrototypePush(lineReports, {

lib/internal/test_runner/utils.js

@@ -35,7 +35,6 @@ const {
   kIsNodeError,
 } = require('internal/errors');
 const { compose } = require('stream');
-const console = require('console');
 
 const coverageColors = {
   __proto__: null,
@@ -300,7 +299,7 @@ function formatLinesToRanges(values) {
 }
 
 function getUncoveredLines(lines) {
-  return ArrayPrototypeFlatMap(lines, (line) => line.count === 0 ? line.line : []);
+  return ArrayPrototypeFlatMap(lines, (line) => (line.count === 0 ? line.line : []));
 }
 
 function formatUncoveredLines(lines, table) {
@@ -332,7 +331,7 @@ function getCoverageReport(pad, summary, symbol, color, table) {
     const columnsWidth = ArrayPrototypeReduce(columnPadLengths, (acc, columnPadLength) => acc + columnPadLength + 3, 0);
 
     uncoveredLinesPadLength = table && ArrayPrototypeReduce(summary.files, (acc, file) => {
-      return MathMax(acc, formatUncoveredLines(getUncoveredLines(file.lines), table).length)
+      return MathMax(acc, formatUncoveredLines(getUncoveredLines(file.lines), table).length);
     }, 0);
     uncoveredLinesPadLength = MathMax(uncoveredLinesPadLength, 'uncovered lines'.length);
     const uncoveredLinesWidth = uncoveredLinesPadLength + 2;

test/parallel/test-runner-coverage.js

@@ -89,11 +89,6 @@ test('test tap coverage reporter', skipIfNoInspector, async (t) => {
     const result = spawnSync(process.execPath, args, options);
     const report = getTapCoverageFixtureReport();
 
-    console.log("=======")
-    console.log("RESULT\n", result.stdout.toString())
-    console.log("EXPECTED\n", report)
-    console.log("=======")
-
     assert(result.stdout.toString().includes(report));
     assert.strictEqual(result.stderr.toString(), '');
     assert.strictEqual(result.status, 0);