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';
-var common = require('../common');
-var assert = require('assert');
+const common = require('../common');
+const assert = require('assert');
 
 if (!common.hasCrypto) {
   common.skip('missing crypto');
   return;
 }
-var tls = require('tls');
+const tls = require('tls');
 
-var fs = require('fs');
+const fs = require('fs');
 
-var options = {
+const options = {
   key: fs.readFileSync(common.fixturesDir + '/keys/agent1-key.pem'),
   cert: fs.readFileSync(common.fixturesDir + '/keys/agent1-cert.pem')
 };
 
-var server = tls.createServer(options, function(cleartext) {
-  var s = cleartext.setTimeout(50, function() {
+const server = tls.createServer(options, common.mustCall(function(cleartext) {
+  const s = cleartext.setTimeout(50, function() {
     cleartext.destroy();
     server.close();
   });
   assert.ok(s instanceof tls.TLSSocket);
-});
+}));
 
-server.listen(0, function() {
+server.listen(0, common.mustCall(function() {
   tls.connect({
     host: '127.0.0.1',
     port: this.address().port,
     rejectUnauthorized: false
   });
-});
+}));