test: add trailing commas in test/common (#45550)

PR-URL: #45550
Reviewed-By: Daeyeon Jeong <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>

Author: aduh95

test/.eslintrc.yaml

@@ -74,7 +74,6 @@ overrides:
   - files:
       - addons/*/*.js
       - async-hooks/*.js
-      - common/*.js
       - es-module/*.js
       - es-module/*.mjs
       - internet/*.js

test/common/benchmark.js

@@ -15,7 +15,7 @@ function runBenchmark(name, env) {
 
   const child = fork(runjs, argv, {
     env: mergedEnv,
-    stdio: ['inherit', 'pipe', 'inherit', 'ipc']
+    stdio: ['inherit', 'pipe', 'inherit', 'ipc'],
   });
   child.stdout.setEncoding('utf8');
 
@@ -35,7 +35,7 @@ function runBenchmark(name, env) {
     // get from testing the benchmark file.
     assert.ok(
       /^(?:\n.+?\n.+?\n)+$/.test(stdout),
-      `benchmark file not running exactly one configuration in test: ${stdout}`
+      `benchmark file not running exactly one configuration in test: ${stdout}`,
     );
   });
 }

test/common/child_process.js

@@ -45,5 +45,5 @@ module.exports = {
   cleanupStaleProcess,
   logAfterTime,
   kExpiringChildRunTime,
-  kExpiringParentTimer
+  kExpiringParentTimer,
 };

test/common/cpu-prof.js

@@ -38,13 +38,13 @@ function verifyFrames(output, file, suffix) {
 const kCpuProfInterval = 50;
 const env = {
   ...process.env,
-  NODE_DEBUG_NATIVE: 'INSPECTOR_PROFILER'
+  NODE_DEBUG_NATIVE: 'INSPECTOR_PROFILER',
 };
 
 module.exports = {
   getCpuProfiles,
   kCpuProfInterval,
   env,
   getFrames,
-  verifyFrames
+  verifyFrames,
 };

test/common/crypto.js

@@ -30,11 +30,11 @@ function testDH({ publicKey: alicePublicKey, privateKey: alicePrivateKey },
                 expectedValue) {
   const buf1 = crypto.diffieHellman({
     privateKey: alicePrivateKey,
-    publicKey: bobPublicKey
+    publicKey: bobPublicKey,
   });
   const buf2 = crypto.diffieHellman({
     privateKey: bobPrivateKey,
-    publicKey: alicePublicKey
+    publicKey: alicePublicKey,
   });
   assert.deepStrictEqual(buf1, buf2);
 
@@ -44,5 +44,5 @@ function testDH({ publicKey: alicePublicKey, privateKey: alicePrivateKey },
 
 module.exports = {
   modp2buf,
-  testDH
+  testDH,
 };

test/common/dns.js

@@ -13,11 +13,11 @@ const types = {
   MX: 15,
   TXT: 16,
   ANY: 255,
-  CAA: 257
+  CAA: 257,
 };
 
 const classes = {
-  IN: 1
+  IN: 1,
 };
 
 // Naïve DNS parser/serializer.
@@ -34,7 +34,7 @@ function readDomainFromPacket(buffer, offset) {
     const { nread, domain } = readDomainFromPacket(buffer, offset + length);
     return {
       nread: 1 + length + nread,
-      domain: domain ? `${chunk}.${domain}` : chunk
+      domain: domain ? `${chunk}.${domain}` : chunk,
     };
   }
   // Pointer to another part of the packet.
@@ -43,7 +43,7 @@ function readDomainFromPacket(buffer, offset) {
   const pointeeOffset = buffer.readUInt16BE(offset) &~ 0xC000;
   return {
     nread: 2,
-    domain: readDomainFromPacket(buffer, pointeeOffset)
+    domain: readDomainFromPacket(buffer, pointeeOffset),
   };
 }
 
@@ -316,5 +316,5 @@ module.exports = {
   parseDNSPacket,
   errorLookupMock,
   mockedErrorCode,
-  mockedSysCall
+  mockedSysCall,
 };

test/common/heap.js

@@ -41,7 +41,7 @@ function createJSHeapSnapshot(stream = getHeapSnapshot()) {
       type,
       to: toNode,
       from: fromNode,
-      name: typeof name_or_index === 'string' ? name_or_index : null
+      name: typeof name_or_index === 'string' ? name_or_index : null,
     };
     toNode.incomingEdges.push(edge);
     fromNode.outgoingEdges.push(edge);
@@ -130,7 +130,7 @@ class State {
           const check = typeof expectedEdge === 'function' ? expectedEdge :
             (edge) => (isEdge(edge, expectedEdge));
           const hasChild = rootNodes.some(
-            (node) => node.outgoingEdges.some(check)
+            (node) => node.outgoingEdges.some(check),
           );
           // Don't use assert with a custom message here. Otherwise the
           // inspection in the message is done eagerly and wastes a lot of CPU
@@ -164,7 +164,7 @@ class State {
   // Validate our internal embedded graph representation
   validateGraph(rootName, expected, { loose = false } = {}) {
     const rootNodes = this.embedderGraph.filter(
-      (node) => node.name === rootName
+      (node) => node.name === rootName,
     );
     if (loose) {
       assert(rootNodes.length >= expected.length,
@@ -185,7 +185,7 @@ class State {
           // inspection in the message is done eagerly and wastes a lot of CPU
           // time.
           const hasChild = rootNodes.some(
-            (node) => node.edges.some(check)
+            (node) => node.edges.some(check),
           );
           if (!hasChild) {
             throw new Error(
@@ -213,5 +213,5 @@ function validateSnapshotNodes(...args) {
 
 module.exports = {
   recordState,
-  validateSnapshotNodes
+  validateSnapshotNodes,
 };

test/common/hijackstdio.js

@@ -28,5 +28,5 @@ module.exports = {
   hijackStdout: hijackStdWritable.bind(null, 'stdout'),
   hijackStderr: hijackStdWritable.bind(null, 'stderr'),
   restoreStdout: restoreWritable.bind(null, 'stdout'),
-  restoreStderr: restoreWritable.bind(null, 'stderr')
+  restoreStderr: restoreWritable.bind(null, 'stderr'),
 };

test/common/http2.js

@@ -144,5 +144,5 @@ module.exports = {
   PingFrame,
   kFakeRequestHeaders,
   kFakeResponseHeaders,
-  kClientMagic
+  kClientMagic,
 };

test/common/index.js

@@ -39,7 +39,7 @@ const hasIntl = !!process.config.variables.v8_enable_i18n_support;
 
 const {
   atob,
-  btoa
+  btoa,
 } = require('buffer');
 
 // Some tests assume a umask of 0o022 so set that up front. Tests that need a
@@ -98,7 +98,7 @@ if (process.argv.length === 2 &&
         console.log(
           'NOTE: The test started as a child_process using these flags:',
           inspect(flags),
-          'Use NODE_SKIP_FLAG_CHECK to run the test with the original flags.'
+          'Use NODE_SKIP_FLAG_CHECK to run the test with the original flags.',
         );
         const args = [...flags, ...process.execArgv, ...process.argv.slice(1)];
         const options = { encoding: 'utf8', stdio: 'inherit' };
@@ -175,7 +175,7 @@ if (process.env.NODE_TEST_WITH_ASYNC_HOOKS) {
       }
       initHandles[id] = {
         resource,
-        stack: inspect(new Error()).substr(6)
+        stack: inspect(new Error()).substr(6),
       };
     },
     before() { },
@@ -433,7 +433,7 @@ function _mustCallInner(fn, criteria = 1, field) {
     [field]: criteria,
     actual: 0,
     stack: inspect(new Error()),
-    name: fn.name || '<anonymous>'
+    name: fn.name || '<anonymous>',
   };
 
   // Add the exit listener only once to avoid listener leak warnings
@@ -476,7 +476,7 @@ function hasMultiLocalhost() {
 
 function skipIfEslintMissing() {
   if (!fs.existsSync(
-    path.join(__dirname, '..', '..', 'tools', 'node_modules', 'eslint')
+    path.join(__dirname, '..', '..', 'tools', 'node_modules', 'eslint'),
   )) {
     skip('missing ESLint');
   }
@@ -565,7 +565,7 @@ function mustNotMutateObjectDeep(original) {
     },
     setPrototypeOf(target, prototype) {
       assert.fail(`Expected no side effects, got set prototype to ${prototype}`);
-    }
+    },
   };
 
   const proxy = new Proxy(original, _mustNotMutateObjectDeepHandler);
@@ -668,7 +668,7 @@ function expectWarning(nameOrMap, expected, code) {
       if (!catchWarning[warning.name]) {
         throw new TypeError(
           `"${warning.name}" was triggered without being expected.\n` +
-          inspect(warning)
+          inspect(warning),
         );
       }
       catchWarning[warning.name](warning);
@@ -1032,5 +1032,5 @@ module.exports = new Proxy(common, {
     if (!validProperties.has(prop))
       throw new Error(`Using invalid common property: '${prop}'`);
     return obj[prop];
-  }
+  },
 });