Skip to content

Commit

Permalink
bootstrap: correct --frozen-intrinsics override fix
Browse files Browse the repository at this point in the history
PR-URL: #35041
Reviewed-By: Bradley Farias <bradley.meck@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
guybedford authored and richardlau committed Sep 7, 2020
1 parent 8044533 commit 1fdfaa5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/internal/freeze_intrinsics.js
Original file line number Diff line number Diff line change
Expand Up @@ -384,8 +384,8 @@ module.exports = function() {
defineProperty(this, prop, {
value: newValue,
writable: true,
enumerable: desc.enumerable,
configurable: desc.configurable
enumerable: true,
configurable: true
});
}
}
Expand Down
2 changes: 2 additions & 0 deletions test/parallel/test-freeze-intrinsics.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,4 +27,6 @@ assert.throws(
const o = {};
o.toString = () => 'Custom toString';
assert.strictEqual(o + 'asdf', 'Custom toStringasdf');
assert.strictEqual(Object.getOwnPropertyDescriptor(o, 'toString').enumerable,
true);
}

0 comments on commit 1fdfaa5

Please sign in to comment.