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 unit tests #15

Closed
uniqueg opened this issue Sep 2, 2019 · 3 comments · Fixed by #150
Closed

Add unit tests #15

uniqueg opened this issue Sep 2, 2019 · 3 comments · Fixed by #150
Labels
flag: good 1st issue Good for newcomers priority: high High priority type: testing Related to code testing workload: days Likely takes days to resolve workload: hours Likely takes hours to resolve

Comments

@uniqueg
Copy link
Member

uniqueg commented Sep 2, 2019

Is your feature request related to a problem? Please describe.

Add extensive unit tests for the entire app.

Describe the solution you'd like

Write unit tests (preferentially using pytest) for every class/method/function, aiming for 100% code coverage.

Describe alternatives you've considered
N/A

@uniqueg uniqueg removed bh19 labels Feb 20, 2020
@uniqueg uniqueg changed the title Add tests Add unit tests Mar 8, 2023
@uniqueg uniqueg mentioned this issue Mar 8, 2023
@uniqueg uniqueg added flag: good 1st issue Good for newcomers priority: high High priority type: testing Related to code testing workload: days Likely takes days to resolve workload: hours Likely takes hours to resolve labels Mar 8, 2023
@uniqueg
Copy link
Member Author

uniqueg commented Mar 8, 2023

This can also be addressed iteratively, module by module, and can therefore be assigned to multiple people. In that case, open a separate issue for the module that you want to write tests for and link to this issue.

@SohamRatnaparkhi
Copy link
Member

Hi @uniqueg ,
I would love to work on this issue. I have prior experience using Pytest and am willing to learn new techniques as needed.
Thank you

@uniqueg
Copy link
Member Author

uniqueg commented Mar 9, 2023

Thanks @SohamRatnaparkhi. Given that there's a lot of code that needs to be tested, I would suggest that you pick one module and open an issue for that and link it here. We can then assign you to that issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flag: good 1st issue Good for newcomers priority: high High priority type: testing Related to code testing workload: days Likely takes days to resolve workload: hours Likely takes hours to resolve
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants