test: prepare for consistent comma-dangle lint rule

Make changes so that tests will pass when the comma-dangle settings applied to the rest of the code base are also applied to tests. PR-URL: #37930 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Derek Lewis <[email protected]>

Author: Trott

test/abort/test-worker-abort-uncaught-exception.js

@@ -12,7 +12,7 @@ if (process.argv[2] === 'child'){
 }
 
 constchild=spawn(process.execPath,[
-'--abort-on-uncaught-exception',__filename,'child'
+'--abort-on-uncaught-exception',__filename,'child',
 ]);
 child.on('exit',common.mustCall((code,sig)=>{
 if(common.isWindows){

test/addons/register-signal-handler/test.js

@@ -34,7 +34,7 @@ for (const raiseSignal of [ 'SIGABRT', 'SIGSEGV' ]){
 {reset: true,count: 1,stderr: [signo],code: 0,signal: null},
 {reset: true,count: 2,stderr: [signo],code: null,signal: raiseSignal},
 {reset: false,count: 1,stderr: [signo],code: 0,signal: null},
-{reset: false,count: 2,stderr: [signo,signo],code: 0,signal: null}
+{reset: false,count: 2,stderr: [signo,signo],code: 0,signal: null},
 ]){
 // We do not want to generate core files when running this test as an
 // addon test. We require this file as an abort test as well, though,

test/addons/worker-addon/test.js

@@ -48,7 +48,7 @@ for (const{test, expected } of [
 // musl doesn't support unloading, so the output may be missing
 // a dtor + ctor pair.
 expected: [
-'ctor cleanup dtor ctor cleanup dtor '
+'ctor cleanup dtor ctor cleanup dtor ',
 ].concat(libcMayBeMusl ? [
 'ctor cleanup cleanup dtor ',
 ] : [])
@@ -57,7 +57,7 @@ for (const{test, expected } of [
 console.log('spawning test',test);
 constproc=child_process.spawnSync(process.execPath,[
 __filename,
-test
+test,
 ]);
 process.stderr.write(proc.stderr.toString());
 assert.strictEqual(proc.stderr.toString(),'');

test/async-hooks/test-graph.tls-write.js

@@ -65,7 +65,7 @@ function onexit(){
 {type: 'TCPCONNECTWRAP',
 id: 'tcpconnect:1',triggerAsyncId: 'tcp:1'},
 {type: 'TCPWRAP',id: 'tcp:2',triggerAsyncId: 'tcpserver:1'},
-{type: 'TLSWRAP',id: 'tls:2',triggerAsyncId: 'tcpserver:1'}
+{type: 'TLSWRAP',id: 'tls:2',triggerAsyncId: 'tcpserver:1'},
 ]
 );
 }

test/common/README.md

@@ -115,12 +115,12 @@ expectWarning('Warning', 'Foobar is really bad');
 expectWarning('DeprecationWarning', 'Foobar is deprecated', 'DEP0XXX');
 
 expectWarning('DeprecationWarning', [
-'Foobar is deprecated', 'DEP0XXX'
+'Foobar is deprecated', 'DEP0XXX',
 ]);
 
 expectWarning('DeprecationWarning', [
  ['Foobar is deprecated', 'DEP0XXX'],
- ['Baz is also deprecated', 'DEP0XX2']
+ ['Baz is also deprecated', 'DEP0XX2'],
 ]);
 
 expectWarning('DeprecationWarning',{
@@ -135,7 +135,7 @@ expectWarning({
  },
  Warning: [
  ['Multiple array entries are fine', 'SpecialWarningCode'],
- ['No code is also fine']
+ ['No code is also fine'],
  ],
  SingleEntry: ['This will also work', 'WarningCode'],
  SingleString:'Single string entries without code will also work'
@@ -665,9 +665,9 @@ validateSnapshotNodes('TLSWRAP', [
  children: [
 {name:'enc_out' },
 {name:'enc_in' },
-{name:'TLSWrap' }
+{name:'TLSWrap' },
  ]
- }
+ },
 ]);
 ```
 

test/common/dns.js

@@ -60,7 +60,7 @@ function parseDNSPacket(buffer){
 ['questions',buffer.readUInt16BE(4)],
 ['answers',buffer.readUInt16BE(6)],
 ['authorityAnswers',buffer.readUInt16BE(8)],
-['additionalRecords',buffer.readUInt16BE(10)]
+['additionalRecords',buffer.readUInt16BE(10)],
 ];
 
 letoffset=12;
@@ -185,7 +185,7 @@ function writeDomainName(domain){
 assert(label.length<64);
 returnBuffer.concat([
 Buffer.from([label.length]),
-Buffer.from(label,'ascii')
+Buffer.from(label,'ascii'),
 ]);
 }).concat([Buffer.alloc(1)]));
 }
@@ -208,7 +208,7 @@ function writeDNSPacket(parsed){
 buffers.push(writeDomainName(q.domain));
 buffers.push(newUint16Array([
 types[q.type],
-q.cls===undefined ? classes.IN : q.cls
+q.cls===undefined ? classes.IN : q.cls,
 ]));
 }
 
@@ -221,7 +221,7 @@ function writeDNSPacket(parsed){
 buffers.push(writeDomainName(rr.domain));
 buffers.push(newUint16Array([
 types[rr.type],
-rr.cls===undefined ? classes.IN : rr.cls
+rr.cls===undefined ? classes.IN : rr.cls,
 ]));
 buffers.push(newInt32Array([rr.ttl]));
 
@@ -266,7 +266,7 @@ function writeDNSPacket(parsed){
 rdLengthBuf[0]=mname.length+rname.length+20;
 buffers.push(mname,rname);
 buffers.push(newUint32Array([
-rr.serial,rr.refresh,rr.retry,rr.expire,rr.minttl
+rr.serial,rr.refresh,rr.retry,rr.expire,rr.minttl,
 ]));
 break;
 }

test/common/index.js

@@ -612,7 +612,7 @@ function getArrayBufferViews(buf){
 Uint32Array,
 Float32Array,
 Float64Array,
-DataView
+DataView,
 ];
 
 for(consttypeofarrayBufferViews){

test/common/inspector-helper.js

@@ -515,7 +515,7 @@ function fires(promise, error, timeoutMs){
 consttimeout=timeoutPromise(error,timeoutMs);
 returnPromise.race([
 onResolvedOrRejected(promise,()=>timeout.clear()),
-timeout
+timeout,
 ]);
 }
 

test/common/tls.js

@@ -44,7 +44,7 @@ class TestTLSSocket extends net.Socket{
 createClientHello(){
 constcompressions=Buffer.from('0100','hex');// null
 constmsg=addHandshakeHeader(0x01,Buffer.concat([
-this.version,this.client_random,this.ciphers,compressions
+this.version,this.client_random,this.ciphers,compressions,
 ]));
 this.emit('handshake',msg);
 returnaddRecordHeader(0x16,msg);

test/doctool/test-doctool-json.js

@@ -79,7 +79,7 @@ const testData = [
 name: 'array',
 type: 'Array'
 }]
-}
+},
 ]
 }],
 type: 'module',
@@ -121,7 +121,7 @@ const testData = [
 {'version': 'v4.2.0',
 'pr-url': 'https://github.com/nodejs/node/pull/3276',
 'description': 'The `error` parameter can now be '+
-'an arrow function.'}
+'an arrow function.'},
 ]
 },
 desc: '<p>Describe <code>Foobar II</code> in more detail '+
@@ -149,11 +149,11 @@ const testData = [
 'Describe <code>Something</code> in more detail here.</p>',
 type: 'module',
 displayName: 'Something'
-}
+},
 ],
 type: 'module',
 displayName: 'Sample Markdown with YAML info'
-}
+},
 ]
 }
 },
@@ -170,61 +170,61 @@ const testData = [
 {
 name: 'fullName',
 textRaw: '`Fqhqwhgads.fullName`'
-}
+},
 ],
 classMethods: [
 {
 name: 'again',
 signatures: [
 {
 params: []
-}
+},
 ],
 textRaw: 'Static method: `Fhqwhgads.again()`',
 type: 'classMethod'
-}
+},
 ],
 classes: [
 {
 textRaw: 'Class: `ComeOn`',
 type: 'class',
 name: 'ComeOn'
-}
+},
 ],
 ctors: [
 {
 name: 'Fhqwhgads',
 signatures: [
 {
 params: []
-}
+},
 ],
 textRaw: 'Constructor: `new Fhqwhgads()`',
 type: 'ctor'
-}
+},
 ],
 methods: [
 {
 textRaw: '`everybody.to(limit)`',
 type: 'method',
 name: 'to',
 signatures: [{params: []}]
-}
+},
 ],
 events: [
 {
 textRaw: "Event: `'FHQWHfest'`",
 type: 'event',
 name: 'FHQWHfest',
 params: []
-}
+},
 ],
 type: 'module',
 displayName: 'Fhqwhgads'
-}
+},
 ]
 }
-}
+},
 ];
 
 testData.forEach((item)=>{