Skip to content

Commit

Permalink
test: replace forEach with for..of in test-process-env
Browse files Browse the repository at this point in the history
PR-URL: nodejs#49825
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
  • Loading branch information
niyashiyas authored and targos committed Nov 11, 2023
1 parent 81886c6 commit 8dd895e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-process-env-ignore-getter-setter.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ assert.throws(

const attributes = ['configurable', 'writable', 'enumerable'];

attributes.forEach((attribute) => {
for (const attribute of attributes) {
assert.throws(
() => {
Object.defineProperty(process.env, 'goo', {
Expand All @@ -55,7 +55,7 @@ attributes.forEach((attribute) => {
' and enumerable data descriptor'
}
);
});
}

assert.strictEqual(process.env.goo, undefined);
Object.defineProperty(process.env, 'goo', {
Expand Down

0 comments on commit 8dd895e

Please sign in to comment.