Merge pull request #1302 from rwaldron/bigint-fixups

Bigint fixups

test/built-ins/BigInt/issafeinteger-true.js

+// Copyright (C) 2017 Robin Templeton. All rights reserved.
+// This code is governed by the BSD license found in the LICENSE file.
+
+/*---
+description: Throws a TypeError if BigInt is called with a new target
+esid: sec-bigint-constructor
+info: |
+  ...
+  3. If Type(prim) is Number, return ? NumberToBigInt(prim).
+  ...
+
+  NumberToBigInt ( number )
+
+  2. If IsSafeInteger(number) is false, throw a RangeError exception.
+
+features: [BigInt]
+---*/
+
+assert.sameValue(BigInt(Number.MAX_SAFE_INTEGER), 9007199254740991n);
+assert.sameValue(BigInt(-Number.MAX_SAFE_INTEGER), -9007199254740991n);

test/built-ins/BigInt/new-target-throws.js

@@ -6,7 +6,6 @@ description: Throws a TypeError if BigInt is called with a new target
 esid: sec-bigint-constructor
 info: |
   1. If NewTarget is not undefined, throw a TypeError exception.
-  2. Let prim be ? ToPrimitive(value, hint Number).
   ...
 features: [BigInt]
 ---*/
@@ -15,19 +14,8 @@ assert.throws(TypeError, function() {
   new BigInt();
 });
 
-assert.throws(TypeError, function() {
-  new BigInt(NaN);
-});
-
 assert.throws(TypeError, function() {
   new BigInt({
     valueOf: function() { throw new Test262Error("unreachable"); }
   });
 });
-
-for (let x of [NaN, Infinity, 0.5, 2**53]) {
-    assert.throws(RangeError, () => BigInt(x));
-    assert.throws(RangeError, () => BigInt(-x));
-}
-assert.sameValue(BigInt(9007199254740991), 9007199254740991n);
-assert.sameValue(BigInt(-9007199254740991), -9007199254740991n);