-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add some more tests for various load models #959
Comments
@JacobDomagala Can you work on this issue next? I want to get some more testing in place for the load models. If you have questions about how they are supposed to work, query me or @PhilMiller. |
Note to self following could be useful: |
@lifflander @PhilMiller Let me know if you have any particular tests in mind for any of the load models. In the meantime i will study the code and docs to get more familiar with the topic and maybe will try to come with some useful tests aswell. |
…x ComposedModel::getNumCompletedPhases()
…x ComposedModel::getNumCompletedPhases()
…oad-models-part-2 959 Add some more tests for various load models (Part 2)
What Needs to be Done?
The
PerCollection
model is tested (along with couple other points in the model codepath), but mostly the load models lack any testing. Write unit tests for them.The text was updated successfully, but these errors were encountered: