Browse files

test: Don't reuse common.PORT in test-child-process-fork-net

This fixes #3447
  • Loading branch information...
1 parent 41421ff commit d614d161c7f1ac340251271e911cd4752c158d38 @isaacs isaacs committed Jun 16, 2012
Showing with 9 additions and 2 deletions.
  1. +9 −2 test/simple/test-child-process-fork-net.js
View
11 test/simple/test-child-process-fork-net.js
@@ -157,8 +157,15 @@ if (process.argv[2] === 'child') {
console.log('PARENT: server closed');
callback();
});
- server.listen(common.PORT, function() {
- var connect = net.connect(common.PORT);
+ // don't listen on the same port, because SmartOS sometimes says
+ // that the server's fd is closed, but it still cannot listen
+ // on the same port again.
+ //
+ // An isolated test for this would be lovely, but for now, this
+ // will have to do.
+ server.listen(common.PORT + 1, function() {
+ console.error('testSocket, listening');
+ var connect = net.connect(common.PORT + 1);
var store = '';
connect.on('data', function(chunk) {
store += chunk;

0 comments on commit d614d16

Please sign in to comment.