tools: enforce trailing commas in test/es-module

PR-URL: #60891
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>

Author: aduh95

test/es-module/test-disable-require-module-with-detection.js

@@ -15,5 +15,5 @@ assert.throws(
   () => require('../fixtures/es-modules/loose.js'),
   {
     name: 'SyntaxError',
-    message: /Unexpected token 'export'/
+    message: /Unexpected token 'export'/,
   });

test/es-module/test-dynamic-import-script-lifetime.js

@@ -28,5 +28,5 @@ vm.runInThisContext(code, {
     await m.link(() => {});
     await m.evaluate();
     return m;
-  }
+  },
 });

test/es-module/test-esm-cjs-named-error.mjs

@@ -27,21 +27,21 @@ await assert.rejects(async () => {
   await import(`${fixtureBase}/single-quote.mjs`);
 }, {
   name: 'SyntaxError',
-  message: expectedRelative
+  message: expectedRelative,
 }, 'should support relative specifiers with single quotes');
 
 await assert.rejects(async () => {
   await import(`${fixtureBase}/double-quote.mjs`);
 }, {
   name: 'SyntaxError',
-  message: expectedRelative
+  message: expectedRelative,
 }, 'should support relative specifiers with double quotes');
 
 await assert.rejects(async () => {
   await import(`${fixtureBase}/renamed-import.mjs`);
 }, {
   name: 'SyntaxError',
-  message: expectedRenamed
+  message: expectedRenamed,
 }, 'should correctly format named imports with renames');
 
 await assert.rejects(async () => {
@@ -55,21 +55,21 @@ await assert.rejects(async () => {
   await import(`${fixtureBase}/json-hack.mjs`);
 }, {
   name: 'SyntaxError',
-  message: expectedPackageHack
+  message: expectedPackageHack,
 }, 'should respect recursive package.json for module type');
 
 await assert.rejects(async () => {
   await import(`${fixtureBase}/bare-import-single.mjs`);
 }, {
   name: 'SyntaxError',
-  message: expectedBare
+  message: expectedBare,
 }, 'should support bare specifiers with single quotes');
 
 await assert.rejects(async () => {
   await import(`${fixtureBase}/bare-import-double.mjs`);
 }, {
   name: 'SyntaxError',
-  message: expectedBare
+  message: expectedBare,
 }, 'should support bare specifiers with double quotes');
 
 await assert.rejects(async () => {

test/es-module/test-esm-data-urls.js

@@ -76,7 +76,7 @@ function createBase64URL(mime, body) {
       import('data:application/json;foo="test,",0',
         { with: { type: 'json' } }), {
       name: 'SyntaxError',
-      message: /Unterminated string in JSON at position 3/
+      message: /Unterminated string in JSON at position 3/,
     });
   }
   {

test/es-module/test-esm-dns-promises.mjs

@@ -10,5 +10,5 @@ assert.throws(() => {
 }, {
   code: 'ERR_INVALID_ARG_VALUE',
   name: 'TypeError',
-  message: `The argument 'address' is invalid. Received '${invalidAddress}'`
+  message: `The argument 'address' is invalid. Received '${invalidAddress}'`,
 });

test/es-module/test-esm-dynamic-import.js

@@ -19,7 +19,7 @@ function expectOkNamespace(result) {
     .then(common.mustCall((ns) => {
       const expected = { default: true };
       Object.defineProperty(expected, Symbol.toStringTag, {
-        value: 'Module'
+        value: 'Module',
       });
       Object.setPrototypeOf(expected, Object.getPrototypeOf(ns));
       assert.deepStrictEqual(ns, expected);

test/es-module/test-esm-import-meta.mjs

@@ -12,7 +12,7 @@ for (const descriptor of Object.values(descriptors)) {
   assert.deepStrictEqual(descriptor, {
     enumerable: true,
     writable: true,
-    configurable: true
+    configurable: true,
   });
 }
 

test/es-module/test-esm-loader-invalid-format.mjs

@@ -5,6 +5,6 @@ import assert from 'assert';
 import('../fixtures/es-modules/test-esm-ok.mjs')
 .then(assert.fail, expectsError({
   code: 'ERR_UNKNOWN_MODULE_FORMAT',
-  message: /Unknown module format: esm/
+  message: /Unknown module format: esm/,
 }))
 .then(mustCall());

test/es-module/test-esm-loader-mock.mjs

@@ -3,21 +3,21 @@ import assert from 'node:assert/strict';
 import { mock } from '../fixtures/es-module-loaders/mock.mjs';
 
 mock('node:events', {
-  EventEmitter: 'This is mocked!'
+  EventEmitter: 'This is mocked!',
 });
 
 // This resolves to node:events
 // It is intercepted by mock-loader and doesn't return the normal value
 assert.deepStrictEqual(await import('events'), Object.defineProperty({
   __proto__: null,
-  EventEmitter: 'This is mocked!'
+  EventEmitter: 'This is mocked!',
 }, Symbol.toStringTag, {
   enumerable: false,
-  value: 'Module'
+  value: 'Module',
 }));
 
 const mutator = mock('node:events', {
-  EventEmitter: 'This is mocked v2!'
+  EventEmitter: 'This is mocked v2!',
 });
 
 // It is intercepted by mock-loader and doesn't return the normal value.
@@ -26,17 +26,17 @@ const mutator = mock('node:events', {
 const mockedV2 = await import('node:events');
 assert.deepStrictEqual(mockedV2, Object.defineProperty({
   __proto__: null,
-  EventEmitter: 'This is mocked v2!'
+  EventEmitter: 'This is mocked v2!',
 }, Symbol.toStringTag, {
   enumerable: false,
-  value: 'Module'
+  value: 'Module',
 }));
 
 mutator.EventEmitter = 'This is mocked v3!';
 assert.deepStrictEqual(mockedV2, Object.defineProperty({
   __proto__: null,
-  EventEmitter: 'This is mocked v3!'
+  EventEmitter: 'This is mocked v3!',
 }, Symbol.toStringTag, {
   enumerable: false,
-  value: 'Module'
+  value: 'Module',
 }));

test/es-module/test-esm-loader-modulemap.js

@@ -55,7 +55,7 @@ const jsonModuleJob = new ModuleJob(loader, jsonModuleDataUrl,
   const errorObj = {
     code: 'ERR_INVALID_ARG_TYPE',
     name: 'TypeError',
-    message: /^The "url" argument must be of type string/
+    message: /^The "url" argument must be of type string/,
   };
 
   [{}, [], true, 1].forEach((value) => {
@@ -73,7 +73,7 @@ const jsonModuleJob = new ModuleJob(loader, jsonModuleDataUrl,
   const errorObj = {
     code: 'ERR_INVALID_ARG_TYPE',
     name: 'TypeError',
-    message: /^The "type" argument must be of type string/
+    message: /^The "type" argument must be of type string/,
   };
 
   [{}, [], true, 1].forEach((value) => {
@@ -91,7 +91,7 @@ const jsonModuleJob = new ModuleJob(loader, jsonModuleDataUrl,
     assert.throws(() => moduleMap.set('', undefined, value), {
       code: 'ERR_INVALID_ARG_TYPE',
       name: 'TypeError',
-      message: /^The "job" argument must be an instance of ModuleJob/
+      message: /^The "job" argument must be an instance of ModuleJob/,
     });
   });
 }