test: Use countdown in test file
Fixes: https://github.com/nodejs/node/issues/17169 PR-URL: https://github.com/nodejs/node/pull/17646 Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
This commit is contained in:
parent
97cee727d5
commit
0a28f94df6
@ -1,40 +1,27 @@
|
|||||||
'use strict';
|
'use strict';
|
||||||
require('../common');
|
require('../common');
|
||||||
const assert = require('assert');
|
|
||||||
const http = require('http');
|
const http = require('http');
|
||||||
const net = require('net');
|
const net = require('net');
|
||||||
|
const Countdown = require('../common/countdown');
|
||||||
|
|
||||||
const big = Buffer.alloc(16 * 1024, 'A');
|
const big = Buffer.alloc(16 * 1024, 'A');
|
||||||
|
|
||||||
const COUNT = 1e4;
|
const COUNT = 1e4;
|
||||||
|
|
||||||
let received = 0;
|
const countdown = new Countdown(COUNT, () => {
|
||||||
|
server.close();
|
||||||
|
client.end();
|
||||||
|
});
|
||||||
|
|
||||||
let client;
|
let client;
|
||||||
const server = http.createServer(function(req, res) {
|
const server = http.createServer(function(req, res) {
|
||||||
res.end(big, function() {
|
res.end(big, function() {
|
||||||
if (++received === COUNT) {
|
countdown.dec();
|
||||||
server.close();
|
|
||||||
client.end();
|
|
||||||
}
|
|
||||||
});
|
});
|
||||||
}).listen(0, function() {
|
}).listen(0, function() {
|
||||||
const req = 'GET / HTTP/1.1\r\n\r\n'.repeat(COUNT);
|
const req = 'GET / HTTP/1.1\r\n\r\n'.repeat(COUNT);
|
||||||
client = net.connect(this.address().port, function() {
|
client = net.connect(this.address().port, function() {
|
||||||
client.write(req);
|
client.write(req);
|
||||||
});
|
});
|
||||||
|
|
||||||
// Just let the test terminate instead of hanging
|
|
||||||
client.on('close', function() {
|
|
||||||
if (received !== COUNT)
|
|
||||||
server.close();
|
|
||||||
});
|
|
||||||
client.resume();
|
client.resume();
|
||||||
});
|
});
|
||||||
|
|
||||||
process.on('exit', function() {
|
|
||||||
// The server should pause connection on pipeline flood, but it shoul still
|
|
||||||
// resume it and finish processing the requests, when its output queue will
|
|
||||||
// be empty again.
|
|
||||||
assert.strictEqual(received, COUNT);
|
|
||||||
});
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user