소스 검색

LibJS: Uncomment Number.isInteger() tests that now pass :^)

Linus Groh 4 년 전
부모
커밋
54e1180f61
1개의 변경된 파일3개의 추가작업 그리고 3개의 파일을 삭제
  1. 3 3
      Userland/Libraries/LibJS/Tests/builtins/Number/Number.isInteger.js

+ 3 - 3
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)).toBeTrue();
     expect(Number.isInteger(5.0)).toBeTrue();
     expect(Number.isInteger(5.0)).toBeTrue();
     expect(Number.isInteger(5 + 1 / 10000000000000000)).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(99999999999999999999999999999999999)).toBeTrue();
 
 
     expect(Number.isInteger(5 + 1 / 1000000000000000)).toBeFalse();
     expect(Number.isInteger(5 + 1 / 1000000000000000)).toBeFalse();