test: fix flaky test-tls-wrap-timeout

Competing timers were causing a race condition and thus the test was flaky. Instead, we check an object property on process exit. Fixes: #7650 Backport-PR-URL: #12567 PR-URL: #7857 Reviewed-By: Santiago Gimeno <[email protected]> Reviewed-By: Fedor Indutny <[email protected]> Reviewed-By: jasnell - James M Snell <[email protected]>

Author: Trott

test/parallel/test-tls-wrap-timeout.js

@@ -1,42 +1,54 @@
 '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');
 
-varnet=require('net');
-varfs=require('fs');
+constnet=require('net');
+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(c){
-setTimeout(function(){
-c.write('hello');
-setTimeout(function(){
-c.destroy();
-server.close();
-},150);
-},150);
-});
+constserver=tls.createServer(options,common.mustCall((c)=>{
+setImmediate(()=>{
+c.write('hello',()=>{
+setImmediate(()=>{
+c.destroy();
+server.close();
+});
+});
+});
+}));
+
+varsocket;
+varlastIdleStart;
 
-server.listen(0,function(){
-varsocket=net.connect(this.address().port,function(){
-vars=socket.setTimeout(common.platformTimeout(240),function(){
+server.listen(0,()=>{
+socket=net.connect(server.address().port,function(){
+consts=socket.setTimeout(Number.MAX_VALUE,function(){
 thrownewError('timeout');
 });
 assert.ok(sinstanceofnet.Socket);
 
-vartsocket=tls.connect({
+assert.notStrictEqual(socket._idleTimeout,-1);
+lastIdleStart=socket._idleStart;
+
+consttsocket=tls.connect({
 socket: socket,
 rejectUnauthorized: false
 });
 tsocket.resume();
 });
 });
+
+process.on('exit',()=>{
+assert.strictEqual(socket._idleTimeout,-1);
+assert(lastIdleStart<socket._idleStart);
+});