-
Notifications
You must be signed in to change notification settings - Fork 30k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: kill subprocess only after last ACK #15186
test: kill subprocess only after last ACK #15186
Conversation
Failing Stress: https://ci.nodejs.org/job/node-stress-single-test/1402/nodes=win-vs2015/ 🔴 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM if CI is green
CI https://ci.nodejs.org/job/node-test-commit/12259/ literally Green |
* Add multiple comments. * Switch to a persistent fixture for subprocess. * Assert that `send` queue is drained. PR-URL: nodejs#15186 Fixes: nodejs#15176 Reviewed-By: James M Snell <jasnell@gmail.com>
1e005d5
to
94be2b1
Compare
* Add multiple comments. * Switch to a persistent fixture for subprocess. * Assert that `send` queue is drained. PR-URL: nodejs#15186 Fixes: nodejs#15176 Reviewed-By: James M Snell <jasnell@gmail.com>
This does not land cleanly in LTS. Please feel free to manually backport by following the guide. Please also feel free to replace do-not-land if it is being backported |
send
queue is drained.Fixes: #15176
Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) passesAffected core subsystem(s)
test, child_process