armi icon indicating copy to clipboard operation
armi copied to clipboard

test_comparedb3.py has very few unit tests

Open opotowsky opened this issue 3 years ago • 1 comments

It has a few small unit tests, and two integration tests that don't touch all possible cases. Unit tests for all the other functions are needed.

This was discovered when the need for PR #786 was discovered.

opotowsky avatar Jul 20 '22 23:07 opotowsky

We have raised the code coverage from 60% to 83%:

https://coveralls.io/builds/51061594/source?filename=armi%2Fbookkeeping%2Fdb%2FcompareDB3.py

I think most of what is left are exceptions and errors. Which we have not done a lot of testing for yet in ARMI. Sadly.

john-science avatar Jul 21 '22 17:07 john-science