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

add go cover to the workflow #73

Closed hsubramanianaks closed 1 year ago

hsubramanianaks commented 1 year ago

Description

Include gocover reporting for unit test coverage

Fixes # (issue) Feature # (details)

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. Is it a breaking change which will impact consuming tool(s)?

Fails the build if coverage base line is not met, increased it to 91 to purposefully fail it. I will keep them as default 80%, it would fail if the changes made in the PR doesn't have at least 80%-unit tests covered.

image

Checklist:

OliverMKing commented 1 year ago

/ok-to-test sha=5fe8c9b

OliverMKing commented 1 year ago

/ok-to-test sha=5fe8c9b

hsubramanianaks commented 1 year ago

/ok-to-test sha=5fe8c9b

hsubramanianaks commented 1 year ago

Closing this PR as changes are made in different branch.