[es6] Partially implement Reflect.getOwnPropertyDescriptor.

test/mjsunit/harmony/reflect.js

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Flags: --harmony-reflect
 
 // TODO(neis): Test with proxies.
 
 
 
@@ skipping 308 matching lines @@
   assertThrows(function() { Reflect.enumerate(42); }, TypeError);
   assertThrows(function() { Reflect.enumerate(null); }, TypeError);
 })();
 
 
 // See reflect-enumerate*.js for further tests.
 
 
 
 ////////////////////////////////////////////////////////////////////////////////
+// Reflect.getOwnPropertyDescriptor
+
+
+(function testReflectGetOwnPropertyDescriptorArity() {
+  assertEquals(2, Reflect.getOwnPropertyDescriptor.length);
+})();
+
+
+(function testReflectGetOwnPropertyDescriptorOnNonObject() {
+  assertThrows(function() { Reflect.getOwnPropertyDescriptor(); }, TypeError);
+  assertThrows(function() { Reflect.getOwnPropertyDescriptor(42); },
+    TypeError);
+  assertThrows(function() { Reflect.getOwnPropertyDescriptor(null); },
+    TypeError);
+})();
+
+
+(function testReflectGetOwnPropertyDescriptorKeyConversion() {
+  var tgt = {bla: 42};

[Jakob Kummerow, 2015/10/26 13:58:26]

nit: s/tgt/target/

[Jakob Kummerow, 2015/10/29 18:06:02]

Forgot this?

[neis, 2015/10/30 08:41:30]

Done.

+  var a = { [Symbol.toPrimitive]: function() { return "bla" } };
+  var b = { [Symbol.toPrimitive]: function() { throw "gaga" } };
+  assertEquals(42, Reflect.getOwnPropertyDescriptor(tgt, a).value);
+  assertThrows(function() { Reflect.getOwnPropertyDescriptor(tgt, b); },
+    "gaga");
+})();
+
+
+// See reflect-get-own-property-descriptor.js for further tests.

[Jakob Kummerow, 2015/10/26 13:58:26]

Any particular reason why the tests are split across two files?

+
+
+
+////////////////////////////////////////////////////////////////////////////////
 // Reflect.preventExtensions
 
 
 (function testReflectPreventExtensionsArity() {
   assertEquals(1, Reflect.preventExtensions.length);
 })();
 
 
 (function testReflectPreventExtensionsOnNonObject() {
   assertThrows(function() { Reflect.preventExtensions(); }, TypeError);
   assertThrows(function() { Reflect.preventExtensions(42); }, TypeError);
   assertThrows(function() { Reflect.preventExtensions(null); }, TypeError);
 })();
 
 
 // See reflect-prevent-extensions.js for further tests.
 
 // TODO(neis): Need proxies to test the situation where
 // [[preventExtensions]] returns false.