test_runner: report coverage thresholds in `test:coverage` by avivkeller · Pull Request #54813 · nodejs/node

avivkeller · 2024-09-06T16:17:34Z

Fixes #54812

Adds a thresholds object to test:coverage's summary property.

nodejs-github-bot · 2024-09-06T16:17:39Z

Review requested:

@nodejs/test_runner

codecov · 2024-09-06T17:44:54Z

Codecov Report

Attention: Patch coverage is 81.81818% with 2 lines in your changes missing coverage. Please review.

Project coverage is 87.90%. Comparing base (ce19715) to head (45db682).
Report is 224 commits behind head on main.

Files with missing lines	Patch %	Lines
lib/internal/test_runner/test.js	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #54813      +/-   ##
==========================================
+ Coverage   87.60%   87.90%   +0.29%     
==========================================
  Files         650      651       +1     
  Lines      182943   183351     +408     
  Branches    35399    35709     +310     
==========================================
+ Hits       160270   161168     +898     
+ Misses      15936    15465     -471     
+ Partials     6737     6718      -19

Files with missing lines	Coverage Δ
lib/internal/test_runner/coverage.js	`20.69% <100.00%> (+0.97%)`	⬆️
lib/internal/test_runner/test.js	`96.93% <0.00%> (ø)`

... and 87 files with indirect coverage changes

mcollina

lgtm

avivkeller · 2024-09-07T22:12:01Z

This PR also fixes an issue that (I believe) is causing #53937 to have failing tests.

cjihrig

Requesting changes since this has the approvals to land.

cjihrig

LGTM with minor comments.

Co-authored-by: Colin Ihrig <cjihrig@gmail.com>

github-actions · 2024-09-09T23:15:42Z

Failed to start CI

   ⚠  Something was pushed to the Pull Request branch since the last approving review.
   ✘  Refusing to run CI on potentially unsafe PR

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

nodejs-github-bot · 2024-09-10T15:46:32Z

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

avivkeller · 2024-09-12T17:08:41Z

The CI appears to have been aborted, can a new one be started in its place?

nodejs-github-bot · 2024-09-12T17:59:15Z

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

nodejs-github-bot · 2024-09-12T19:52:32Z

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

nodejs-github-bot · 2024-09-13T02:57:13Z

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

avivkeller · 2024-09-13T12:14:40Z

CI is 🟢

nodejs-github-bot · 2024-09-13T13:31:39Z

Landed in 99bbf80

avivkeller · 2024-09-16T14:29:48Z

Is this semver-minor and/or notable change? I know it's being released in v22.9.0, but it does add new information, so I wanted to be sure.

PR-URL: #54813 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

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 Sep 6, 2024

avivkeller added the coverage Issues and PRs related to Node.js code coverage support. label Sep 6, 2024

avivkeller mentioned this pull request Sep 6, 2024

Running tests with --test-coverage-branches (same for lines|functions) doesn't emit test:fail when threshold isn't met #54812

Closed

mcollina approved these changes Sep 7, 2024

View reviewed changes

rozzilla reviewed Sep 7, 2024

View reviewed changes

Comment thread doc/api/test.md Outdated

avivkeller mentioned this pull request Sep 7, 2024

test_runner: add coverage support to run function #53937

Merged

MoLow approved these changes Sep 8, 2024

View reviewed changes

atlowChemi approved these changes Sep 8, 2024

View reviewed changes

jasnell approved these changes Sep 8, 2024

View reviewed changes

avivkeller added the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Sep 8, 2024

cjihrig requested changes Sep 8, 2024

View reviewed changes

Comment thread doc/api/test.md Outdated

Comment thread lib/internal/main/test_runner.js Outdated

Comment thread lib/internal/test_runner/harness.js Outdated

avivkeller requested a review from cjihrig September 8, 2024 20:52

test_runner: report coverage thresholds in test:coverage

be77741

jasnell approved these changes Sep 8, 2024

View reviewed changes

cjihrig approved these changes Sep 8, 2024

View reviewed changes

Comment thread doc/api/test.md Outdated

Comment thread lib/internal/test_runner/test.js Outdated

Update doc/api/test.md

f86abda

Co-authored-by: Colin Ihrig <cjihrig@gmail.com>

avivkeller added the commit-queue-squash PRs the Commit Queue should land as one squashed commit. label Sep 8, 2024

move to coverage.js

45db682

github-actions Bot added request-ci-failed Starting CI with the request-ci label failed and requires manual intervention. and removed request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. labels Sep 9, 2024

avivkeller removed the request-ci-failed Starting CI with the request-ci label failed and requires manual intervention. label Sep 10, 2024

cjihrig approved these changes Sep 13, 2024

View reviewed changes

cjihrig added author ready PRs with CI started, the required approvals, and no outstanding review comments. commit-queue PRs queued for automated landing through the Commit Queue. and removed needs-ci PRs that need a full CI run. labels Sep 13, 2024

nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Sep 13, 2024

nodejs-github-bot merged commit 99bbf80 into nodejs:main Sep 13, 2024

RafaelGSS mentioned this pull request Sep 16, 2024

v22.9.0 proposal #54966

Merged

cjihrig added the semver-minor PRs that contain new features and should be released in the next minor version. label Sep 17, 2024

targos added the dont-land-on-v20.x label Sep 22, 2024

Uh oh!

Uh oh!

Conversation

avivkeller commented Sep 6, 2024

Uh oh!

nodejs-github-bot commented Sep 6, 2024

Uh oh!

codecov Bot commented Sep 6, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

mcollina left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

avivkeller commented Sep 7, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cjihrig left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cjihrig left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Sep 9, 2024

Uh oh!

nodejs-github-bot commented Sep 10, 2024

Uh oh!

avivkeller commented Sep 12, 2024

Uh oh!

nodejs-github-bot commented Sep 12, 2024

Uh oh!

nodejs-github-bot commented Sep 12, 2024

Uh oh!

nodejs-github-bot commented Sep 13, 2024

Uh oh!

avivkeller commented Sep 13, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nodejs-github-bot commented Sep 13, 2024

Uh oh!

avivkeller commented Sep 16, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

codecov Bot commented Sep 6, 2024 •

edited

Loading

avivkeller commented Sep 7, 2024 •

edited

Loading

cjihrig left a comment •

edited

Loading

avivkeller commented Sep 13, 2024 •

edited

Loading