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]>
nodejs · Oct 6, 2016 · 647e8e5 · 647e8e5
1 parent 0f944ab
commit 647e8e5
Showing 1 changed file with 2 additions and 8 deletions.
diff --git a/test/pummel/test-tls-securepair-client.js b/test/pummel/test-tls-securepair-client.js
@@ -39,20 +39,14 @@ function test2() {
 }
 
 function test(keyfn, certfn, check, next) {
-  // FIXME: Avoid the common PORT as this test currently hits a C-level
-  // assertion error with node_g. The program aborts without HUPing
-  // the openssl s_server thus causing many tests to fail with
-  // EADDRINUSE.
-  var PORT = common.PORT + 5;
-
   keyfn = join(common.fixturesDir, keyfn);
   var key = fs.readFileSync(keyfn).toString();
 
   certfn = join(common.fixturesDir, certfn);
   var cert = fs.readFileSync(certfn).toString();
 
   var server = spawn(common.opensslCli, ['s_server',
-                                         '-accept', PORT,
+                                         '-accept', common.PORT,
                                          '-cert', certfn,
                                          '-key', keyfn]);
   server.stdout.pipe(process.stdout);
@@ -121,7 +115,7 @@ function test(keyfn, certfn, check, next) {
     pair.encrypted.pipe(s);
     s.pipe(pair.encrypted);
 
-    s.connect(PORT);
+    s.connect(common.PORT);
 
     s.on('connect', function() {
       console.log('client connected');