-
Notifications
You must be signed in to change notification settings - Fork 1
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
Comments
What would the tests look like, exactly? |
@andrewda lol, what he said 😂 |
Making sure |
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. |
Oh, K. Yeah, that SGTM. |
I think we should use the |
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. |
No description provided.
The text was updated successfully, but these errors were encountered: