Skip to content

Commit

Permalink
test: replace forEach() with for .. of
Browse files Browse the repository at this point in the history
PR-URL: #50605
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
bliakher authored and targos committed Nov 14, 2023
1 parent e978fd4 commit b701567
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions test/parallel/test-async-wrap-constructor.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,15 +6,16 @@ require('../common');
const assert = require('assert');
const async_hooks = require('async_hooks');

[0, 1, false, true, null, 'hello'].forEach((badArg) => {
const falsyValues = [0, 1, false, true, null, 'hello'];
for (const badArg of falsyValues) {
const hookNames = ['init', 'before', 'after', 'destroy', 'promiseResolve'];
hookNames.forEach((field) => {
for (const hookName of hookNames) {
assert.throws(() => {
async_hooks.createHook({ [field]: badArg });
async_hooks.createHook({ [hookName]: badArg });
}, {
code: 'ERR_ASYNC_CALLBACK',
name: 'TypeError',
message: `hook.${field} must be a function`
message: `hook.${hookName} must be a function`
});
});
});
}
}

0 comments on commit b701567

Please sign in to comment.