Exclude deprecated functions from mutation testing
What the developer wrote
Authored by Jamil Lambert, PhD
Exclude deprecated functions from mutation testing
There are new mutants from two deprecated functions in the units module.
Exclude them from mutation testing.
What changed, and why it matters
This commit only updates the project's mutation-testing configuration file. It tells the mutation-testing tool to skip two more deprecated functions when looking for untested code changes. There is no change to the actual Bitcoin library code, no bug fix, and no security relevance.
No security action needed. This is a benign configuration-only change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff adds two regular-expression exclude rules to .cargo/mutants.toml so that cargo-mutants will not generate mutants for NumberOfBlocks::to_consensus_u32 and NumberOf512Seconds::value, both marked as deprecated. This is purely a tooling/CI configuration change.
Changed components
.cargo/mutants.tomlInspect captured patch +2 / −0
diff --git a/.cargo/mutants.toml b/.cargo/mutants.toml
index 005039b0..455bc8e7 100644
--- a/.cargo/mutants.toml
+++ b/.cargo/mutants.toml
@@ -26,8 +26,10 @@ exclude_re = [
"units/.* FeeRate::fee_vb", # Deprecated
"units/.* FeeRate::fee_wu", # Deprecated
"units/.* SignedAmount::checked_abs", # Deprecated
+ "units/.* NumberOfBlocks::to_consensus_u32", # Deprecated
"units/.* NumberOfBlocks::value", # Deprecated
"units/.* NumberOf512Seconds::to_consensus_u32", # Deprecated
+ "units/.* NumberOf512Seconds::value", # Deprecated
"units/.* MedianTimePast::to_consensus_u32", # Deprecated
"units/.* Height::to_consensus_u32", # Deprecated
"units/.* Sequence::to_hex", # Deprecated
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.