mirror of
https://github.com/nodejs/node.git
synced 2024-11-21 21:19:50 +01:00
1cd9a95e29
Object.defineProperty allows to change the value for non-writable properties if they are configurable. We missed that case when checking if a property is read-only. Fixes: https://github.com/nodejs/node/issues/47799 PR-URL: https://github.com/nodejs/node/pull/51602 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
16 lines
438 B
JavaScript
16 lines
438 B
JavaScript
'use strict';
|
|
// https://github.com/nodejs/node/issues/47799
|
|
|
|
require('../common');
|
|
const assert = require('assert');
|
|
const vm = require('vm');
|
|
|
|
const ctx = vm.createContext();
|
|
|
|
const window = vm.runInContext('this', ctx);
|
|
|
|
Object.defineProperty(window, 'x', { value: '1', configurable: true });
|
|
assert.strictEqual(window.x, '1');
|
|
Object.defineProperty(window, 'x', { value: '2', configurable: true });
|
|
assert.strictEqual(window.x, '2');
|