1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: refactored http test to use countdown

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/17241
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
This commit is contained in:
Mithun Sasidharan 2017-11-22 15:39:07 +05:30 committed by Anna Henningsen
parent 7dc24062fc
commit cd625e0658
No known key found for this signature in database
GPG Key ID: 9C63F3A6CD2AD8F9
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/parallel/test-http-end-throw-socket-handling.js
View File

@ -21,6 +21,7 @@
'use strict'; 'use strict';
const common = require('../common'); const common = require('../common');
const Countdown = require('../common/countdown');
// Make sure that throwing in 'end' handler doesn't lock // Make sure that throwing in 'end' handler doesn't lock
// up the socket forever. // up the socket forever.
@ -29,10 +30,10 @@ const common = require('../common');
// the same, we should not be so brittle and easily broken. // the same, we should not be so brittle and easily broken.
const http = require('http'); const http = require('http');
const countdown = new Countdown(10, () => server.close());
let n = 0;
const server = http.createServer((req, res) => { const server = http.createServer((req, res) => {
if (++n === 10) server.close(); countdown.dec();
res.end('ok'); res.end('ok');
}); });