-
Notifications
You must be signed in to change notification settings - Fork 276
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
Missing test coverage #58
Comments
Can I be assigned this? |
Yes! Sadly I can't actually assign people to issues on GitHub, but consider it yours! Thank you 😸 |
Ok. |
FYI @nkprince007, @arbylee wrote tests for let me know if you have any questions! |
Thanks a lot, guys, but I'm a little caught up ATM, definitely like to contribute, soon as I find time. |
Sounds good @nkprince007! If you don't think you can get to this issue for a few weeks and don't have strong feeling towards it, we can un-assign you and when you have more time you take on whatever's free. If you're exciting about this issue though, we can keep it until you have the time :) let me know! |
A great first PR is to pick something on this list and write tests for it!
Symbols.js
solveEquation/EquationOperations.js
CombineChecks.test.js
separately test the functionsCombineChecks.canMultiplyLikeTermPolynomialNodes
,CombineChecks.canAddLikeTermPolynomialNodes
, andCombineChecks.canRearrangeCoefficient
feel free to point out more missing tests below!
The text was updated successfully, but these errors were encountered: