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 #9

Open
andrewda opened this issue Sep 20, 2017 · 7 comments
Open

Add unit tests #9

andrewda opened this issue Sep 20, 2017 · 7 comments

Comments

@andrewda
Copy link
Member

No description provided.

@snowsignal
Copy link
Contributor

What would the tests look like, exactly?

@SUPERCILEX
Copy link
Contributor

@andrewda lol, what he said 😂

@andrewda
Copy link
Member Author

Making sure cv_utils.py functions work as intended, and maybe abstracting out some more methods and testing those. I basically want to make sure it works on the low-level with certain images.

@andrewda
Copy link
Member Author

Right now we test that the output is what we expect, which is good and all, but making sure the individual steps work would be even better so we know what breaks when it breaks.

@SUPERCILEX
Copy link
Contributor

Oh, K. Yeah, that SGTM.

@snowsignal
Copy link
Contributor

I think we should use the unittest module for testing. I've used it before, and it's very complete and surprisingly easy to use.

@andrewda
Copy link
Member Author

Yep. The problem right now is that the way this program is organized unit tests won't really work. We'll have to refactor quite a bit to make it all testable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants