tyler.durden
/
test262
mirror of https://github.com/tc39/test262.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Bring back `verifyProp` param to `isWritable`

This commit is contained in:
Alexey Shvayka 2020-03-23 23:39:49 +02:00 committed by Rick Waldron
parent 3bf630cdda
commit 79146e5cf4
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
harness/propertyHelper.js
View File

@ -139,7 +139,7 @@ function isSameValue(a, b) {
return a === b;
}
function isWritable(obj, name, value) {
function isWritable(obj, name, verifyProp, value) {
var unlikelyValue = Array.isArray(obj) && name === "length" ?
Math.pow(2, 32) - 1 :
"unlikelyValue";
@ -156,7 +156,7 @@ function isWritable(obj, name, value) {
}
}
writeSucceeded = isSameValue(obj[name], newValue);
writeSucceeded = isSameValue(obj[verifyProp || name], newValue);
// Revert the change only if it was successful (in other cases, reverting
// is unnecessary and may trigger exceptions for certain property
@ -184,7 +184,7 @@ function verifyWritable(obj, name, verifyProp, value) {
assert(Object.getOwnPropertyDescriptor(obj, name).writable,
"Expected obj[" + String(name) + "] to have writable:true.");
}
if (!isWritable(obj, name, value)) {
if (!isWritable(obj, name, verifyProp, value)) {
$ERROR("Expected obj[" + String(name) + "] to be writable, but was not.");
}
}
@ -194,7 +194,7 @@ function verifyNotWritable(obj, name, verifyProp, value) {
assert(!Object.getOwnPropertyDescriptor(obj, name).writable,
"Expected obj[" + String(name) + "] to have writable:false.");
}
if (isWritable(obj, name, value)) {
if (isWritable(obj, name, verifyProp)) {
$ERROR("Expected obj[" + String(name) + "] NOT to be writable, but was.");
}
}