diff options
author | Brian White <mscdex@mscdex.net> | 2016-05-29 03:06:56 -0400 |
---|---|---|
committer | Evan Lucas <evanlucas@me.com> | 2016-06-15 18:12:31 -0500 |
commit | eded11705b30b8e07df3f17af2661ebf0fc8ec6b (patch) | |
tree | f435a43645c4095a92add479a6b31e7621836e26 /test/parallel/test-http-response-splitting.js | |
parent | 27ed7fc56c24cac8bc2622a39e8902a888d984e5 (diff) | |
download | node-new-eded11705b30b8e07df3f17af2661ebf0fc8ec6b.tar.gz |
test: use random ports where possible
This helps to prevent issues where a failed test can keep a bound
socket open long enough to cause other tests to fail with EADDRINUSE
because the same port number is used.
PR-URL: https://github.com/nodejs/node/pull/7045
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Diffstat (limited to 'test/parallel/test-http-response-splitting.js')
-rw-r--r-- | test/parallel/test-http-response-splitting.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/parallel/test-http-response-splitting.js b/test/parallel/test-http-response-splitting.js index 3675f8182d..07ced3fd14 100644 --- a/test/parallel/test-http-response-splitting.js +++ b/test/parallel/test-http-response-splitting.js @@ -44,9 +44,9 @@ const server = http.createServer((req, res) => { server.close(); res.end('ok'); }); -server.listen(common.PORT, () => { +server.listen(0, () => { const end = 'HTTP/1.1\r\n\r\n'; - const client = net.connect({port: common.PORT}, () => { + const client = net.connect({port: server.address().port}, () => { client.write(`GET ${str} ${end}`); client.write(`GET / ${end}`); client.write(`GET / ${end}`); |