test: replace forEach() with for ... of loop in test-global.js

shizuka1308 · UlisesGascon · commit 7cacddfcc1b3 · 2023-12-11T11:12:51.000Z
PR-URL: #49772 Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/test/parallel/test-global.js b/test/parallel/test-global.js
@@ -30,7 +30,7 @@ const assert = require('assert');
 const{builtinModules } = require('module');
 
 // Load all modules to actually cover most code parts.
-builtinModules.forEach((moduleName) =>{
+for (const moduleName of builtinModules){
 if (!moduleName.includes('/')){
 try{
 // This could throw for e.g., crypto if the binary is not compiled
@@ -40,7 +40,7 @@ builtinModules.forEach((moduleName) =>{
 // Continue regardless of error.
 }
 }
-});
+}
 
{
 const expected = [

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ const assert = require('assert');`
`30`	`30`	`const{ builtinModules }=require('module');`
`31`	`31`
`32`	`32`	`// Load all modules to actually cover most code parts.`
`33`		`-builtinModules.forEach((moduleName)=>{`
	`33`	`+for(constmoduleNameofbuiltinModules){`
`34`	`34`	`if(!moduleName.includes('/')){`
`35`	`35`	`try{`
`36`	`36`	`// This could throw for e.g., crypto if the binary is not compiled`
`@@ -40,7 +40,7 @@ builtinModules.forEach((moduleName) =>{`
`40`	`40`	`// Continue regardless of error.`
`41`	`41`	`}`
`42`	`42`	`}`
`43`		`-});`
	`43`	`+}`
`44`	`44`
`45`	`45`	`{`
`46`	`46`	`constexpected=[`