bootstrap: correct --frozen-intrinsics override fix

PR-URL: nodejs#35041 Reviewed-By: Bradley Farias <bradley.meck@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Rich Trott <rtrott@gmail.com>
joesepi · Oct 22, 2020 · 6209b15 · 6209b15
1 parent d4d5e4c
commit 6209b15
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/lib/internal/freeze_intrinsics.js b/lib/internal/freeze_intrinsics.js
@@ -384,8 +384,8 @@ module.exports = function() {
           defineProperty(this, prop, {
             value: newValue,
             writable: true,
-            enumerable: desc.enumerable,
-            configurable: desc.configurable
+            enumerable: true,
+            configurable: true
           });
         }
       }

diff --git a/test/parallel/test-freeze-intrinsics.js b/test/parallel/test-freeze-intrinsics.js
@@ -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);
 }