test: refactor callback functions to arrow functions

seanhealy · seanhealy · commit d3a8ae0217c4 · 2018-10-12T10:58:27.000-07:00
Refactor callback functions to modern arrow functions.

Also, added `common.mustCall` to `online` callbacks.
diff --git a/test/parallel/test-cluster-worker-forced-exit.js b/test/parallel/test-cluster-worker-forced-exit.js
@@ -37,10 +37,8 @@ const SENTINEL = 42;
 // 3 disconnect worker with child_process's disconnect, confirm
 //   no sentinel value
 if (cluster.isWorker) {
-  process.on('disconnect', function(msg) {
-    setTimeout(function() {
-      process.exit(SENTINEL);
-    }, 10);
+  process.on('disconnect', (msg) => {
+    setTimeout(() => process.exit(SENTINEL), 10);
   });
   return;
 }
@@ -49,17 +47,17 @@ checkUnforced();
 checkForced();
 
 function checkUnforced() {
-  cluster.fork()
-    .on('online', function() { this.disconnect(); })
-    .on('exit', common.mustCall(function(status) {
+  const worker = cluster.fork();
+  worker
+    .on('online', common.mustCall(() => worker.disconnect()))
+    .on('exit', common.mustCall((status) => {
       assert.strictEqual(status, SENTINEL);
     }));
 }
 
 function checkForced() {
-  cluster.fork()
-    .on('online', function() { this.process.disconnect(); })
-    .on('exit', common.mustCall(function(status) {
-      assert.strictEqual(status, 0);
-    }));
+  const worker = cluster.fork();
+  worker
+    .on('online', common.mustCall(() => worker.process.disconnect()))
+    .on('exit', common.mustCall((status) => assert.strictEqual(status, 0)));
 }
