Skip to content

Commit 21a6091

Browse files
bchirgwintargos
authored andcommitted
test: refactor test-tls-interleave
var -> let / const added common.mustCall() to callback assert.equal() -> assert.strictEqual() PR-URL: #10017 Reviewed-By: Colin Ihrig <[email protected]>
1 parent e53506a commit 21a6091

File tree

1 file changed

+18
-16
lines changed

1 file changed

+18
-16
lines changed
Lines changed: 18 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,37 +1,38 @@
11
'use strict';
2-
varcommon=require('../common');
3-
varassert=require('assert');
2+
constcommon=require('../common');
43

54
if(!common.hasCrypto){
65
common.skip('missing crypto');
76
return;
87
}
9-
vartls=require('tls');
8+
constassert=require('assert');
109

11-
varfs=require('fs');
10+
consttls=require('tls');
1211

13-
vardir=common.fixturesDir;
14-
varoptions={key: fs.readFileSync(dir+'/test_key.pem'),
15-
cert: fs.readFileSync(dir+'/test_cert.pem'),
16-
ca: [fs.readFileSync(dir+'/test_ca.pem')]};
12+
constfs=require('fs');
1713

18-
varwrites=[
14+
constdir=common.fixturesDir;
15+
constoptions={key: fs.readFileSync(dir+'/test_key.pem'),
16+
cert: fs.readFileSync(dir+'/test_cert.pem'),
17+
ca: [fs.readFileSync(dir+'/test_ca.pem')]};
18+
19+
constwrites=[
1920
'some server data',
2021
'and a separate packet',
2122
'and one more',
2223
];
23-
varreceivedWrites=0;
24+
letreceivedWrites=0;
2425

25-
varserver=tls.createServer(options,function(c){
26+
constserver=tls.createServer(options,function(c){
2627
writes.forEach(function(str){
2728
c.write(str);
2829
});
29-
}).listen(0,function(){
30+
}).listen(0,common.mustCall(function(){
3031
constconnectOpts={rejectUnauthorized: false};
31-
varc=tls.connect(this.address().port,connectOpts,function(){
32+
constc=tls.connect(this.address().port,connectOpts,function(){
3233
c.write('some client data');
3334
c.on('readable',function(){
34-
vardata=c.read();
35+
letdata=c.read();
3536
if(data===null)
3637
return;
3738

@@ -47,8 +48,9 @@ var server = tls.createServer(options, function(c){
4748
}
4849
});
4950
});
50-
});
51+
}));
52+
5153

5254
process.on('exit',function(){
53-
assert.equal(receivedWrites,writes.length);
55+
assert.strictEqual(receivedWrites,writes.length);
5456
});

0 commit comments

Comments
(0)