Skip to content

Commit 19ed33d

Browse files
Trottrvagg
authored andcommitted
test: skip test if in FreeBSD jail
Test test-net-socket-local-address is flaky in FreeBSD jail but robust otherwise. Fixes: #2475 PR-URL: #3995 Reviewed-By: Rod Vagg <[email protected]> Reviewed-By: Johan Bergström <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent baac81d commit 19ed33d

File tree

2 files changed

+13
-8
lines changed

2 files changed

+13
-8
lines changed

‎test/parallel/parallel.status‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,4 +19,3 @@ test-child-process-exit-code : PASS,FLAKY
1919
[$system==solaris] # Also applies to SmartOS
2020

2121
[$system==freebsd]
22-
test-net-socket-local-address : PASS,FLAKY

‎test/parallel/test-net-socket-local-address.js‎

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,30 @@
11
'use strict';
2-
varcommon=require('../common');
3-
varassert=require('assert');
4-
varnet=require('net');
2+
constcommon=require('../common');
3+
constassert=require('assert');
4+
constnet=require('net');
5+
6+
// skip test in FreeBSD jails
7+
if(common.inFreeBSDJail){
8+
console.log('1..0 # Skipped: In a FreeBSD jail');
9+
return;
10+
}
511

612
varconns=0;
713
varclientLocalPorts=[];
814
varserverRemotePorts=[];
915

10-
varserver=net.createServer(function(socket){
16+
constserver=net.createServer(function(socket){
1117
serverRemotePorts.push(socket.remotePort);
1218
conns++;
1319
});
1420

15-
varclient=newnet.Socket();
21+
constclient=newnet.Socket();
1622

17-
server.on('close',function(){
23+
server.on('close',common.mustCall(function(){
1824
assert.deepEqual(clientLocalPorts,serverRemotePorts,
1925
'client and server should agree on the ports used');
2026
assert.equal(2,conns);
21-
});
27+
}));
2228

2329
server.listen(common.PORT,common.localhostIPv4,testConnect);
2430

0 commit comments

Comments
(0)