-
Notifications
You must be signed in to change notification settings - Fork 1
Setup Initial Tests #87
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
Conversation
|
Pytest is 3.10+
Setup a smoke test for python 3.9 below |
|
I think this is good for now. running the help for each command should catch some things. We can expand as we go. I don't think it should be tests against a CDA/database build at this time at least. if it expands we might need to do something like that. |
This sets up the framework to run recursive tests for the help pages of the CLI commands.
There are a few decisions we can make on how we handle these tests. Tossing some decisions to see if you agree @Enovotny?
cwms-cli blob downloadThis would ensure no changes of the outputs expected and break downstream processes.
For example, only run the
usgstests if changes to theusgsdir/src are in the PR. etc.Note i'm targeting a dev python version to run the tests of 3.10, but the library itself (cwms-cli) is targeting 3.9