From ea5c1e871cd17ff6f60357049fbcd788d7d8b4c1 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Sun, 10 Dec 2017 15:55:00 -0800
Subject: [PATCH] test: refactor test-child-process-pass-fd

Add a comment explaining the test (especailly why it forks 80 processes.
Use destructuring and an arrow function callback.
---
 test/sequential/test-child-process-pass-fd.js | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/test/sequential/test-child-process-pass-fd.js b/test/sequential/test-child-process-pass-fd.js
index 8cc11f6c11f5b4..73e469cdede2be 100644
--- a/test/sequential/test-child-process-pass-fd.js
+++ b/test/sequential/test-child-process-pass-fd.js
@@ -1,11 +1,20 @@
 'use strict';
 const common = require('../common');
+
+// On some OS X versions, when passing fd's between processes:
+// When the handle associated to a specific file descriptor is closed by the
+// sender process before it's received in the destination, the handle is indeed
+// closed while it should remain opened. In order to fix this behavior, don't
+// close the handle until the `NODE_HANDLE_ACK` is received by the sender.
+// This test is basically `test-cluster-net-send` but creating lots of workers
+// so the issue reproduces on OS X consistently.
+
 if ((process.config.variables.arm_version === '6') ||
     (process.config.variables.arm_version === '7'))
   common.skip('Too slow for armv6 and armv7 bots');
 
 const assert = require('assert');
-const fork = require('child_process').fork;
+const { fork } = require('child_process');
 const net = require('net');
 
 const N = 80;
@@ -46,14 +55,14 @@ if (process.argv[2] !== 'child') {
   process.on('message', common.mustCall());
 
   const server = net.createServer((c) => {
-    process.once('message', function(msg) {
+    process.once('message', (msg) => {
       assert.strictEqual(msg, 'got');
       c.end('hello');
     });
     socketConnected();
   }).unref();
   server.listen(0, common.localhostIPv4, () => {
-    const port = server.address().port;
+    const { port } = server.address();
     socket = net.connect(port, common.localhostIPv4, socketConnected).unref();
   });
 }