Skip to content

Commit f9adf51

Browse files
mithunsasidharangibfahn
authored andcommitted
test: use common.mustCall in test-http-malformed-request
PR-URL: #17439 Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: Jon Moss <me@jonathanmoss.me> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 8fc1969 commit f9adf51

File tree

1 file changed

+4
-13
lines changed

1 file changed

+4
-13
lines changed
Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,20 @@
11
'use strict';
2-
require('../common');
3-
const assert = require('assert');
2+
const common = require('../common');
43
const net = require('net');
54
const http = require('http');
65
const url = require('url');
76

87
// Make sure no exceptions are thrown when receiving malformed HTTP
98
// requests.
10-
11-
let nrequests_completed = 0;
12-
const nrequests_expected = 1;
13-
14-
const server = http.createServer(function(req, res) {
9+
const server = http.createServer(common.mustCall((req, res) => {
1510
console.log(`req: ${JSON.stringify(url.parse(req.url))}`);
1611

1712
res.writeHead(200, {'Content-Type': 'text/plain'});
1813
res.write('Hello World');
1914
res.end();
2015

21-
if (++nrequests_completed === nrequests_expected) server.close();
22-
});
16+
server.close();
17+
}));
2318
server.listen(0);
2419

2520
server.on('listening', function() {
@@ -29,7 +24,3 @@ server.on('listening', function() {
2924
c.end();
3025
});
3126
});
32-
33-
process.on('exit', function() {
34-
assert.strictEqual(nrequests_expected, nrequests_completed);
35-
});

0 commit comments

Comments
 (0)