Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

Add unit tests #808

Closed
CecileRobertMichon opened this issue Apr 2, 2018 · 6 comments
Closed

Add unit tests #808

CecileRobertMichon opened this issue Apr 2, 2018 · 6 comments
Labels

Comments

@CecileRobertMichon
Copy link
Contributor

Currently coverage is 38%. We should add unit tests as much as possible and get closer to 100%. Good places to start:

@CecileRobertMichon
Copy link
Contributor Author

Update: coverage is now 47% (https://codecov.io/gh/Azure/acs-engine). Codecov integration was added in #2719 to help us track coverage in PRs.

@CecileRobertMichon
Copy link
Contributor Author

Coverage is now 53%

@mboersma mboersma transferred this issue from Azure/acs-engine Mar 20, 2019
@mboersma
Copy link
Member

Coverage is now 68%.

@tariq1890
Copy link
Contributor

There is no such thing as "too many unit tests".

@stale
Copy link

stale bot commented May 20, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label May 20, 2019
@stale
Copy link

stale bot commented Jul 19, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jul 19, 2019
@stale stale bot closed this as completed Jul 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants