Browse Source

test: fix test/simple/test-net-server-max-connections.js is racey

Fixes #1333.
v0.7.4-release
koichik 13 years ago
parent
commit
827180097c
  1. 12
      test/simple/test-net-server-max-connections.js

12
test/simple/test-net-server-max-connections.js

@ -42,9 +42,7 @@ var server = net.createServer(function(connection) {
}); });
server.listen(common.PORT, function() { server.listen(common.PORT, function() {
for (var i = 0; i < N; i++) { makeConnection(0);
makeConnection(i);
}
}); });
server.maxConnections = N / 2; server.maxConnections = N / 2;
@ -53,10 +51,15 @@ console.error('server.maxConnections = %d', server.maxConnections);
function makeConnection(index) { function makeConnection(index) {
setTimeout(function() {
var c = net.createConnection(common.PORT); var c = net.createConnection(common.PORT);
var gotData = false; var gotData = false;
c.on('connect', function() {
if (index + 1 < N) {
makeConnection(index + 1);
}
});
c.on('end', function() { c.end(); }); c.on('end', function() { c.end(); });
c.on('data', function(b) { c.on('data', function(b) {
@ -96,7 +99,6 @@ function makeConnection(index) {
index + ' got data, but shouldn\'t have'); index + ' got data, but shouldn\'t have');
} }
}); });
}, index);
} }

Loading…
Cancel
Save