test: add trailing commas in test/known_issues

aduh95 · danielleadams · commit 9bb08527f043 · 2023-04-02T21:36:04.000-04:00
PR-URL: #46408 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
@@ -75,7 +75,6 @@ overrides:
       - es-module/*.js
       - es-module/*.mjs
       - js-native-api/*/*.js
-      - known_issues/*.js
       - parallel/*.js
       - parallel/*.mjs
       - pummel/*.js
diff --git a/test/known_issues/test-dgram-bind-shared-ports-after-port-0.js b/test/known_issues/test-dgram-bind-shared-ports-after-port-0.js
@@ -51,6 +51,6 @@ if (cluster.isPrimary) {
   socket1.on('error', PRT1 === 0 ? () => {} : assert.fail);
   socket1.bind(
     { address: common.localhostIPv4, port: PRT1, exclusive: false },
-    () => process.send({ message: 'success', port1: socket1.address().port })
+    () => process.send({ message: 'success', port1: socket1.address().port }),
   );
 }
diff --git a/test/known_issues/test-http-path-contains-unicode.js b/test/known_issues/test-http-path-contains-unicode.js
@@ -26,7 +26,7 @@ server.listen(0, () => {
   http.request({
     port: server.address().port,
     path: expected,
-    method: 'GET'
+    method: 'GET',
   }, common.mustCall(function(res) {
     res.resume();
   })).on('error', function(e) {
diff --git a/test/known_issues/test-repl-require-context.js b/test/known_issues/test-repl-require-context.js
@@ -11,7 +11,7 @@ const fixture = path('is-object.js');
 const r = repl.start({
   input: inputStream,
   output: outputStream,
-  useGlobal: false
+  useGlobal: false,
 });
 
 let output = '';
diff --git a/test/known_issues/test-stdin-is-always-net.socket.js b/test/known_issues/test-stdin-is-always-net.socket.js
@@ -14,7 +14,7 @@ if (process.argv[2] === 'child') {
 const proc = spawn(
   process.execPath,
   [__filename, 'child'],
-  { stdio: 'ignore' }
+  { stdio: 'ignore' },
 );
 // To double-check this test, set stdio to 'pipe' and uncomment the line below.
 // proc.stderr.pipe(process.stderr);
diff --git a/test/known_issues/test-url-parse-conformance.js b/test/known_issues/test-url-parse-conformance.js
@@ -6,7 +6,7 @@ const url = require('url');
 const assert = require('assert');
 const fixtures = require('../common/fixtures');
 const tests = require(
-  fixtures.path('wpt', 'url', 'resources', 'urltestdata.json')
+  fixtures.path('wpt', 'url', 'resources', 'urltestdata.json'),
 );
 
 let failed = 0;
diff --git a/test/known_issues/test-vm-ownkeys.js b/test/known_issues/test-vm-ownkeys.js
@@ -8,7 +8,7 @@ const sym1 = Symbol('1');
 const sym2 = Symbol('2');
 const sandbox = {
   a: true,
-  [sym1]: true
+  [sym1]: true,
 };
 Object.defineProperty(sandbox, 'b', { value: true });
 Object.defineProperty(sandbox, sym2, { value: true });
diff --git a/test/known_issues/test-vm-ownpropertynames.js b/test/known_issues/test-vm-ownpropertynames.js
@@ -8,7 +8,7 @@ const sym1 = Symbol('1');
 const sym2 = Symbol('2');
 const sandbox = {
   a: true,
-  [sym1]: true
+  [sym1]: true,
 };
 Object.defineProperty(sandbox, 'b', { value: true });
 Object.defineProperty(sandbox, sym2, { value: true });
diff --git a/test/known_issues/test-vm-ownpropertysymbols.js b/test/known_issues/test-vm-ownpropertysymbols.js
@@ -8,7 +8,7 @@ const sym1 = Symbol('1');
 const sym2 = Symbol('2');
 const sandbox = {
   a: true,
-  [sym1]: true
+  [sym1]: true,
 };
 Object.defineProperty(sandbox, 'b', { value: true });
 Object.defineProperty(sandbox, sym2, { value: true });
diff --git a/test/known_issues/test-vm-timeout-escape-nexttick.js b/test/known_issues/test-vm-timeout-escape-nexttick.js
@@ -36,11 +36,11 @@ for (let i = 0; i < 4; i++) {
       {
         hrtime,
         nextTick,
-        loop
+        loop,
       },
-      { timeout: common.platformTimeout(10) }
+      { timeout: common.platformTimeout(10) },
     );
   }, {
-    code: 'ERR_SCRIPT_EXECUTION_TIMEOUT'
+    code: 'ERR_SCRIPT_EXECUTION_TIMEOUT',
   });
 }
diff --git a/test/known_issues/test-vm-timeout-escape-queuemicrotask.js b/test/known_issues/test-vm-timeout-escape-queuemicrotask.js
@@ -33,11 +33,11 @@ assert.throws(() => {
     {
       hrtime,
       queueMicrotask,
-      loop
+      loop,
     },
-    { timeout, microtaskMode: 'afterScriptRun' }
+    { timeout, microtaskMode: 'afterScriptRun' },
   );
 }, {
   code: 'ERR_SCRIPT_EXECUTION_TIMEOUT',
-  message: `Script execution timed out after ${timeout}ms`
+  message: `Script execution timed out after ${timeout}ms`,
 });

Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,6 @@ if (cluster.isPrimary) {`
`51`	`51`	`socket1.on('error', PRT1 === 0 ? () => {} : assert.fail);`
`52`	`52`	`socket1.bind(`
`53`	`53`	`{ address: common.localhostIPv4, port: PRT1, exclusive: false },`
`54`		`- () => process.send({ message: 'success', port1: socket1.address().port })`
	`54`	`+ () => process.send({ message: 'success', port1: socket1.address().port }),`
`55`	`55`	`);`
`56`	`56`	`}`