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

[WIP] Improve cli code coverage #64

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[WIP] Improve cli code coverage #64

wants to merge 2 commits into from

Conversation

thom4parisot
Copy link
Owner

cli code coverage is about 25% which makes it pretty weak to forthcoming changes.

Best would be to:

  • lib-ify it
  • test the surface of the cli (probably with the help of spawn and some stream inspectors)

@thom4parisot
Add code coverage and add additional tests
aa612cb
@thom4parisot
Add cli test file
d30d338 
The point here is to demonstrate that cli testing should be improved.
@thom4parisot thom4parisot added this to the v3.2 milestone Sep 22, 2016
@thom4parisot thom4parisot removed this from the v3.3 milestone Feb 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
feature needs-tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@thom4parisot