-
Notifications
You must be signed in to change notification settings - Fork 5
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
need tests #6
Comments
As part of #4,Travis CI was enabled. The library is being installed but not tested. |
orbeckst
added a commit
that referenced
this issue
Oct 11, 2018
- start #6 - tests for - _read_description() - _sha256() - _fetch_remote() - get_data_home()
Merged
@orbeckst wrt more tests, I was thinking we could mock the call that does the download. So stop it just before it downloads. Then mock the file exists check too etc |
Mock is probably the way to go. I will merge #16 as a "bonda fide" test (it ran pretty fast) and then we should look into mocking. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Need tests that run on travis.
Downloading data as part of the tests might be expensive for big datasets but we should do something.
The text was updated successfully, but these errors were encountered: