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

Add some more tests for various load models #959

Closed
lifflander opened this issue Jul 30, 2020 · 5 comments · Fixed by #1012
Closed

Add some more tests for various load models #959

lifflander opened this issue Jul 30, 2020 · 5 comments · Fixed by #1012
Assignees
Labels
intro A potentially good introductory issue for a new developer to address type: task

Comments

@lifflander
Copy link
Collaborator

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.

@lifflander lifflander added type: task intro A potentially good introductory issue for a new developer to address labels Jul 30, 2020
@lifflander
Copy link
Collaborator Author

@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.

@JacobDomagala
Copy link
Contributor

JacobDomagala commented Aug 18, 2020

@JacobDomagala
Copy link
Contributor

@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.

JacobDomagala added a commit that referenced this issue Aug 24, 2020
JacobDomagala added a commit that referenced this issue Aug 25, 2020
JacobDomagala added a commit that referenced this issue Aug 25, 2020
JacobDomagala added a commit that referenced this issue Aug 25, 2020
JacobDomagala added a commit that referenced this issue Aug 25, 2020
JacobDomagala added a commit that referenced this issue Aug 25, 2020
JacobDomagala added a commit that referenced this issue Aug 25, 2020
JacobDomagala added a commit that referenced this issue Aug 25, 2020
JacobDomagala added a commit that referenced this issue Aug 25, 2020
JacobDomagala added a commit that referenced this issue Aug 25, 2020
JacobDomagala added a commit that referenced this issue Sep 2, 2020
JacobDomagala added a commit that referenced this issue Sep 9, 2020
JacobDomagala added a commit that referenced this issue Sep 10, 2020
JacobDomagala added a commit that referenced this issue Sep 11, 2020
lifflander added a commit that referenced this issue Sep 11, 2020
…oad-models-part-2

959 Add some more tests for various load models (Part 2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
intro A potentially good introductory issue for a new developer to address type: task
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants