-
Notifications
You must be signed in to change notification settings - Fork 461
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Basic tests for BigInt #1198
Basic tests for BigInt #1198
Conversation
Merging this will allow me to follow with more coverage in the same parts that are not complete yet. |
assert.sameValue(0 >= 0n, true); | ||
assert.sameValue(0n >= 0, true); | ||
assert.sameValue(1n >= Number.MAX_VALUE, false); | ||
assert.sameValue(Number.MIN_VALUE >= 0n, false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand why the expected value for this case should be false
(also applies to the other comparison operators tests with Number.MIN_VALUE
). Number.MIN_VALUE
is positive, non-zero number, so shouldn't this comparison return true
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, I missed this assertion when updating the assertions from the similar <=
file.
This is only a refactor of tests from #1148. I'm not trying neither willing to add coverage in this PR.