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

Improve test coverage #38

Closed
dgil opened this issue May 13, 2016 · 3 comments
Closed

Improve test coverage #38

dgil opened this issue May 13, 2016 · 3 comments
Labels
enhancement
Milestone
v1.0.0

Comments

@dgil
Copy link
Contributor

dgil commented May 13, 2016

The test we have now only test a few simple cases. We should improve test coverage.

Also, we could add in the main page a badge with the coverage.
@dgil dgil added this to the 1.0.0 milestone May 13, 2016
@dkapanidis
Copy link
Member

Let's split this to two different scopes. Add CI integration to captain and improve test coverage.

The CI integration is now on #50 issue.

@dgil
Copy link
Contributor Author

dgil commented May 20, 2016

Great!

@dkapanidis dkapanidis mentioned this issue May 20, 2016
Merged
@dkapanidis
Copy link
Member

Created an issue for adding coveralls integration #53

dkapanidis added a commit that referenced this issue May 24, 2016
@dkapanidis
Issue #38 Added tests for captain.Pre()
98c7758 
Execute function now returns errors instead of handling them internally. This improves testability.
@dkapanidis dkapanidis mentioned this issue May 24, 2016
Merged
@dkapanidis dkapanidis changed the title Improve test coverage Improve test coverage up to 60% May 24, 2016
@dkapanidis dkapanidis changed the title Improve test coverage up to 60% Improve test coverage May 24, 2016
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
v1.0.0
Development

No branches or pull requests
2 participants
@dkapanidis @dgil