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

check that error conditions produce human-readable error messages #211

Open
grahamgower opened this issue Feb 22, 2021 · 0 comments
Open

Comments

@grahamgower
Copy link
Member

grahamgower commented Feb 22, 2021

This might be tough to test, but we should be able to check some things systematically. Like if there's a missing "end_time" in an epoch, we should check that the deme ID and epoch index are in the error message. We might also consider having a collection of erroneous input files (as already suggested for the spec repository: popsim-consortium/demes-spec#27), and creating a table of error messages for each input file, so we can manually inspect the error messages for any obvious improvements or regressions.

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

No branches or pull requests

1 participant