test: use common.mustCall in test-http-malformed-request

mithunsasidharan · gibfahn · commit 876966f9c377 · 2017-12-20T00:28:06.000Z
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>
diff --git a/test/parallel/test-http-malformed-request.js b/test/parallel/test-http-malformed-request.js
@@ -20,27 +20,22 @@
 // USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 'use strict';
-require('../common');
-const assert = require('assert');
+const common = require('../common');
 const net = require('net');
 const http = require('http');
 const url = require('url');
 
 // Make sure no exceptions are thrown when receiving malformed HTTP
 // requests.
-
-let nrequests_completed = 0;
-const nrequests_expected = 1;
-
-const server = http.createServer(function(req, res) {
+const server = http.createServer(common.mustCall((req, res) => {
   console.log(`req: ${JSON.stringify(url.parse(req.url))}`);
 
   res.writeHead(200, { 'Content-Type': 'text/plain' });
   res.write('Hello World');
   res.end();
 
-  if (++nrequests_completed === nrequests_expected) server.close();
-});
+  server.close();
+}));
 server.listen(0);
 
 server.on('listening', function() {
@@ -50,7 +45,3 @@ server.on('listening', function() {
     c.end();
   });
 });
-
-process.on('exit', function() {
-  assert.strictEqual(nrequests_expected, nrequests_completed);
-});
