Skip to content

Commit 46f8a9e

Browse files
BamiehMylesBorins
authored andcommitted
test: update test-http-content-length to use countdown
PR-URL: #17201 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Jon Moss <[email protected]>
1 parent d1af106 commit 46f8a9e

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

test/parallel/test-http-content-length.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
require('../common');
33
const assert = require('assert');
44
const http = require('http');
5+
const Countdown = require('../common/countdown');
56

67
const expectedHeadersMultipleWrites = {
78
'connection': 'close',
@@ -18,8 +19,8 @@ const expectedHeadersEndNoData = {
1819
'content-length': '0',
1920
};
2021

21-
let receivedRequests = 0;
22-
const totalRequests = 3;
22+
23+
const countdown = new Countdown(3, () => server.close());
2324

2425
const server = http.createServer(function(req, res) {
2526
res.removeHeader('Date');
@@ -42,8 +43,7 @@ const server = http.createServer(function(req, res) {
4243
throw new Error('Unreachable');
4344
}
4445

45-
receivedRequests++;
46-
if (totalRequests === receivedRequests) server.close();
46+
countdown.dec();
4747
});
4848

4949
server.listen(0, function() {

0 commit comments

Comments
 (0)