[v20.x backport] test_runner: make end of work check stricter
This commit updates the logic that checks for the end of the test run. Prior to this change, it was possible for root.run() to be called multiple times because of the way pending subtests were tracked. The extra calls to root.run() were harmless, but could trigger an EventEmitter leak warning due to 'abort' listeners being created.
PR-URL: https://github.com/nodejs/node/pull/52326 Reviewed-By: Benjamin Gruenbaum [email protected] Reviewed-By: Moshe Atlow [email protected] Reviewed-By: Chemi Atlow [email protected]
Review requested:
- [ ] @nodejs/test_runner
CI: https://ci.nodejs.org/job/node-test-pull-request/59391/
CI: https://ci.nodejs.org/job/node-test-pull-request/59407/
CI: https://ci.nodejs.org/job/node-test-pull-request/59414/
CI: https://ci.nodejs.org/job/node-test-pull-request/59462/
CI: https://ci.nodejs.org/job/node-test-pull-request/59731/
CI: https://ci.nodejs.org/job/node-test-pull-request/59831/
CI: https://ci.nodejs.org/job/node-test-pull-request/59974/
CI: https://ci.nodejs.org/job/node-test-pull-request/60348/
CI: https://ci.nodejs.org/job/node-test-pull-request/60439/
CI: https://ci.nodejs.org/job/node-test-pull-request/61218/
This issue/PR was marked as stalled, it will be automatically closed in 30 days. If it should remain open, please leave a comment explaining why it should remain open.
Closing this because it has stalled. Feel free to reopen if this issue/PR is still relevant, or to ping the collaborator who labelled it stalled if you have any questions.