test: add trailing commas in async-hooks tests (#45549)

PR-URL: #45549
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

@@ -72,7 +72,6 @@ globals:
 
 overrides:
   - files:
-      - async-hooks/*.js
       - es-module/*.js
       - es-module/*.mjs
       - internet/*.js

test/async-hooks/hook-checks.js

@@ -21,7 +21,7 @@ exports.checkInvocations = function checkInvocations(activity, hooks, stage) {
 
   assert.ok(activity != null,
             `${stageInfo} Trying to check invocation for an activity, ` +
-            'but it was empty/undefined.'
+            'but it was empty/undefined.',
   );
 
   // Check that actual invocations for all hooks match the expected invocations

test/async-hooks/init-hooks.js

@@ -27,7 +27,7 @@ class ActivityCollector {
     ondestroy,
     onpromiseResolve,
     logid = null,
-    logtype = null
+    logtype = null,
   } = {}) {
     this._start = start;
     this._allowNoInit = allowNoInit;
@@ -49,7 +49,7 @@ class ActivityCollector {
       before: this._before.bind(this),
       after: this._after.bind(this),
       destroy: this._destroy.bind(this),
-      promiseResolve: this._promiseResolve.bind(this)
+      promiseResolve: this._promiseResolve.bind(this),
     });
   }
 
@@ -181,7 +181,7 @@ class ActivityCollector {
       // In some cases (e.g. Timeout) the handle is a function, thus the usual
       // `typeof handle === 'object' && handle !== null` check can't be used.
       handleIsObject: handle instanceof Object,
-      handle
+      handle,
     };
     this._stamp(activity, 'init');
     this._activities.set(uid, activity);
@@ -233,7 +233,7 @@ exports = module.exports = function initHooks({
   onpromiseResolve,
   allowNoInit,
   logid,
-  logtype
+  logtype,
 } = {}) {
   return new ActivityCollector(process.hrtime(), {
     oninit,
@@ -243,6 +243,6 @@ exports = module.exports = function initHooks({
     onpromiseResolve,
     allowNoInit,
     logid,
-    logtype
+    logtype,
   });
 };

test/async-hooks/test-async-await.js

@@ -22,7 +22,7 @@ const hooks = initHooks({
   onbefore,
   onafter,
   ondestroy: null,  // Intentionally not tested, since it will be removed soon
-  onpromiseResolve
+  onpromiseResolve,
 });
 hooks.enable();
 

test/async-hooks/test-async-exec-resource-http-32060.js

@@ -12,7 +12,7 @@ const hooked = {};
 createHook({
   init(asyncId, type, triggerAsyncId, resource) {
     hooked[asyncId] = resource;
-  }
+  },
 }).enable();
 
 const server = http.createServer((req, res) => {

test/async-hooks/test-async-exec-resource-http.js

@@ -13,7 +13,7 @@ const hooked = {};
 createHook({
   init(asyncId, type, triggerAsyncId, resource) {
     hooked[asyncId] = resource;
-  }
+  },
 }).enable();
 
 const server = http.createServer((req, res) => {

test/async-hooks/test-async-exec-resource-match.js

@@ -6,7 +6,7 @@ const { readFile } = require('fs');
 const {
   createHook,
   executionAsyncResource,
-  AsyncResource
+  AsyncResource,
 } = require('async_hooks');
 
 // Ignore any asyncIds created before our hook is active.
@@ -32,7 +32,7 @@ createHook({
     if (asyncId >= firstSeenAsyncId) {
       afterHook(asyncId);
     }
-  }
+  },
 }).enable();
 
 const beforeHook = common.mustCallAtLeast(

test/async-hooks/test-async-local-storage-http-agent.js

@@ -7,7 +7,7 @@ const http = require('http');
 const asyncLocalStorage = new AsyncLocalStorage();
 
 const agent = new http.Agent({
-  maxSockets: 1
+  maxSockets: 1,
 });
 
 const N = 3;

test/async-hooks/test-async-local-storage-stop-propagation.js

@@ -18,7 +18,7 @@ function onPropagate(type, store) {
 }
 
 const als = new AsyncLocalStorage({
-  onPropagate: common.mustCall(onPropagate, 2)
+  onPropagate: common.mustCall(onPropagate, 2),
 });
 
 const myStore = {};
@@ -40,11 +40,11 @@ als.run(myStore, common.mustCall(() => {
 assert.throws(() => new AsyncLocalStorage(15), {
   message: 'The "options" argument must be of type object. Received type number (15)',
   code: 'ERR_INVALID_ARG_TYPE',
-  name: 'TypeError'
+  name: 'TypeError',
 });
 
 assert.throws(() => new AsyncLocalStorage({ onPropagate: 'bar' }), {
   message: 'The "options.onPropagate" property must be of type function. Received type string (\'bar\')',
   code: 'ERR_INVALID_ARG_TYPE',
-  name: 'TypeError'
+  name: 'TypeError',
 });

test/async-hooks/test-async-local-storage-thenable.js

@@ -17,7 +17,7 @@ const then = common.mustCall((cb) => {
 
 function thenable() {
   return {
-    then
+    then,
   };
 }