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

Add tests for Engine and CharDataFiller classes #27

Open
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@paultcochrane
Contributor

paultcochrane commented Jan 2, 2016

These changes add tests of the Engine and CharDataFiller classes. Their addition improves the code coverage; further testing of e.g. the Engine class, although possibly beneficial, would likely require a large amount of work to bring the coverage up only a small amount. The statement coverage of the entire distribution is now 88%, and this is hopefully high enough to be reasonably sure that future code changes will be checked by the test suite.

paultcochrane added some commits Jan 2, 2016

Fix docstring for hexary
... after having worked out from the test, that the description didn't match
what the code was actually doing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment