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: #49825
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
  • Loading branch information
niyashiyas committed Oct 7, 2023
1 parent 2449247 commit fce8fba
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 fce8fba

Please sign in to comment.