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){
 
 constchild=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){
 constkCpuProfInterval=50;
 constenv={
  ...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){
 constbuf1=crypto.diffieHellman({
 privateKey: alicePrivateKey,
-publicKey: bobPublicKey
+publicKey: bobPublicKey,
 });
 constbuf2=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,
 };
 
 constclasses={
-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){
 constpointeeOffset=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: typeofname_or_index==='string' ? name_or_index : null
+name: typeofname_or_index==='string' ? name_or_index : null,
 };
 toNode.incomingEdges.push(edge);
 fromNode.outgoingEdges.push(edge);
@@ -130,7 +130,7 @@ class State{
 constcheck=typeofexpectedEdge==='function' ? expectedEdge :
 (edge)=>(isEdge(edge,expectedEdge));
 consthasChild=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}={}){
 constrootNodes=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.
 consthasChild=rootNodes.some(
-(node)=>node.edges.some(check)
+(node)=>node.edges.some(check),
 );
 if(!hasChild){
 thrownewError(
@@ -221,16 +221,16 @@ function getHeapSnapshotOptionTests(){
 // We don't have anything special to test here yet
 // because we don't use cppgc or embedder heap tracer.
 {edge_name: 'nonNumeric',node_name: 'test'},
-]
-}]
+],
+}],
 },
 {
 options: {exposeNumericValues: true},
 expected: [{
 children: [
 {edge_name: 'numeric',node_name: 'smi number'},
-]
-}]
+],
+}],
 },
 ];
 return{
@@ -245,5 +245,5 @@ function getHeapSnapshotOptionTests(){
 module.exports={
  recordState,
  validateSnapshotNodes,
- getHeapSnapshotOptionTests
+ getHeapSnapshotOptionTests,
 };

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.',
 );
 constargs=[...flags, ...process.execArgv, ...process.argv.slice(1)];
 constoptions={encoding: 'utf8',stdio: 'inherit'};
@@ -175,7 +175,7 @@ if (process.env.NODE_TEST_WITH_ASYNC_HOOKS){
 }
 initHandles[id]={
  resource,
-stack: inspect(newError()).substr(6)
+stack: inspect(newError()).substr(6),
 };
 },
 before(){},
@@ -435,7 +435,7 @@ function _mustCallInner(fn, criteria = 1, field){
 [field]: criteria,
 actual: 0,
 stack: inspect(newError()),
-name: fn.name||'<anonymous>'
+name: fn.name||'<anonymous>',
 };
 
 // Add the exit listener only once to avoid listener leak warnings
@@ -478,7 +478,7 @@ function hasMultiLocalhost(){
 
 functionskipIfEslintMissing(){
 if(!fs.existsSync(
-path.join(__dirname,'..','..','tools','node_modules','eslint')
+path.join(__dirname,'..','..','tools','node_modules','eslint'),
 )){
 skip('missing ESLint');
 }
@@ -567,7 +567,7 @@ function mustNotMutateObjectDeep(original){
 },
 setPrototypeOf(target,prototype){
 assert.fail(`Expected no side effects, got set prototype to ${prototype}`);
-}
+},
 };
 
 constproxy=newProxy(original,_mustNotMutateObjectDeepHandler);
@@ -670,7 +670,7 @@ function expectWarning(nameOrMap, expected, code){
 if(!catchWarning[warning.name]){
 thrownewTypeError(
 `"${warning.name}" was triggered without being expected.\n`+
-inspect(warning)
+inspect(warning),
 );
 }
 catchWarning[warning.name](warning);
@@ -1034,5 +1034,5 @@ module.exports = new Proxy(common,{
 if(!validProperties.has(prop))
 thrownewError(`Using invalid common property: '${prop}'`);
 returnobj[prop];
-}
+},
 });