Index: test/mjsunit/object-define-property.js |
diff --git a/test/mjsunit/object-define-property.js b/test/mjsunit/object-define-property.js |
index b258aa75bf375e279e207746c383168d5ba62dd6..57d79b595f7b1171bdcaea7902e220b7d990ecd5 100644 |
--- a/test/mjsunit/object-define-property.js |
+++ b/test/mjsunit/object-define-property.js |
@@ -866,4 +866,15 @@ assertFalse(desc.writable); |
assertFalse(desc.enumerable); |
assertFalse(desc.configurable); |
+// See issue 968: http://code.google.com/p/v8/issues/detail?id=968 |
+var o = { x : 42 }; |
+Object.defineProperty(o, "x", { writable: false }); |
+assertEquals(42, o.x); |
+ |
+o = { x : 42 }; |
+Object.defineProperty(o, "x", {}); |
+assertEquals(42, o.x); |
+o.x = 37; |
+// Writability is preserved. |
+assertEquals(37, o.x); |
Rico
2010/12/10 10:42:35
Maybe add an additional test that makes sure that
Lasse Reichstein
2010/12/10 11:21:45
More tests added, for both non-existing property,
|