Skip to content

Commit a2621a2

Browse files
ALJCepedaMyles Borins
authored and
Myles Borins
committed
test: remove FIXME pummel/test-tls-securepair-client
Reverts: 85827bd Using `common.PORT` no longer causes other tests to fail Refs: #4640 PR-URL: #8757 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ilkka Myller <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
1 parent 4571c84 commit a2621a2

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

test/pummel/test-tls-securepair-client.js

+2-8
Original file line numberDiff line numberDiff line change
@@ -39,20 +39,14 @@ function test2() {
3939
}
4040

4141
function test(keyfn, certfn, check, next) {
42-
// FIXME: Avoid the common PORT as this test currently hits a C-level
43-
// assertion error with node_g. The program aborts without HUPing
44-
// the openssl s_server thus causing many tests to fail with
45-
// EADDRINUSE.
46-
var PORT = common.PORT + 5;
47-
4842
keyfn = join(common.fixturesDir, keyfn);
4943
var key = fs.readFileSync(keyfn).toString();
5044

5145
certfn = join(common.fixturesDir, certfn);
5246
var cert = fs.readFileSync(certfn).toString();
5347

5448
var server = spawn(common.opensslCli, ['s_server',
55-
'-accept', PORT,
49+
'-accept', common.PORT,
5650
'-cert', certfn,
5751
'-key', keyfn]);
5852
server.stdout.pipe(process.stdout);
@@ -121,7 +115,7 @@ function test(keyfn, certfn, check, next) {
121115
pair.encrypted.pipe(s);
122116
s.pipe(pair.encrypted);
123117

124-
s.connect(PORT);
118+
s.connect(common.PORT);
125119

126120
s.on('connect', function() {
127121
console.log('client connected');

0 commit comments

Comments
 (0)