Skip to content
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

Tests are lacking #5

Open
hlapp opened this issue Feb 5, 2016 · 2 comments
Open

Tests are lacking #5

hlapp opened this issue Feb 5, 2016 · 2 comments
Labels

Comments

@hlapp
Copy link
Member

hlapp commented Feb 5, 2016

Tests would seem to be particularly important for this repo, because for one, the structure of the axiom sets expected to come out of the translations are not particularly well defined or documented, yet will significantly influence the reasoning possible within the KB. So at the very least some basic assumptions about what axioms should (or should not) be there should be tested.

In addition to that, functioning tests can also greatly help with stepping through the code for debugging from a start and end point that are known to be good.

@hlapp hlapp added the bug label Feb 5, 2016
@hlapp
Copy link
Member Author

hlapp commented Feb 5, 2016

@balhoff can you write at least one or a few tests for an ingestor known to work, so we have a starting point?

@balhoff
Copy link
Member

balhoff commented Feb 5, 2016

@hlapp yes, good idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants