Skip to content

Commit

Permalink
fix: exclude dalek from source test coverage
Browse files Browse the repository at this point in the history
For some reason (non-std cargo importing maybe?), the dalek crate was
being included in the test coverage. This change explicitly filters it
sound to get a better test coverage metric for this crate.
  • Loading branch information
CjS77 committed Dec 14, 2022
1 parent 2f063be commit 7cae60b
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion test_coverage.sh
Expand Up @@ -40,6 +40,7 @@ cargo +nightly cov -- \
--show-region-summary \
--ignore-filename-regex='/.cargo/registry' \
--ignore-filename-regex="^/rustc" \
--ignore-filename-regex="curve25519-dalek" \
--instr-profile=cov_raw/bulletproofs-plus.profdata \
$files \
> cov_raw/bulletproofs-plus.lcov
Expand All @@ -52,7 +53,8 @@ cargo +nightly cov -- \
--show-region-summary \
--ignore-filename-regex='/.cargo/registry' \
--ignore-filename-regex="^/rustc" \
--instr-profile=cov_raw/bulletproofs-plus.profdata \
--ignore-filename-regex="curve25519-dalek" \
--instr-profile=cov_raw/bulletproofs-plus.profdata \
$files \
> cov_raw/bulletproofs-plus.txt

Expand Down

0 comments on commit 7cae60b

Please sign in to comment.