test: test-async-wrap-constructor prefer forEach

dericop · BethGriggs · commit b4ef06267d97 · 2020-04-28T16:28:28.000+01:00
PR-URL: #32631 Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>
diff --git a/test/parallel/test-async-wrap-constructor.js b/test/parallel/test-async-wrap-constructor.js
@@ -6,14 +6,15 @@ require('../common');
 const assert = require('assert');
 const async_hooks = require('async_hooks');
 
-for (const badArg of [0, 1, false, true, null, 'hello']){
+[0, 1, false, true, null, 'hello'].forEach((badArg) =>{
 const hookNames = ['init', 'before', 'after', 'destroy', 'promiseResolve'];
- for (const field of hookNames){
+ hookNames.forEach((field) =>{
 assert.throws(() =>{
 async_hooks.createHook({[field]: badArg });
 },{
 code: 'ERR_ASYNC_CALLBACK',
 name: 'TypeError',
+ message: `hook.${field} must be a function`
 });
- }
-}
+ });
+});

-Original file line number
+Diff line change
 constassert=require('assert');
 constasync_hooks=require('async_hooks');
 -for(constbadArgof[0,1,false,true,null,'hello']){
 +[0,1,false,true,null,'hello'].forEach((badArg)=>{
 consthookNames=['init','before','after','destroy','promiseResolve'];
 -for(constfieldofhookNames){
 +hookNames.forEach((field)=>{
 assert.throws(()=>{
 async_hooks.createHook({[field]: badArg});
 },{
 code: 'ERR_ASYNC_CALLBACK',
 name: 'TypeError',
 +message: `hook.${field} must be a function`
 });
 -}
 -}
 +});
 +});