-
Notifications
You must be signed in to change notification settings - Fork 34
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
fix unit test failures #139
Comments
Current issues:
@Patashu I need your thoughts about what to do with these |
|
Would like to pick up some of these? I'm going to fix the infinity one at least |
next time I'm in the mood to code I'll do whatever you haven't done yet (optimistically it'll be sometime this week) |
I would suggest grabbing hyperbolic functions since I have no idea how they work %) |
I got them from slabdrill iirc lol but I can figure out a good replacement |
7:08 AM] Razenpok: Hey, Pata
I've ported Number-compatibility tests from C# version to check if Decimal operations produce same values as Number operations. You can check them on
tests
branch by runningnpm ci
and thennpm run test
[7:08 AM] Razenpok: It will run for a bit (takes several seconds on my PC)
7:08 AM] Razenpok: And there are a lot of test cases because combinatorial explosion
[7:09 AM] Razenpok: You can check all the source in tests folder
[7:09 AM] Razenpok: Currently, I observe a lot of "Unreachable code" errors and I'm not sure what to do with them
[7:09 AM] Razenpok: Either to disable these test cases or fix the bugs
The text was updated successfully, but these errors were encountered: