test: refactor test-tls-timeout-server-2

* Use `common.mustCall` for all callbacks * Use `const` instead of `var` PR-URL: #9876 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: drifkin

test/parallel/test-tls-timeout-server-2.js

@@ -1,32 +1,32 @@
 'use strict';
-varcommon=require('../common');
-varassert=require('assert');
+constcommon=require('../common');
+constassert=require('assert');
 
 if(!common.hasCrypto){
 common.skip('missing crypto');
 return;
 }
-vartls=require('tls');
+consttls=require('tls');
 
-varfs=require('fs');
+constfs=require('fs');
 
-varoptions={
+constoptions={
 key: fs.readFileSync(common.fixturesDir+'/keys/agent1-key.pem'),
 cert: fs.readFileSync(common.fixturesDir+'/keys/agent1-cert.pem')
 };
 
-varserver=tls.createServer(options,function(cleartext){
-vars=cleartext.setTimeout(50,function(){
+constserver=tls.createServer(options,common.mustCall(function(cleartext){
+consts=cleartext.setTimeout(50,function(){
 cleartext.destroy();
 server.close();
 });
 assert.ok(sinstanceoftls.TLSSocket);
-});
+}));
 
-server.listen(0,function(){
+server.listen(0,common.mustCall(function(){
 tls.connect({
 host: '127.0.0.1',
 port: this.address().port,
 rejectUnauthorized: false
 });
-});
+}));