diff options
author | Rich Trott <rtrott@gmail.com> | 2016-01-27 22:00:33 -0800 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2016-01-31 13:45:04 -0800 |
commit | 25f8a0d2dc7be14dcdd28c69ed542c8d699a65ce (patch) | |
tree | 28b9e2d4ecc5feee5ca9911315285cde0989a7da /test/parallel/test-cluster-shared-leak.js | |
parent | 0c113e8fde95ea0b61fe2c9864807a92cc4d3337 (diff) | |
download | android-node-v8-25f8a0d2dc7be14dcdd28c69ed542c8d699a65ce.tar.gz android-node-v8-25f8a0d2dc7be14dcdd28c69ed542c8d699a65ce.tar.bz2 android-node-v8-25f8a0d2dc7be14dcdd28c69ed542c8d699a65ce.zip |
test: fix flaky cluster test on Windows 10
test-cluster-shared-leak was flaky on Windows 10. Remove unnecessary
.send() calls and replace with .disconnect() to avoid spurious EPIPE.
Fixes: https://github.com/nodejs/node/issues/4887
PR-URL: https://github.com/nodejs/node/pull/4934
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'test/parallel/test-cluster-shared-leak.js')
-rw-r--r-- | test/parallel/test-cluster-shared-leak.js | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/test/parallel/test-cluster-shared-leak.js b/test/parallel/test-cluster-shared-leak.js index 55df57c983..9235c1ea60 100644 --- a/test/parallel/test-cluster-shared-leak.js +++ b/test/parallel/test-cluster-shared-leak.js @@ -17,8 +17,8 @@ if (cluster.isMaster) { worker2 = cluster.fork(); worker2.on('online', function() { conn = net.connect(common.PORT, common.mustCall(function() { - worker1.send('die'); - worker2.send('die'); + worker1.disconnect(); + worker2.disconnect(); })); conn.on('error', function(e) { // ECONNRESET is OK @@ -39,17 +39,10 @@ if (cluster.isMaster) { return; } -var server = net.createServer(function(c) { +const server = net.createServer(function(c) { c.end('bye'); }); server.listen(common.PORT, function() { process.send('listening'); }); - -process.on('message', function(msg) { - if (msg !== 'die') return; - server.close(function() { - setImmediate(() => process.disconnect()); - }); -}); |