diff options
author | Rich Trott <rtrott@gmail.com> | 2017-07-08 09:05:34 -0700 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2017-07-11 10:09:06 -0700 |
commit | b20a0b662bf57e6873ce808543f266c63db75947 (patch) | |
tree | 0b49c07416fca0a8fbecb544e542babb5be4e48c /test/parallel/test-http-set-timeout-server.js | |
parent | 22889347dfe5315fe03abb9a4263c30cdd74b436 (diff) | |
download | android-node-v8-b20a0b662bf57e6873ce808543f266c63db75947.tar.gz android-node-v8-b20a0b662bf57e6873ce808543f266c63db75947.tar.bz2 android-node-v8-b20a0b662bf57e6873ce808543f266c63db75947.zip |
test: fix flaky test-https-set-timeout-server
Because of a race condition, connection listener may not be invoked if
test is run under load. Remove `common.mustCall()` wrapper from the
listener. Move the test to `parallel` because it now works under load.
Make similar change to http test to keep them in synch even though it is
much harder to trigger the race in http.
PR-URL: https://github.com/nodejs/node/pull/14134
Fixes: https://github.com/nodejs/node/issues/14133
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Diffstat (limited to 'test/parallel/test-http-set-timeout-server.js')
-rw-r--r-- | test/parallel/test-http-set-timeout-server.js | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/test/parallel/test-http-set-timeout-server.js b/test/parallel/test-http-set-timeout-server.js index 83fc7dc046..4eb41d01f9 100644 --- a/test/parallel/test-http-set-timeout-server.js +++ b/test/parallel/test-http-set-timeout-server.js @@ -42,9 +42,10 @@ function run() { } test(function serverTimeout(cb) { - const server = http.createServer(common.mustCall((req, res) => { - // just do nothing, we should get a timeout event. - })); + const server = http.createServer((req, res) => { + // Do nothing. We should get a timeout event. + // Might not be invoked. Do not wrap in common.mustCall(). + }); server.listen(common.mustCall(() => { const s = server.setTimeout(50, common.mustCall((socket) => { socket.destroy(); |