search

arcus-azure / arcus.security

Security for Azure development in a breeze.
https://security.arcus-azure.net/
MIT License
39 stars 12 forks source link

chore: use coverlet code coverage #396

Closed stijnmoreels closed 1 year ago

stijnmoreels commented 1 year ago

Use Coverlet code coverage as GitHub action.

Relates to https://github.com/arcus-azure/arcus/issues/264

netlify[bot] commented 1 year ago

Deploy Preview for arcus-security canceled.

Name Link
Latest commit 691dd83d94adbda61d5b5d3a456a11bc32b0ecc5
Latest deploy log https://app.netlify.com/sites/arcus-security/deploys/63c5238c6e6f2900080fbacd
stijnmoreels commented 1 year ago

Good job extending the test-suite based on the coverage results. (Although it could've been done in a separate PR).

Excluded the test functionality and the provider projects as this last one is mostly integration stuff towards other environments (i.e. Azure Key Vault, HashiCorp Vault...) and these are tested in integration tests.

stijnmoreels commented 1 year ago

Good job extending the test-suite based on the coverage results. (Although it could've been done in a separate PR).

Could be in a diff PR, but I added 80% as a threshold to be above, and wanted to be above that here. 👍