0
0
mirror of https://github.com/nodejs/node.git synced 2024-11-21 21:19:50 +01:00
nodejs/test/parallel/test-http-client-set-timeout.js
Robert Nagy b04e88439a http: don't destroy completed request
Calling destroy() on a completed ClientRequest, i.e.
once 'close' will be emitted should be a noop. Also
before emitting 'close' destroyed === true.

Fixes: https://github.com/nodejs/node/issues/32851

PR-URL: https://github.com/nodejs/node/pull/33120
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
2020-05-11 18:02:32 +02:00

50 lines
1.1 KiB
JavaScript

'use strict';
// Test that the `'timeout'` event is emitted exactly once if the `timeout`
// option and `request.setTimeout()` are used together.
const { expectsError, mustCall } = require('../common');
const { strictEqual } = require('assert');
const { createServer, get } = require('http');
const server = createServer(() => {
// Never respond.
});
server.listen(0, mustCall(() => {
const req = get({
port: server.address().port,
timeout: 2000,
});
req.setTimeout(1000);
req.on('socket', mustCall((socket) => {
strictEqual(socket.timeout, 2000);
socket.on('connect', mustCall(() => {
strictEqual(socket.timeout, 1000);
// Reschedule the timer to not wait 1 sec and make the test finish faster.
socket.setTimeout(10);
strictEqual(socket.timeout, 10);
}));
}));
req.on('error', expectsError({
name: 'Error',
code: 'ECONNRESET',
message: 'socket hang up'
}));
req.on('close', mustCall(() => {
strictEqual(req.destroyed, true);
server.close();
}));
req.on('timeout', mustCall(() => {
strictEqual(req.socket.listenerCount('timeout'), 1);
req.destroy();
}));
}));