Skip to content

Commit 4e38655

Browse files
TrottMyles Borins
authored and
Myles Borins
committed
test: refactor test-tls-reuse-host-from-socket
Replace booleans with `common.mustCall()`, migrate from `var` to `const`, and apply minor formatting changes. PR-URL: #6756 Reviewed-By: Fedor Indutny <[email protected]> Reviewed-By: Santiago Gimeno <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 30ab6a8 commit 4e38655

File tree

1 file changed

+8
-18
lines changed

1 file changed

+8
-18
lines changed
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,17 @@
11
'use strict';
2-
var common = require('../common');
3-
var assert = require('assert');
2+
const common = require('../common');
43

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

11-
var net = require('net');
12-
var connected = false;
13-
var secure = false;
10+
const net = require('net');
1411

15-
process.on('exit', function() {
16-
assert(connected);
17-
assert(secure);
18-
console.log('ok');
19-
});
20-
21-
var socket = net.connect(443, 'www.google.com', function() {
22-
connected = true;
23-
var secureSocket = tls.connect({ socket: socket }, function() {
24-
secure = true;
12+
const socket = net.connect(443, 'www.example.org', common.mustCall(() => {
13+
const secureSocket = tls.connect({socket: socket}, common.mustCall(() => {
2514
secureSocket.destroy();
26-
});
27-
});
15+
console.log('ok');
16+
}));
17+
}));

0 commit comments

Comments
 (0)