diff options
| author | Ben Noordhuis <info@bnoordhuis.nl> | 2016-02-22 12:30:33 +0100 |
|---|---|---|
| committer | Ben Noordhuis <info@bnoordhuis.nl> | 2016-02-25 12:46:37 +0100 |
| commit | 66f4586dd031435984ed77837512531dfc4fe893 (patch) | |
| tree | ea21b7ab7f04866ad833a3ee9c15f372be712237 /test/parallel | |
| parent | 0dc216f89ddc09cc76030c85928d3976260dcc3c (diff) | |
| download | node-new-66f4586dd031435984ed77837512531dfc4fe893.tar.gz | |
cluster: emit worker as first 'message' event arg
It's documented as such but didn't actually behave that way.
Bug introduced in commit 66fc8ca ("cluster: emit 'message' event on
cluster master"), which is the commit that introduced the event.
Fixes: https://github.com/nodejs/node/issues/5126
PR-URL: https://github.com/nodejs/node/pull/5361
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Diffstat (limited to 'test/parallel')
| -rw-r--r-- | test/parallel/test-cluster-message.js | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/test/parallel/test-cluster-message.js b/test/parallel/test-cluster-message.js index 427c305890..7b074b9034 100644 --- a/test/parallel/test-cluster-message.js +++ b/test/parallel/test-cluster-message.js @@ -84,7 +84,8 @@ else if (cluster.isMaster) { worker.on('message', function(message) { check('master', message === 'message from worker'); }); - cluster.on('message', function(message) { + cluster.on('message', function(worker_, message) { + assert.strictEqual(worker_, worker); check('global', message === 'message from worker'); }); |