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

Increase code coverage to at least 80% #82

Open
Ellzord opened this issue May 10, 2015 · 8 comments
Open

Increase code coverage to at least 80% #82

Ellzord opened this issue May 10, 2015 · 8 comments
Labels
enhancement help wanted important testing

Comments

@Ellzord
Copy link
Owner

Ellzord commented May 10, 2015

Currently at 8.82% https://coveralls.io/builds/2526982 #25
@Ellzord Ellzord added this to the Utility release milestone May 10, 2015
@pbremer
Copy link
Contributor

pbremer commented May 10, 2015

It should also be noted that if anyone makes any new claases/methods that they should also be effectively unit tested. If anyone does not know how to use Junit or what an effective unit test is just message me

@Ellzord
Copy link
Owner Author

Ellzord commented May 10, 2015

Yup as much as possible. As far as test quality goes I'll settle for low right now as we have such little coverage. When bugs come in I think it should be part of the process to ensure there was a test that would of picked it up and that passes too (little ambitious but useful).

@Ellzord
Copy link
Owner Author

Ellzord commented May 13, 2015

@wlmiller
COVERAGE INCREASED (+36.02%) TO 44.84%
https://coveralls.io/builds/2555690

Nice one!

@wlmiller
Copy link
Contributor

👍

@wlmiller wlmiller mentioned this issue May 14, 2015
Merged
@Ellzord
Copy link
Owner Author

Ellzord commented May 14, 2015

@wlmiller
COVERAGE INCREASED (+4.17%) TO 49.01%
https://coveralls.io/builds/2566611

~50% coverage is not too bad at all!

@wlmiller wlmiller mentioned this issue May 16, 2015
Merged
@Ellzord
Copy link
Owner Author

Ellzord commented May 16, 2015

@wlmiller
COVERAGE INCREASED (+11.15%) TO 60.16%
https://coveralls.io/builds/2580017

@Ellzord Ellzord mentioned this issue May 18, 2015
Closed
@Ellzord
Copy link
Owner Author

Ellzord commented May 24, 2015

COVERAGE DECREASED (-0.11%) TO 70.06%
https://coveralls.io/builds/2639372

@Ellzord Ellzord assigned Ellzord and unassigned Ellzord Jun 29, 2015
@Ellzord
Copy link
Owner Author

Ellzord commented Jul 9, 2015

@dting
COVERAGE INCREASED (+0.69%) TO 70.15%
https://coveralls.io/builds/3014767

@Ellzord Ellzord removed this from the Utility release milestone Aug 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement help wanted important testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Ellzord @wlmiller @pbremer