-
Notifications
You must be signed in to change notification settings - Fork 7
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
Unit testing - getting simstudy fully "covered" #131
Comments
That sounds great! I still want to use Hedgehog more but more for "integration"-type tests (checking the whole process instead of single units) so you can go ahead an add tests. As I would like to switch to testthat 3edition (#110) at some point (probably 2.0) I would recommend this video showing the differences and advantages of 3rd edition: https://www.youtube.com/watch?v=1ZrjWKcG1C If you want to use some of the new features I think there is some way to use the 3e features locally in one test. |
@assignUser Having taken my first crack at developing some unit tests for the additions to
genOrdCat
, I've gained an appreciation for what you have set up, and now I have a decent (if rudimentary) understanding of how to build these tests. I also took a look at the other sections of code that are not currently "covered" by tests, and was wondering if it makes sense for me to just start working through the code and writing basic tests. But before doing that, I was wondering if you had a larger vision there, related to hedgehog, and if I should hold off. I think hedgehog looks cool, but was wondering how that fits in to our plans.The text was updated successfully, but these errors were encountered: