replace forEach() with for ... of loop in test-global.js for enhanced code readability and maintainability.

shizuka1308 · shizuka1308 · commit f23a7ccefc56 · 2023-09-22T14:48:41.000-04:00
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 (const moduleName of builtinModules) {`
`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`	`const expected = [`