test_runner: report covered lines, functions and branches to reporters by philnash · Pull Request #49320 · nodejs/node

@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

Aug 25, 2023

ljharb

@philnash

This is a breaking change for the format of test:coverage events. But
the test coverage is still experimental, so I don't believe it requires
a semver-major bump.

Fixes nodejs#49303

@philnash

@philnash

rluvaton

rluvaton

rluvaton

benjamingr

@philnash

Removed additional loop from calculating max counts for functions.
Simplified reporting of count for each line.
Returned arrow function to implicit return.

@philnash

atlowChemi

@philnash

@atlowChemi atlowChemi added experimental

Issues and PRs related to experimental features.

commit-queue

Add this label to land a pull request using GitHub Actions.

commit-queue-squash

Add this label to instruct the Commit Queue to squash all the PR commits into the first one.

labels

Aug 30, 2023

This was referenced

Aug 31, 2023

UlisesGascon pushed a commit that referenced this pull request

Sep 10, 2023
This is a breaking change for the format of test:coverage events. But
the test coverage is still experimental, so I don't believe it requires
a semver-major bump.

Fixes #49303

PR-URL: #49320
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>

alexfernandez pushed a commit to alexfernandez/node that referenced this pull request

Nov 1, 2023
This is a breaking change for the format of test:coverage events. But
the test coverage is still experimental, so I don't believe it requires
a semver-major bump.

Fixes nodejs#49303

PR-URL: nodejs#49320
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>