test_runner: format coverage report for tap reporter #51119

pulkit-30 · 2023-12-11T16:35:48Z

this PR enhances the tap coverage report output (similar to the spec reporter).

code:

import assert from "node:assert";
import { test } from "node:test";

test("some test", () => {
    assert.strictEqual(1,1)
})

before:

TAP version 13
# Subtest: some test
ok 1 - some test
  ---
  duration_ms: 1.008875
  ...
1..1
# tests 1
# suites 0
# pass 1
# fail 0
# cancelled 0
# skipped 0
# todo 0
# duration_ms 9.619875
# start of coverage report
# file | line % | branch % | funcs % | uncovered lines
# index.mjs | 100.00 | 100.00 | 100.00 | 
# all files | 100.00 | 100.00 | 100.00 |
# end of coverage report

now:

TAP version 13
# Subtest: some test
ok 1 - some test
  ---
  duration_ms: 1.41925
  ...
1..1
# tests 1
# suites 0
# pass 1
# fail 0
# cancelled 0
# skipped 0
# todo 0
# duration_ms 12.865292
# start of coverage report
# ----------------------------------------------------------
# file      | line % | branch % | funcs % | uncovered lines
# ----------------------------------------------------------
# index.mjs | 100.00 |   100.00 |  100.00 | 
# ----------------------------------------------------------
# all files | 100.00 |   100.00 |  100.00 |
# ----------------------------------------------------------
# end of coverage report

nodejs-github-bot · 2023-12-11T16:35:54Z

Review requested:

@nodejs/test_runner

rluvaton

I verified that this change still follows the tap specification and other tap parsers are working with this as well

Great job

test/parallel/test-runner-coverage.js

nodejs-github-bot · 2023-12-11T20:28:16Z

CI: https://ci.nodejs.org/job/node-test-pull-request/56234/

nodejs-github-bot · 2023-12-12T12:11:28Z

CI: https://ci.nodejs.org/job/node-test-pull-request/56246/

nodejs-github-bot · 2023-12-13T16:38:46Z

Commit Queue failed

- Loading data for nodejs/node/pull/51119 ✔ Done loading data for nodejs/node/pull/51119 ----------------------------------- PR info ------------------------------------ Title test_runner: format coverage report for tap reporter (#51119) Author Pulkit Gupta (@pulkit-30) Branch pulkit-30:tap-coverage -> nodejs:main Labels author ready, needs-ci, test_runner Commits 2 - test_runner: format coverage report for tap reporter - fix lint Committers 1 - pulkit-30 PR-URL: https://github.com/nodejs/node/pull/51119 Reviewed-By: Raz Luvaton Reviewed-By: Luigi Pinca Reviewed-By: Moshe Atlow ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/51119 Reviewed-By: Raz Luvaton Reviewed-By: Luigi Pinca Reviewed-By: Moshe Atlow -------------------------------------------------------------------------------- ℹ This PR was created on Mon, 11 Dec 2023 16:35:48 GMT ✔ Approvals: 3 ✔ - Raz Luvaton (@rluvaton): https://github.com/nodejs/node/pull/51119#pullrequestreview-1776094670 ✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/51119#pullrequestreview-1776238268 ✔ - Moshe Atlow (@MoLow) (TSC): https://github.com/nodejs/node/pull/51119#pullrequestreview-1776345233 ✔ Last GitHub CI successful ℹ Last Full PR CI on 2023-12-12T12:11:28Z: https://ci.nodejs.org/job/node-test-pull-request/56246/ - Querying data for job/node-test-pull-request/56246/ ✔ Last Jenkins CI successful -------------------------------------------------------------------------------- ✔ No git cherry-pick in progress ✔ No git am in progress ✔ No git rebase in progress -------------------------------------------------------------------------------- - Bringing origin/main up to date... From https://github.com/nodejs/node * branch main -> FETCH_HEAD ✔ origin/main is now up-to-date - Downloading patch for 51119 From https://github.com/nodejs/node * branch refs/pull/51119/merge -> FETCH_HEAD ✔ Fetched commits as 228bc5c457a9..eb98b15f1352 -------------------------------------------------------------------------------- [main 787445b822] test_runner: format coverage report for tap reporter Author: pulkit-30 Date: Mon Dec 11 22:02:35 2023 +0530 2 files changed, 20 insertions(+), 18 deletions(-) [main e1358436fb] fix lint Author: pulkit-30 Date: Mon Dec 11 23:33:41 2023 +0530 1 file changed, 2 insertions(+) ✔ Patches applied There are 2 commits in the PR. Attempting autorebase. Rebasing (2/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
test_runner: format coverage report for tap reporter

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]

[detached HEAD caac4b4984] test_runner: format coverage report for tap reporter
Author: pulkit-30 [email protected]
Date: Mon Dec 11 22:02:35 2023 +0530
2 files changed, 20 insertions(+), 18 deletions(-)
Rebasing (3/4)
Rebasing (4/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
fix lint

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]

[detached HEAD 2079f69752] fix lint
Author: pulkit-30 [email protected]
Date: Mon Dec 11 23:33:41 2023 +0530
1 file changed, 2 insertions(+)

Successfully rebased and updated refs/heads/main.

ℹ Add commit-queue-squash label to land the PR as one commit, or commit-queue-rebase to land as separate commits.

https://github.com/nodejs/node/actions/runs/7198280065

nodejs-github-bot · 2023-12-15T06:02:36Z

Commit Queue failed

- Loading data for nodejs/node/pull/51119 ✔ Done loading data for nodejs/node/pull/51119 ----------------------------------- PR info ------------------------------------ Title test_runner: format coverage report for tap reporter (#51119) Author Pulkit Gupta (@pulkit-30) Branch pulkit-30:tap-coverage -> nodejs:main Labels author ready, needs-ci, test_runner Commits 2 - test_runner: format coverage report for tap reporter - fix lint Committers 1 - pulkit-30 PR-URL: https://github.com/nodejs/node/pull/51119 Reviewed-By: Raz Luvaton Reviewed-By: Luigi Pinca Reviewed-By: Moshe Atlow Reviewed-By: Debadree Chatterjee ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/51119 Reviewed-By: Raz Luvaton Reviewed-By: Luigi Pinca Reviewed-By: Moshe Atlow Reviewed-By: Debadree Chatterjee -------------------------------------------------------------------------------- ℹ This PR was created on Mon, 11 Dec 2023 16:35:48 GMT ✔ Approvals: 4 ✔ - Raz Luvaton (@rluvaton): https://github.com/nodejs/node/pull/51119#pullrequestreview-1776094670 ✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/51119#pullrequestreview-1776238268 ✔ - Moshe Atlow (@MoLow) (TSC): https://github.com/nodejs/node/pull/51119#pullrequestreview-1776345233 ✔ - Debadree Chatterjee (@debadree25): https://github.com/nodejs/node/pull/51119#pullrequestreview-1783172811 ✔ Last GitHub CI successful ℹ Last Full PR CI on 2023-12-13T16:38:46Z: https://ci.nodejs.org/job/node-test-pull-request/56246/ - Querying data for job/node-test-pull-request/56246/ ✔ Last Jenkins CI successful -------------------------------------------------------------------------------- ✔ No git cherry-pick in progress ✔ No git am in progress ✔ No git rebase in progress -------------------------------------------------------------------------------- - Bringing origin/main up to date... From https://github.com/nodejs/node * branch main -> FETCH_HEAD ✔ origin/main is now up-to-date - Downloading patch for 51119 From https://github.com/nodejs/node * branch refs/pull/51119/merge -> FETCH_HEAD ✔ Fetched commits as 5ac658102d8f..eb98b15f1352 -------------------------------------------------------------------------------- [main 867deb34eb] test_runner: format coverage report for tap reporter Author: pulkit-30 Date: Mon Dec 11 22:02:35 2023 +0530 2 files changed, 20 insertions(+), 18 deletions(-) [main 43b7a78955] fix lint Author: pulkit-30 Date: Mon Dec 11 23:33:41 2023 +0530 1 file changed, 2 insertions(+) ✔ Patches applied There are 2 commits in the PR. Attempting autorebase. Rebasing (2/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
test_runner: format coverage report for tap reporter

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]
Reviewed-By: Debadree Chatterjee [email protected]

[detached HEAD dced0311b4] test_runner: format coverage report for tap reporter
Author: pulkit-30 [email protected]
Date: Mon Dec 11 22:02:35 2023 +0530
2 files changed, 20 insertions(+), 18 deletions(-)
Rebasing (3/4)
Rebasing (4/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
fix lint

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]
Reviewed-By: Debadree Chatterjee [email protected]

[detached HEAD 95f42e504f] fix lint
Author: pulkit-30 [email protected]
Date: Mon Dec 11 23:33:41 2023 +0530
1 file changed, 2 insertions(+)

Successfully rebased and updated refs/heads/main.

ℹ Add commit-queue-squash label to land the PR as one commit, or commit-queue-rebase to land as separate commits.

https://github.com/nodejs/node/actions/runs/7218561408

nodejs-github-bot · 2023-12-15T06:53:10Z

Landed in 452d29c

PR-URL: #51119 Reviewed-By: Raz Luvaton <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Debadree Chatterjee <[email protected]>

richardlau · 2024-03-25T18:14:01Z

This doesn't land cleanly on v20.x-staging and will need a manual backport.

atlowChemi · 2024-03-29T05:23:48Z

@richardlau backported in #52260

PR-URL: nodejs#51119 Reviewed-By: Raz Luvaton <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Debadree Chatterjee <[email protected]>

