-
Notifications
You must be signed in to change notification settings - Fork 11
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
Mock CAVEclient for testing #259
Conversation
Note on the bugs and issues:
|
On reflection, I dont' like the implementation in the third bullet point. Will move that feature to part of the existing |
Added a parameter |
Adding a small submodule
caveclient.tools.testing
that makes it much easier to mock CAVEclients for testing.CAVEclientMock lets you specify which services you anticipate loading, can take a datastack or use a only global-only version, can select what versions of each service you want to test against, specify api version availability, set materialization versions, etc. In general, CAVEclient initialization options are mocked out.
CAVEclient conftest.py was altered to use this tooling and several small tests were added to use these features.
This caught one or two bugs/issues thanks to these new tests.