-
Notifications
You must be signed in to change notification settings - Fork 38
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
Added unit test manager #12
Conversation
fd80bed
to
bd4f318
Compare
Current progress: Have migrated all of Now we just have to do the rest (ugh) |
Also ran into a bad test with geometry:
CalcPressure doesn't exist. Additionally,
This is apparently True I'm currently marking all failing tests with "BAD TEST" and commenting them out since I don't know if their behavior changes were intended. |
Author class was missing the constructor for only being given a last name; I added it. |
|
moved to another issue / downgraded in priority if someone tries to run these and they explode, poke me |
Hrrrm... maybe I'm not understanding what it means by "Line Coverage" vs. "Function Coverage". How can you have 100% Line Coverage, but less than 100% Function Coverage? More generally, some of those look better covered than I would have ever expected. ;-) |
Looks like that's where the coverage metrics run afoul of the One thing to keep track of is that though the code is covered it doesn't But that's true of nearly all testing setups. On Thu, Nov 5, 2015 at 7:19 PM Charles Ofria [email protected]
|
Nope, nevermind |
Alright, current difficulty: finding a good way to see if test coverage has dropped/changed/code was added that's not covered. That might be an issue I just table for now and hammer on later. |
@mercere99 Ready for merge at your leisure |
(the 12k deleted files are from where I forgot to not include the doc/build directory in commits and I deleted them, if you're wondering. gitignore and make clean have been updated accordingly) |
Added unit test manager
TODO:
add diff-cover maketarget to loudly complain about differences in coverageSquash all the thingsNot ready for merge.Technically depends on #10
Addresses #1