Azure / aks-app-routing-operator

Kubernetes operator that implements AKS Application Routing

https://learn.microsoft.com/en-us/azure/aks/app-routing

MIT License

30 stars 20 forks source link

Bump test coverage up to at least 80% where possible #82

Closed NickKeller closed 1 year ago

NickKeller commented 1 year ago

Description

Bump test coverage to at least 80% where possible

Fixes # (issue) Feature # (details)

Type of change

Please delete options that are not relevant.

[ ] Test Coverage increase

How Has This Been Tested?

[ ] Added unit tests

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 5904359165

0 of 54 (0.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 86.035%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
pkg/controller/testutils/fakemanager.go	0	54	0.0%
<!--	Total:	0	54	0.0%	-->

Totals
Change from base Build 5871209609:	-0.02%
Covered Lines:	1842
Relevant Lines:	2141

💛 - Coveralls

OliverMKing commented 1 year ago

/ok-to-test sha=e21d3a