mirror of
https://github.com/nodejs/node.git
synced 2024-11-21 21:19:50 +01:00
b04e88439a
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>
50 lines
1.1 KiB
JavaScript
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();
|
|
}));
|
|
}));
|