From e8244f7f0770d325690a2f2831d57ac73adf140e Mon Sep 17 00:00:00 2001 From: John-David Dalton Date: Wed, 9 Sep 2015 22:57:06 -0700 Subject: [PATCH] Adjust property descriptors in tests. --- test/test.js | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/test/test.js b/test/test.js index 544ef5047..2b0a92f97 100644 --- a/test/test.js +++ b/test/test.js @@ -195,7 +195,7 @@ /** Poison the free variable `root` in Node.js */ try { Object.defineProperty(global.root, 'root', { - 'configurable': true, + 'configurable': false, 'enumerable': false, 'get': function() { throw new ReferenceError; } }); @@ -5800,12 +5800,14 @@ pass = true; defineProperty(object, 'a', { + 'enumerable': true, + 'configurable': true, 'get': _.constant(value), 'set': function() { pass = false; } }); func(object, { 'a': value }); - assert.ok(pass, value); + assert.ok(pass); } else { skipTest(assert); @@ -16161,12 +16163,14 @@ pass = true; defineProperty(object, 'a', { + 'enumerable': true, + 'configurable': true, 'get': _.constant(value), 'set': function() { pass = false; } }); func(object, 'a', value); - assert.ok(pass, value); + assert.ok(pass); } else { skipTest(assert); @@ -19203,7 +19207,9 @@ if (!isStrict && defineProperty) { defineProperty(object, 'a', { 'configurable': false, - 'value': 1 + 'enumerable': true, + 'writable': true, + 'value': 1, }); assert.strictEqual(_.unset(object, 'a'), false); }