test: update test-http-request-dont-override-options to use common.mustCall

PR-URL: #17438 Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>

Author: mithunsasidharan

test/parallel/test-http-request-dont-override-options.js

@@ -4,14 +4,13 @@ const common = require('../common');
 constassert=require('assert');
 consthttp=require('http');
 
-letrequests=0;
 
-http.createServer(function(req,res){
+constserver=http.createServer(common.mustCall(function(req,res){
 res.writeHead(200);
 res.end('ok');
+}));
 
-requests++;
-}).listen(0,function(){
+server.listen(0,function(){
 constagent=newhttp.Agent();
 agent.defaultPort=this.address().port;
 
@@ -30,16 +29,12 @@ http.createServer(function(req, res){
 
 http.request(options,function(res){
 res.resume();
-}).end();
-
-process.on('exit',function(){
-assert.strictEqual(requests,1);
-
+server.close();
 assert.strictEqual(options.host,undefined);
 assert.strictEqual(options.hostname,common.localhostIPv4);
 assert.strictEqual(options.port,undefined);
 assert.strictEqual(options.defaultPort,undefined);
 assert.strictEqual(options.path,undefined);
 assert.strictEqual(options.method,undefined);
-});
-}).unref();
+}).end();
+});