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 the automated test coverage to an acceptable value #7

Closed

6 of 8 tasks

cristim opened this issue Aug 19, 2016 · 3 comments

Closed

6 of 8 tasks

Increase the automated test coverage to an acceptable value #7

cristim opened this issue Aug 19, 2016 · 3 comments

Labels

Member

cristim commented Aug 19, 2016 •

edited by xlr-8

Loading

I'd like to see it achieve somewhere around 80%

Summary of files:

autoscaling (ref https://github.com/cristim/autospotting/issues/41)
connections (no ref)
instance (ref https://github.com/cristim/autospotting/issues/42)
launch_configuration (ref https://github.com/cristim/autospotting/issues/43)
main (ref https://github.com/cristim/autospotting/issues/272)
region (ref https://github.com/cristim/autospotting/issues/44)
spot_instance_request (ref https://github.com/cristim/autospotting/issues/45)
spot_price (no ref)

cristim added the Type: Epic label

xlr-8 mentioned this issue

Test autoscaling #89

Merged

Contributor

xlr-8 commented Jun 17, 2018

I guess we can close it now: https://coveralls.io/github/cristim/autospotting?branch=master

Except region.go & main.go the coverage of the other one has been reached, and overall coverage as well.

Member Author

cristim commented Jun 18, 2018

Thanks, I'll close this.

But I'd like to also increase the coverage on region.go and main.go at some point.

cristim closed this as completed

Contributor

xlr-8 commented Jun 18, 2018

Agreed, which is why I created a new one, and update this description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment