From 54e1180f618442154b12a9b984549c644b3b3762 Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Sun, 11 Jul 2021 22:03:04 +0100 Subject: [PATCH] LibJS: Uncomment Number.isInteger() tests that now pass :^) --- .../LibJS/Tests/builtins/Number/Number.isInteger.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Userland/Libraries/LibJS/Tests/builtins/Number/Number.isInteger.js b/Userland/Libraries/LibJS/Tests/builtins/Number/Number.isInteger.js index ff0bc9370c..d9a646cc1c 100644 --- a/Userland/Libraries/LibJS/Tests/builtins/Number/Number.isInteger.js +++ b/Userland/Libraries/LibJS/Tests/builtins/Number/Number.isInteger.js @@ -7,9 +7,9 @@ test("basic functionality", () => { expect(Number.isInteger(5)).toBeTrue(); expect(Number.isInteger(5.0)).toBeTrue(); expect(Number.isInteger(5 + 1 / 10000000000000000)).toBeTrue(); - // FIXME: values outside of i32's range should still return true - // expect(Number.isInteger(+2147483647 + 1)).toBeTrue(); - // expect(Number.isInteger(-2147483648 - 1)).toBeTrue(); + expect(Number.isInteger(+2147483647 + 1)).toBeTrue(); + expect(Number.isInteger(-2147483648 - 1)).toBeTrue(); + // FIXME: This should return true // expect(Number.isInteger(99999999999999999999999999999999999)).toBeTrue(); expect(Number.isInteger(5 + 1 / 1000000000000000)).toBeFalse();