test: refactor test-tls-destroy-whilst-write

cByst · Fishrock123 · commit c456ca3601c1 · 2016-12-05T11:10:16.000-05:00
Update var to let/const and replace arbitrary timeout. PR-URL: #10064 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
diff --git a/test/parallel/test-tls-destroy-whilst-write.js b/test/parallel/test-tls-destroy-whilst-write.js
@@ -1,26 +1,26 @@
 'use strict';
-var common = require('../common');
+const common = require('../common');
 
 if (!common.hasCrypto){
 common.skip('missing crypto');
 return;
 }
-var tls = require('tls');
-var stream = require('stream');
+const tls = require('tls');
+const stream = require('stream');
 
-var delay = new stream.Duplex({
+const delay = new stream.Duplex({
 read: function read(){
 },
 write: function write(data, enc, cb){
 console.log('pending');
- setTimeout(function(){
+ setImmediate(function(){
 console.log('done');
 cb();
- }, 200);
+ });
 }
 });
 
-var secure = tls.connect({
+const secure = tls.connect({
 socket: delay
 });
 setImmediate(function(){

-Original file line number
+Diff line change
@@ @@ -1,26 +1,26 @@ @@
 'use strict';
 -varcommon=require('../common');
 +constcommon=require('../common');
 if(!common.hasCrypto){
 common.skip('missing crypto');
 return;
+}
 -vartls=require('tls');
 -varstream=require('stream');
 +consttls=require('tls');
 +conststream=require('stream');
 -vardelay=newstream.Duplex({
 +constdelay=newstream.Duplex({
 read: functionread(){
 },
 write: functionwrite(data,enc,cb){
 console.log('pending');
 -setTimeout(function(){
 +setImmediate(function(){
 console.log('done');
 cb();
 -},200);
 +});
+}
 });
 -varsecure=tls.connect({
 +constsecure=tls.connect({
 socket: delay
 });
 setImmediate(function(){