Skip to content

Commit

Permalink
test: skip test if in FreeBSD jail
Browse files Browse the repository at this point in the history
Test test-net-socket-local-address is flaky in FreeBSD jail but robust
otherwise.

Fixes: nodejs#2475
PR-URL: nodejs#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]>
  • Loading branch information
Trott authored and Michael Scovetta committed Apr 2, 2016
1 parent d7b199d commit 56efb3d
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 8 deletions.
1 change: 0 additions & 1 deletion test/parallel/parallel.status
Original file line number Diff line number Diff line change
Expand Up @@ -19,4 +19,3 @@ test-child-process-exit-code : PASS,FLAKY
[$system==solaris] # Also applies to SmartOS

[$system==freebsd]
test-net-socket-local-address : PASS,FLAKY
20 changes: 13 additions & 7 deletions test/parallel/test-net-socket-local-address.js
Original file line number Diff line number Diff line change
@@ -1,24 +1,30 @@
'use strict';
var common = require('../common');
var assert = require('assert');
var net = require('net');
const common = require('../common');
const assert = require('assert');
const net = require('net');

// skip test in FreeBSD jails
if (common.inFreeBSDJail) {
console.log('1..0 # Skipped: In a FreeBSD jail');
return;
}

var conns = 0;
var clientLocalPorts = [];
var serverRemotePorts = [];

var server = net.createServer(function(socket) {
const server = net.createServer(function(socket) {
serverRemotePorts.push(socket.remotePort);
conns++;
});

var client = new net.Socket();
const client = new net.Socket();

server.on('close', function() {
server.on('close', common.mustCall(function() {
assert.deepEqual(clientLocalPorts, serverRemotePorts,
'client and server should agree on the ports used');
assert.equal(2, conns);
});
}));

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

Expand Down

0 comments on commit 56efb3d

Please sign in to comment.