test_runner: added restart log in test-runner/runner.js#57375
test_runner: added restart log in test-runner/runner.js#57375sivakumar-77 wants to merge 1 commit intonodejs:mainfrom
Conversation
Fixes nodejs#57206 Added restart log in test-runner/runner.js to log with timestamp and test-file, whenever the test file is restart testing in `--watch` mode.
|
Review requested:
|
|
Hey @sivakumar-77, thanks for opening this PR! 🚀 I think we should avoid adding I would suggest adding a new event, like the existing one, Could you also please add tests to cover this feature? |
|
Hey @pmarchini thanks for the feedback! 🙌 That makes sense—I’ll update the PR to introduce a new event instead of directly modifying the runner restart logic. I’ll also update the internal reporters to handle it appropriately. I'll add tests to cover this feature as well. Thanks |
|
@pmarchini I've taken over this piece of work here #57903 |
|
Hey @sivakumar-77, as this PR seems stale, shall we close it, or are you still working on it? |
Fixes #57206
Added restart log in test-runner/runner.js to log with timestamp and test-file
whenever the test file is restart testing in
--watchmode.