test: refactor cluster-net-listen-relative-path

Refactor test-cluster-net-listen-relative-path:
* Use arrow funcitons for callbacks.
* Move skip-test code closer to start of file.
* Use assert.ok() where appropriate.
* Capitalize and punctuate comments.
PR-URL: #21863
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

Author: Trott

test/parallel/test-cluster-net-listen-relative-path.js

@@ -1,48 +1,46 @@
 'use strict';
 constcommon=require('../common');
-constassert=require('assert');
-constcluster=require('cluster');
-constnet=require('net');
-constpath=require('path');
-constfs=require('fs');
-
-consttmpdir=require('../common/tmpdir');
 
 if(common.isWindows)
 common.skip('On Windows named pipes live in their own '+
 'filesystem and don\'t have a ~100 byte limit');
 if(!common.isMainThread)
 common.skip('process.chdir is not available in Workers');
 
+constassert=require('assert');
+constcluster=require('cluster');
+constfs=require('fs');
+constnet=require('net');
+constpath=require('path');
+
+consttmpdir=require('../common/tmpdir');
+
 // Choose a socket name such that the absolute path would exceed 100 bytes.
 constsocketDir='./unix-socket-dir';
 constsocketName='A'.repeat(100-socketDir.length-1);
 
-// Make sure we're not in a weird environment
-assert.strictEqual(path.resolve(socketDir,socketName).length>100,true,
-'absolute socket path should be longer than 100 bytes');
+// Make sure we're not in a weird environment.
+assert.ok(path.resolve(socketDir,socketName).length>100,
+'absolute socket path should be longer than 100 bytes');
 
 if(cluster.isMaster){
-// ensure that the worker exits peacefully
+// Ensure that the worker exits peacefully.
 tmpdir.refresh();
 process.chdir(tmpdir.path);
 fs.mkdirSync(socketDir);
-cluster.fork().on('exit',common.mustCall(function(statusCode){
+cluster.fork().on('exit',common.mustCall((statusCode)=>{
 assert.strictEqual(statusCode,0);
 
-assert.strictEqual(
-fs.existsSync(path.join(socketDir,socketName)),false,
-'Socket should be removed when the worker exits');
+assert.ok(!fs.existsSync(path.join(socketDir,socketName)),
+'Socket should be removed when the worker exits');
 }));
 }else{
 process.chdir(socketDir);
 
 constserver=net.createServer(common.mustNotCall());
 
-server.listen(socketName,common.mustCall(function(){
-assert.strictEqual(
-fs.existsSync(socketName),true,
-'Socket created in CWD');
+server.listen(socketName,common.mustCall(()=>{
+assert.ok(fs.existsSync(socketName),'Socket created in CWD');
 
 process.disconnect();
 }));