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.

Sign up for GitHub

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

Jump to bottom

Write tests #44

Closed
vikstrous opened this issue Mar 26, 2015 · 5 comments
Closed

Write tests #44

vikstrous opened this issue Mar 26, 2015 · 5 comments
Labels
enhancement

Comments

@vikstrous
Copy link
Owner

This project is getting big enough that it's non-trivial to verify that all of its options work. We need write tests that mock out the network and test all the different command line options.
@vikstrous
Copy link
Owner Author

I took the first step. We have CI set up and the first example test. We need to decouple things more in order to make them testable.

@vikstrous
Copy link
Owner Author

We also have coverage reports under artifacts in circle ci

@vikstrous vikstrous mentioned this issue Sep 3, 2015
Open
6 tasks
@vikstrous
Copy link
Owner Author

I think the highest priority for this project right now is to get near 100% code coverage and refactor things so that we can expand and add more features.

@vikstrous
Copy link
Owner Author

35% code coverage so far.

@vikstrous vikstrous mentioned this issue Sep 20, 2015
Open
@vikstrous
Copy link
Owner Author

I think the code coverage is high enough to enable refactoring with some degree of confidence. I'm closing this for now.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vikstrous