FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test_runner: format coverage report for tap reporter · nodejs/node@3ccd5fa · GitHub

/ node Public

Commit 3ccd5fa

Browse files
pulkit-30
authored and
RafaelGSS
committed
test_runner: format coverage report for tap reporter
PR-URL: #51119 Reviewed-By: Raz Luvaton <rluvaton@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
1 parent ed07075 commit 3ccd5fa

2 files changed

Lines changed: 22 additions & 18 deletions

File tree

‎lib/internal/test_runner/reporter/tap.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ async function * tapReporter(source) {
5858
yield `${indent(data.nesting)}# ${tapEscape(data.message)}\n`;
5959
break;
6060
case 'test:coverage':
61-
yield getCoverageReport(indent(data.nesting), data.summary, '# ', '');
61+
yield getCoverageReport(indent(data.nesting), data.summary, '# ', '', true);
6262
break;
6363
}
6464
}

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

Lines changed: 21 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -22,16 +22,21 @@ function findCoverageFileForPid(pid) {
2222
}
2323

2424
function getTapCoverageFixtureReport() {
25+
/* eslint-disable max-len */
2526
const report = [
2627
'# start of coverage report',
27-
'# file | line % | branch % | funcs % | uncovered lines',
28-
'# test/fixtures/test-runner/coverage.js | 78.65 | 38.46 | 60.00 | 12, ' +
29-
'13, 16, 17, 18, 19, 20, 21, 22, 27, 39, 43, 44, 61, 62, 66, 67, 71, 72',
30-
'# test/fixtures/test-runner/invalid-tap.js | 100.00 | 100.00 | 100.00 | ',
31-
'# test/fixtures/v8-coverage/throw.js | 71.43 | 50.00 | 100.00 | 5, 6',
32-
'# all files | 78.35 | 43.75 | 60.00 |',
28+
'# -------------------------------------------------------------------------------------------------------------------',
29+
'# file | line % | branch % | funcs % | uncovered lines',
30+
'# -------------------------------------------------------------------------------------------------------------------',
31+
'# test/fixtures/test-runner/coverage.js | 78.65 | 38.46 | 60.00 | 12-13 16-22 27 39 43-44 61-62 66-67 71-72',
32+
'# test/fixtures/test-runner/invalid-tap.js | 100.00 | 100.00 | 100.00 | ',
33+
'# test/fixtures/v8-coverage/throw.js | 71.43 | 50.00 | 100.00 | 5-6',
34+
'# -------------------------------------------------------------------------------------------------------------------',
35+
'# all files | 78.35 | 43.75 | 60.00 |',
36+
'# -------------------------------------------------------------------------------------------------------------------',
3337
'# end of coverage report',
3438
].join('\n');
39+
/* eslint-enable max-len */
3540

3641
if (common.isWindows) {
3742
return report.replaceAll('/', '\\');
@@ -88,7 +93,6 @@ test('test tap coverage reporter', skipIfNoInspector, async (t) => {
8893
const options = { env: { ...process.env, NODE_V8_COVERAGE: tmpdir.path } };
8994
const result = spawnSync(process.execPath, args, options);
9095
const report = getTapCoverageFixtureReport();
91-
9296
assert(result.stdout.toString().includes(report));
9397
assert.strictEqual(result.stderr.toString(), '');
9498
assert.strictEqual(result.status, 0);
@@ -152,16 +156,16 @@ test('single process coverage is the same with --test', skipIfNoInspector, () =>
152156
test('coverage is combined for multiple processes', skipIfNoInspector, () => {
153157
let report = [
154158
'# start of coverage report',
155-
'# file | line % | branch % | funcs % | uncovered lines',
156-
'# common.js | 89.86 | ' +
157-
'62.50 | 100.00 | 8, 13, 14, 18, 34, 35, 53',
158-
'# first.test.js | 83.33 | ' +
159-
'100.00 | 50.00 | 5, 6',
160-
'# second.test.js | 100.00 ' +
161-
'| 100.00 | 100.00 | ',
162-
'# third.test.js | 100.00 | ' +
163-
'100.00 | 100.00 | ',
164-
'# all files | 92.11 | 72.73 | 88.89 |',
159+
'# -------------------------------------------------------------------',
160+
'# file | line % | branch % | funcs % | uncovered lines',
161+
'# -------------------------------------------------------------------',
162+
'# common.js | 89.86 | 62.50 | 100.00 | 8 13-14 18 34-35 53',
163+
'# first.test.js | 83.33 | 100.00 | 50.00 | 5-6',
164+
'# second.test.js | 100.00 | 100.00 | 100.00 | ',
165+
'# third.test.js | 100.00 | 100.00 | 100.00 | ',
166+
'# -------------------------------------------------------------------',
167+
'# all files | 92.11 | 72.73 | 88.89 |',
168+
'# -------------------------------------------------------------------',
165169
'# end of coverage report',
166170
].join('\n');
167171

0 commit comments

Comments
 (0)
Back | FazBrowse Home | New Git URL