What the developer wrote
Authored by Jamil Lambert, PhD
Add mutation exclude for timeouts
Two functions in WitnessEncoder cause infinite loops when mutated.
Add excludes for them.
What changed, and why it matters
This commit only updates a mutation-testing configuration file. It tells the automated mutation-testing tool not to alter two specific functions because those alterations would make tests run forever. It does not change any actual Bitcoin library code, fix a runtime bug, or address a security vulnerability.
No security action needed. Treat as routine test-configuration maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff adds two exclude_re entries to .cargo/mutants.toml for <impl Encoder for WitnessEncoder<’a>>::current_chunk and advance. The comments explain that certain mutations (returning Some(vec![]) or true respectively) would cause infinite loops during mutation testing. This is a test-infrastructure-only change with no effect on the shipped library code or its behavior.
Changed components
.cargo/mutants.tomlInspect captured patch +2 / −0
diff --git a/.cargo/mutants.toml b/.cargo/mutants.toml
index 74680b18..7dfd9912 100644
--- a/.cargo/mutants.toml
+++ b/.cargo/mutants.toml
@@ -46,4 +46,6 @@ exclude_re = [
"Script<T>::to_hex", # Deprecated
"ScriptBuf::to_hex", # Deprecated
"ScriptBuf<T>::to_hex", # Deprecated
+ "<impl Encoder for WitnessEncoder<'a>>::current_chunk", # Replacing the return with Some(vec![]) causes an infinite loop.
+ "<impl Encoder for WitnessEncoder<'a>>::advance", # Replacing the return with true causes an infinite loop.
]
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.