test: add trailing commas in test/internet

aduh95 · danielleadams · commit 231f0ee9b215 · 2023-04-02T21:36:04.000-04:00
PR-URL: #46407 Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
diff --git a/test/.eslintrc.yaml b/test/.eslintrc.yaml
@@ -74,7 +74,6 @@ overrides:
 - files:
 - es-module/*.js
 - es-module/*.mjs
- - internet/*.js
 - js-native-api/*/*.js
 - known_issues/*.js
 - parallel/*.js
diff --git a/test/internet/test-corepack-yarn-install.js b/test/internet/test-corepack-yarn-install.js
@@ -25,13 +25,13 @@ const corepackYarnPath = path.join(
 'deps',
 'corepack',
 'dist',
- 'yarn.js'
+ 'yarn.js',
 );
 
 const pkgContent = JSON.stringify({
 dependencies:{
- 'package-name': fixtures.path('packages/main')
- }
+ 'package-name': fixtures.path('packages/main'),
+ },
 });
 
 const pkgPath = path.join(installDir, 'package.json');
@@ -46,7 +46,7 @@ const env ={...process.env,
 
 exec(`${process.execPath} ${corepackYarnPath} install`,{
 cwd: installDir,
- env: env
+ env: env,
 }, common.mustCall(handleExit));
 
 function handleExit(error, stdout, stderr){
diff --git a/test/internet/test-dgram-broadcast-multi-process.js b/test/internet/test-dgram-broadcast-multi-process.js
@@ -166,7 +166,7 @@ if (process.argv[2] !== 'child'){
 
 const sendSocket = dgram.createSocket({
 type: 'udp4',
- reuseAddr: true
+ reuseAddr: true,
 });
 
 // Bind the address explicitly for sending
@@ -203,7 +203,7 @@ if (process.argv[2] !== 'child'){
 LOCAL_BROADCAST_HOST, common.PORT);
 
 process.nextTick(sendSocket.sendNext);
- }
+ },
 );
 };
 
@@ -219,7 +219,7 @@ if (process.argv[2] === 'child'){
 const receivedMessages = [];
 const listenSocket = dgram.createSocket({
 type: 'udp4',
- reuseAddr: true
+ reuseAddr: true,
 });
 
 listenSocket.on('message', (buf, rinfo) =>{
diff --git a/test/internet/test-dgram-membership.js b/test/internet/test-dgram-membership.js
@@ -19,7 +19,7 @@ const setup = dgram.createSocket.bind(dgram,{type: 'udp4', reuseAddr: true });
 const socket = setup();
 assert.throws(
 () =>{socket.dropMembership(multicastAddress)},
- /^Error: dropMembership EADDRNOTAVAIL$/
+ /^Error: dropMembership EADDRNOTAVAIL$/,
 );
 socket.close();
 }
diff --git a/test/internet/test-dgram-multicast-multi-process.js b/test/internet/test-dgram-multicast-multi-process.js
@@ -188,7 +188,7 @@ if (process.argv[2] !== 'child'){
 buf.toString(),
 LOCAL_BROADCAST_HOST, common.PORT);
 process.nextTick(sendSocket.sendNext);
- }
+ },
 );
 };
 }
@@ -197,7 +197,7 @@ if (process.argv[2] === 'child'){
 const receivedMessages = [];
 const listenSocket = dgram.createSocket({
 type: 'udp4',
- reuseAddr: true
+ reuseAddr: true,
 });
 
 listenSocket.on('listening', function(){
diff --git a/test/internet/test-dgram-multicast-set-interface-lo.js b/test/internet/test-dgram-multicast-set-interface-lo.js
@@ -25,7 +25,7 @@ const networkInterfaces = require('os').networkInterfaces();
 const fork = require('child_process').fork;
 const MULTICASTS ={
 IPv4: ['224.0.0.115', '224.0.0.116', '224.0.0.117'],
- IPv6: ['ff02::1:115', 'ff02::1:116', 'ff02::1:117']
+ IPv6: ['ff02::1:115', 'ff02::1:116', 'ff02::1:117'],
 };
 const LOOPBACK ={IPv4: '127.0.0.1', IPv6: '::1' };
 const ANY ={IPv4: '0.0.0.0', IPv6: '::' };
@@ -201,7 +201,7 @@ if (process.argv[2] !== 'child'){
 
 const sendSocket = dgram.createSocket({
 type: UDP[FAM],
- reuseAddr: true
+ reuseAddr: true,
 });
 
 // Don't bind the address explicitly when sending and start with
@@ -241,7 +241,7 @@ if (process.argv[2] !== 'child'){
 msg.mcast, PORTS[msg.mcast]);
 
 process.nextTick(sendSocket.sendNext);
- }
+ },
 );
 };
 }
@@ -256,7 +256,7 @@ if (process.argv[2] === 'child'){
 console.error(`pid ${process.pid} iface ${IFACE} MULTICAST ${MULTICAST}`);
 const listenSocket = dgram.createSocket({
 type: UDP[FAM],
- reuseAddr: true
+ reuseAddr: true,
 });
 
 listenSocket.on('message', (buf, rinfo) =>{
diff --git a/test/internet/test-dgram-multicast-ssm-multi-process.js b/test/internet/test-dgram-multicast-ssm-multi-process.js
@@ -182,7 +182,7 @@ if (process.argv[2] !== 'child'){
 buf.toString(),
 GROUP_ADDRESS, common.PORT);
 process.nextTick(sendSocket.sendNext);
- }
+ },
 );
 };
 }
@@ -191,7 +191,7 @@ if (process.argv[2] === 'child'){
 const receivedMessages = [];
 const listenSocket = dgram.createSocket({
 type: 'udp4',
- reuseAddr: true
+ reuseAddr: true,
 });
 
 listenSocket.on('listening', function(){
diff --git a/test/internet/test-dgram-multicast-ssmv6-multi-process.js b/test/internet/test-dgram-multicast-ssmv6-multi-process.js
@@ -182,7 +182,7 @@ if (process.argv[2] !== 'child'){
 buf.toString(),
 GROUP_ADDRESS, common.PORT);
 process.nextTick(sendSocket.sendNext);
- }
+ },
 );
 };
 }
@@ -191,7 +191,7 @@ if (process.argv[2] === 'child'){
 const receivedMessages = [];
 const listenSocket = dgram.createSocket({
 type: 'udp6',
- reuseAddr: true
+ reuseAddr: true,
 });
 
 listenSocket.on('listening', function(){
diff --git a/test/internet/test-dns-any.js b/test/internet/test-dns-any.js
@@ -78,7 +78,7 @@ const checkers ={
 assert.strictEqual(typeof r.priority, 'number');
 assert.strictEqual(typeof r.weight, 'number');
 assert.strictEqual(r.type, 'SRV');
- }
+ },
 };
 
 function TEST(f){
diff --git a/test/internet/test-dns-idna2008.js b/test/internet/test-dns-idna2008.js
@@ -34,7 +34,7 @@ dns.lookup(
 }
 assert.ifError(err);
 assert.strictEqual(address, fixture.expectedAddress);
- })
+ }),
 );
 
 dns.promises.lookup(fixture.hostname,{family: fixture.family })
diff --git a/test/internet/test-dns-ipv4.js b/test/internet/test-dns-ipv4.js
@@ -127,7 +127,7 @@ TEST(async function test_lookup_ipv4_explicit_object(done){
 validateResult(await dnsPromises.lookup(addresses.INET4_HOST,{family: 4 }));
 
 const req = dns.lookup(addresses.INET4_HOST,{
- family: 4
+ family: 4,
 }, common.mustSucceed((ip, family) =>{
 validateResult({address: ip, family });
 done();
@@ -143,11 +143,11 @@ TEST(async function test_lookup_ipv4_hint_addrconfig(done){
 }
 
 validateResult(await dnsPromises.lookup(addresses.INET4_HOST,{
- hints: dns.ADDRCONFIG
+ hints: dns.ADDRCONFIG,
 }));
 
 const req = dns.lookup(addresses.INET4_HOST,{
- hints: dns.ADDRCONFIG
+ hints: dns.ADDRCONFIG,
 }, common.mustSucceed((ip, family) =>{
 validateResult({address: ip, family });
 done();
@@ -203,7 +203,7 @@ TEST(async function test_lookup_all_ipv4(done){
 
 validateResult(await dnsPromises.lookup(addresses.INET4_HOST,{
 all: true,
- family: 4
+ family: 4,
 }));
 
 const req = dns.lookup(
@@ -212,7 +212,7 @@ TEST(async function test_lookup_all_ipv4(done){
 common.mustSucceed((ips) =>{
 validateResult(ips);
 done();
- })
+ }),
 );
 
 checkWrap(req);
@@ -232,7 +232,7 @@ TEST(async function test_lookupservice_ip_ipv4(done){
 common.mustSucceed((hostname, service) =>{
 validateResult({hostname, service });
 done();
- })
+ }),
 );
 
 checkWrap(req);
diff --git a/test/internet/test-dns-ipv6.js b/test/internet/test-dns-ipv6.js
@@ -122,7 +122,7 @@ TEST(async function test_lookup_ipv6_explicit_object(done){
 validateResult(await dnsPromises.lookup(addresses.INET6_HOST,{family: 6 }));
 
 const req = dns.lookup(addresses.INET6_HOST,{
- family: 6
+ family: 6,
 }, common.mustSucceed((ip, family) =>{
 validateResult({address: ip, family });
 done();
@@ -134,7 +134,7 @@ TEST(async function test_lookup_ipv6_explicit_object(done){
 TEST(function test_lookup_ipv6_hint(done){
 const req = dns.lookup(addresses.INET6_HOST,{
 family: 6,
- hints: dns.V4MAPPED
+ hints: dns.V4MAPPED,
 }, common.mustCall((err, ip, family) =>{
 if (err){
 // FreeBSD does not support V4MAPPED
@@ -191,7 +191,7 @@ TEST(async function test_lookup_all_ipv6(done){
 
 validateResult(await dnsPromises.lookup(addresses.INET6_HOST,{
 all: true,
- family: 6
+ family: 6,
 }));
 
 const req = dns.lookup(
@@ -200,7 +200,7 @@ TEST(async function test_lookup_all_ipv6(done){
 common.mustSucceed((ips) =>{
 validateResult(ips);
 done();
- })
+ }),
 );
 
 checkWrap(req);
@@ -220,7 +220,7 @@ TEST(function test_lookupservice_ip_ipv6(done){
 assert(host);
 assert(['http', 'www', '80'].includes(service));
 done();
- })
+ }),
 );
 
 checkWrap(req);
diff --git a/test/internet/test-dns-lookup.js b/test/internet/test-dns-lookup.js
@@ -11,35 +11,35 @@ assert.rejects(
 dnsPromises.lookup(addresses.NOT_FOUND,{
 hints: 0,
 family: 0,
- all: false
+ all: false,
 }),
{
 code: 'ENOTFOUND',
- message: `getaddrinfo ENOTFOUND ${addresses.NOT_FOUND}`
- }
+ message: `getaddrinfo ENOTFOUND ${addresses.NOT_FOUND}`,
+ },
 );
 
 assert.rejects(
 dnsPromises.lookup(addresses.NOT_FOUND,{
 hints: 0,
 family: 0,
- all: true
+ all: true,
 }),
{
 code: 'ENOTFOUND',
- message: `getaddrinfo ENOTFOUND ${addresses.NOT_FOUND}`
- }
+ message: `getaddrinfo ENOTFOUND ${addresses.NOT_FOUND}`,
+ },
 );
 
 dns.lookup(addresses.NOT_FOUND,{
 hints: 0,
 family: 0,
- all: true
+ all: true,
 }, common.mustCall((error) =>{
 assert.strictEqual(error.code, 'ENOTFOUND');
 assert.strictEqual(
 error.message,
- `getaddrinfo ENOTFOUND ${addresses.NOT_FOUND}`
+ `getaddrinfo ENOTFOUND ${addresses.NOT_FOUND}`,
 );
 assert.strictEqual(error.syscall, 'getaddrinfo');
 assert.strictEqual(error.hostname, addresses.NOT_FOUND);
@@ -48,7 +48,7 @@ dns.lookup(addresses.NOT_FOUND,{
 assert.throws(
 () => dnsPromises.lookup(addresses.NOT_FOUND,{
 family: 'ipv4',
- all: 'all'
+ all: 'all',
 }),
-{code: 'ERR_INVALID_ARG_VALUE' }
+{code: 'ERR_INVALID_ARG_VALUE' },
 );
diff --git a/test/internet/test-dns-promises-resolve.js b/test/internet/test-dns-promises-resolve.js
@@ -12,8 +12,8 @@ const dnsPromises = require('dns').promises;
{
 code: 'ERR_INVALID_ARG_VALUE',
 name: 'TypeError',
- message: `The argument 'rrtype' is invalid. Received '${rrtype}'`
- }
+ message: `The argument 'rrtype' is invalid. Received '${rrtype}'`,
+ },
 );
 }
 
@@ -26,8 +26,8 @@ const dnsPromises = require('dns').promises;
 code: 'ERR_INVALID_ARG_TYPE',
 name: 'TypeError',
 message: 'The "rrtype" argument must be of type string. ' +
- `Received type ${typeof rrtype} (${rrtype})`
- }
+ `Received type ${typeof rrtype} (${rrtype})`,
+ },
 );
 }
 
diff --git a/test/internet/test-dns.js b/test/internet/test-dns.js
diff --git a/test/internet/test-http-dns-fail.js b/test/internet/test-http-dns-fail.js
diff --git a/test/internet/test-tls-add-ca-cert.js b/test/internet/test-tls-add-ca-cert.js
diff --git a/test/internet/test-trace-events-dns.js b/test/internet/test-trace-events-dns.js

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ const setup = dgram.createSocket.bind(dgram,{type: 'udp4', reuseAddr: true });`
`19`	`19`	`constsocket=setup();`
`20`	`20`	`assert.throws(`
`21`	`21`	`()=>{socket.dropMembership(multicastAddress);},`
`22`		`-/^Error:dropMembershipEADDRNOTAVAIL$/`
	`22`	`+/^Error:dropMembershipEADDRNOTAVAIL$/,`
`23`	`23`	`);`
`24`	`24`	`socket.close();`
`25`	`25`	`}`
-Original file line number
+Diff line change
 buf.toString(),
 LOCAL_BROADCAST_HOST,common.PORT);
 process.nextTick(sendSocket.sendNext);
 -}
 +},
 );
 };
+}
 constreceivedMessages=[];
 constlistenSocket=dgram.createSocket({
 type: 'udp4',
 -reuseAddr: true
 +reuseAddr: true,
 });
 listenSocket.on('listening',function(){
-Original file line number
+Diff line change
 assert.strictEqual(typeofr.priority,'number');
 assert.strictEqual(typeofr.weight,'number');
 assert.strictEqual(r.type,'SRV');
 -}
 +},
 };
 functionTEST(f){
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ dns.lookup(`
`34`	`34`	`}`
`35`	`35`	`assert.ifError(err);`
`36`	`36`	`assert.strictEqual(address,fixture.expectedAddress);`
`37`		`-})`
	`37`	`+}),`
`38`	`38`	`);`
`39`	`39`
`40`	`40`	`dns.promises.lookup(fixture.hostname,{family: fixture.family})`