cluster: return worker reference from disconnect()

Changes disconnect() to return a refererence to the worker.
This will enable method chaining such as

    worker.disconnect().once('disconnect', doThis);

PR-URL: #10019
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Santiago Gimeno <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
Reviewed-By: Sam Roberts <[email protected]>

Author: stv8

doc/api/cluster.md

@@ -257,6 +257,8 @@ It is not emitted in the worker.
 added: v0.7.7
 -->
 
+* Returns: {Worker} A reference to `worker`.
+
 In a worker, this function will close all servers, wait for the `'close'` event on
 those servers, and then disconnect the IPC channel.
 

lib/cluster.js

@@ -430,6 +430,7 @@ function masterInit() {
     send(this, { act: 'disconnect' });
     removeHandlesForWorker(this);
     removeWorker(this);
+    return this;
   };
 
   Worker.prototype.destroy = function(signo) {
@@ -687,6 +688,7 @@ function workerInit() {
 
   Worker.prototype.disconnect = function() {
     _disconnect.call(this);
+    return this;
   };
 
   Worker.prototype.destroy = function() {

test/parallel/test-cluster-worker-destroy.js

@@ -8,6 +8,7 @@
  */
 
 const common = require('../common');
+const assert = require('assert');
 var cluster = require('cluster');
 var worker1, worker2;
 
@@ -26,7 +27,8 @@ if (cluster.isMaster) {
       cluster.worker.destroy();
     });
 
-    cluster.worker.disconnect();
+    const w = cluster.worker.disconnect();
+    assert.strictEqual(w, cluster.worker, 'did not return a reference');
   } else {
     // Call destroy when worker is not disconnected yet
     cluster.worker.destroy();

test/parallel/test-cluster-worker-disconnect.js

@@ -40,7 +40,8 @@ if (cluster.isWorker) {
 
   // Disconnect worker when it is ready
   worker.once('listening', common.mustCall(() => {
-    worker.disconnect();
+    const w = worker.disconnect();
+    assert.strictEqual(worker, w, 'did not return a reference');
   }));
 
   // Check cluster events

test/parallel/test-cluster-worker-init.js

@@ -13,7 +13,8 @@ if (cluster.isMaster) {
 
   worker.on('message', common.mustCall((message) => {
     assert.strictEqual(message, true, 'did not receive expected message');
-    worker.disconnect();
+    const w = worker.disconnect();
+    assert.strictEqual(worker, w, 'did not return a reference');
   }));
 
   worker.on('online', () => {