PR-URL: #51119 Reviewed-By: Raz Luvaton <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Debadree Chatterjee <[email protected]>

PR-URL: #51119 Backport-PR-URL: #52260 Reviewed-By: Raz Luvaton <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Debadree Chatterjee <[email protected]>

test_runner: format coverage report for tap reporter

e9d638f

nodejs-github-bot added needs-ci PRs that need a full CI run. test_runner Issues and PRs related to the test runner subsystem. labels Dec 11, 2023

rluvaton approved these changes Dec 11, 2023

View reviewed changes

test/parallel/test-runner-coverage.js Show resolved Hide resolved

fix lint

eb98b15

rluvaton approved these changes Dec 11, 2023

View reviewed changes

rluvaton added the request-ci Add this label to start a Jenkins CI on a PR. label Dec 11, 2023

github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 11, 2023

lpinca approved these changes Dec 11, 2023

View reviewed changes

rluvaton added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Dec 11, 2023

MoLow approved these changes Dec 11, 2023

View reviewed changes

debadree25 added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 13, 2023

nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Dec 13, 2023

debadree25 removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Dec 15, 2023

debadree25 approved these changes Dec 15, 2023

View reviewed changes

debadree25 added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 15, 2023

nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Dec 15, 2023

nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 15, 2023

nodejs-github-bot merged commit 452d29c into nodejs:main Dec 15, 2023
72 checks passed

RafaelGSS mentioned this pull request Dec 15, 2023

v21.5.0 proposal #51166

Merged

richardlau added the backport-requested-v20.x PRs awaiting manual backport to the v20.x-staging branch. label Mar 25, 2024

atlowChemi mentioned this pull request Mar 29, 2024

[v20.x backport] test_runner: format coverage report for tap reporter #52260

Closed

atlowChemi added the backport-open-v20.x Indicate that the PR has an open backport label Mar 29, 2024

marco-ippolito mentioned this pull request May 2, 2024

v20.13.0 proposal #52793

Merged

targos added backported-to-v20.x PRs backported to the v20.x-staging branch. and removed backport-requested-v20.x PRs awaiting manual backport to the v20.x-staging branch. backport-open-v20.x Indicate that the PR has an open backport labels Sep 30, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

test_runner: format coverage report for tap reporter #51119

test_runner: format coverage report for tap reporter #51119

pulkit-30 commented Dec 11, 2023

nodejs-github-bot commented Dec 11, 2023

rluvaton left a comment

nodejs-github-bot commented Dec 11, 2023

nodejs-github-bot commented Dec 12, 2023

nodejs-github-bot commented Dec 13, 2023

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]

Successfully rebased and updated refs/heads/main.

nodejs-github-bot commented Dec 15, 2023

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]
Reviewed-By: Debadree Chatterjee [email protected]

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]
Reviewed-By: Debadree Chatterjee [email protected]

Successfully rebased and updated refs/heads/main.

nodejs-github-bot commented Dec 15, 2023

richardlau commented Mar 25, 2024

atlowChemi commented Mar 29, 2024

test_runner: format coverage report for tap reporter #51119

test_runner: format coverage report for tap reporter #51119

Conversation

pulkit-30 commented Dec 11, 2023

nodejs-github-bot commented Dec 11, 2023

rluvaton left a comment

Choose a reason for hiding this comment

nodejs-github-bot commented Dec 11, 2023

nodejs-github-bot commented Dec 12, 2023

nodejs-github-bot commented Dec 13, 2023

PR-URL: #51119 Reviewed-By: Raz Luvaton [email protected] Reviewed-By: Luigi Pinca [email protected] Reviewed-By: Moshe Atlow [email protected]

PR-URL: #51119 Reviewed-By: Raz Luvaton [email protected] Reviewed-By: Luigi Pinca [email protected] Reviewed-By: Moshe Atlow [email protected]

Successfully rebased and updated refs/heads/main.

nodejs-github-bot commented Dec 15, 2023

PR-URL: #51119 Reviewed-By: Raz Luvaton [email protected] Reviewed-By: Luigi Pinca [email protected] Reviewed-By: Moshe Atlow [email protected] Reviewed-By: Debadree Chatterjee [email protected]

PR-URL: #51119 Reviewed-By: Raz Luvaton [email protected] Reviewed-By: Luigi Pinca [email protected] Reviewed-By: Moshe Atlow [email protected] Reviewed-By: Debadree Chatterjee [email protected]

Successfully rebased and updated refs/heads/main.

nodejs-github-bot commented Dec 15, 2023

richardlau commented Mar 25, 2024

atlowChemi commented Mar 29, 2024

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]
Reviewed-By: Debadree Chatterjee [email protected]

PR-URL: #51119
Reviewed-By: Raz Luvaton [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Moshe Atlow [email protected]
Reviewed-By: Debadree Chatterjee [email protected]