search

liftedinit / manifest-ledger

CosmosSDK-based blockchain ledger for the Manifest Network
Apache License 2.0
0 stars 2 forks source link

feat: binary coverage #45

Closed fmorency closed 2 months ago

fmorency commented 2 months ago

Retrieve coverage from instrumented manifestd binary.

This PR supports measuring the coverage of an instrumented manifestd binary.

The manifestd binary is used in interchaintest to perform various operations. This PR enables retrieving the manifestd coverage information from ephemeral containers.

The way of doing so is hackish for lack of a better word. I'm curious to know if there's a better way...

Also, interchaintest helpers function, e.g., AuthzExec, TokenFactoryMintDenom, etc, don't allow passing environment variables. I had to work around it.

I should probably decouple the interchaintest from the coverage feature at some point...

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 92.85714% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 59.20%. Comparing base (39214eb) to head (974cce3).

Files Patch % Lines
interchaintest/setup.go 92.00% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #45 +/- ## =========================================== + Coverage 44.06% 59.20% +15.13% =========================================== Files 34 34 Lines 2115 2140 +25 =========================================== + Hits 932 1267 +335 + Misses 1139 808 -331 - Partials 44 65 +21 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.