test: updated tls-getcipher test

Changed var to ES6 syntax const/let. Added common.mustCall function wrapper to all functions called exactly once. Changed assert.equal to assert.strictEqual. PR-URL: #9923 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: Ethan-Arrowood

test/parallel/test-tls-getcipher.js

@@ -1,37 +1,37 @@
 'use strict';
-varcommon=require('../common');
-varassert=require('assert');
+constcommon=require('../common');
 
 if(!common.hasCrypto){
 common.skip('missing crypto');
 return;
 }
-vartls=require('tls');
+constassert=require('assert');
+consttls=require('tls');
 
-varfs=require('fs');
-varcipher_list=['AES128-SHA256','AES256-SHA256'];
-varcipher_version_pattern=/TLS|SSL/;
-varoptions={
+constfs=require('fs');
+constcipher_list=['AES128-SHA256','AES256-SHA256'];
+constcipher_version_pattern=/TLS|SSL/;
+constoptions={
 key: fs.readFileSync(common.fixturesDir+'/keys/agent2-key.pem'),
 cert: fs.readFileSync(common.fixturesDir+'/keys/agent2-cert.pem'),
 ciphers: cipher_list.join(':'),
 honorCipherOrder: true
 };
 
-varserver=tls.createServer(options,
-common.mustCall(function(cleartextStream){}));
+constserver=tls.createServer(options,
+common.mustCall(function(cleartextStream){}));
 
-server.listen(0,'127.0.0.1',function(){
-varclient=tls.connect({
+server.listen(0,'127.0.0.1',common.mustCall(function(){
+constclient=tls.connect({
 host: '127.0.0.1',
 port: this.address().port,
 ciphers: cipher_list.join(':'),
 rejectUnauthorized: false
-},function(){
-varcipher=client.getCipher();
-assert.equal(cipher.name,cipher_list[0]);
+},common.mustCall(function(){
+constcipher=client.getCipher();
+assert.strictEqual(cipher.name,cipher_list[0]);
 assert(cipher_version_pattern.test(cipher.version));
 client.end();
 server.close();
-});
-});
+}));
+}));