test_runner: emit test:watch:restarted event on watched file changes

[JacopoPatroclo]

The goal of this PR is to add a test:watch:restarted event to signal to the reporter that some tests have restarted while in watch mode.

This will enable a trivial fix for nodejs/node#57206, which I plan to address later.

[nodejs-github-bot]

Review requested:

• @nodejs/test_runner

[pmarchini]

Hey @JacopoPatroclo, thanks for the PR. Could you please add tests to cover the new behaviour? Also, could you please update the documentation?

[pmarchini]

LGTM

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.28%. Comparing base (6cdcaa9) to head (9ef7789).
Report is 204 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #57903      +/-   ##
==========================================
+ Coverage   90.26%   90.28%   +0.01%     
==========================================
  Files         630      630              
  Lines      185933   186331     +398     
  Branches    36447    36508      +61     
==========================================
+ Hits       167840   168233     +393     
- Misses      10971    10976       +5     
  Partials     7122     7122              

Files with missing lines	Coverage Δ
lib/internal/test_runner/runner.js	92.49% <100.00%> (+3.61%)	⬆️

... and 61 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[jakecastelli]

one small nit - the commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. label would squash all your commits and use the first one upon landing - the first commit message in this case is test_runner: added restart log while in watch mode which seems not very accurate, I think since the CI is already green, we can manually land this, wdyt @pmarchini

[pmarchini]

@jakecastelli thanks for pointing it out!!! there's no rush, I'd say we could just ask @JacopoPatroclo to manually rewrite the history!

[jakecastelli]

Ahh no worries at all! Happy to do another approval / CI once the commit message is fixed 👍

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[nodejs-github-bot]

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