From 32b05132b542a657911feb5f8477d17c84334502 Mon Sep 17 00:00:00 2001 From: "Jamil Lambert, PhD" Date: Tue, 11 Feb 2025 17:45:14 +0000 Subject: [PATCH] Reorganize mutants.toml Generalize `deserialize` as an exception for everywhere. Reword comment to make clearer --- .cargo/mutants.toml | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/.cargo/mutants.toml b/.cargo/mutants.toml index b6ed18c55..924747689 100644 --- a/.cargo/mutants.toml +++ b/.cargo/mutants.toml @@ -8,20 +8,14 @@ exclude_re = [ "impl Arbitrary", "impl Display", ".*Error", - # --------------------------------------------Crate-specific exclusions-------------------------------------------- + "deserialize", # Skip serde mutation tests + + # ----------------------------------Crate-specific exclusions---------------------------------- # Units # src/amount/mod.rs "parse_signed_to_satoshi", # Can't kill all mutants since there is no denomination smaller than Satoshi "fmt_satoshi_in", # Related to formatting/display "dec_width", # Replacing num /= 10 with num %=10 in a loop causes a timeout due to infinite loop - # src/fee_rate/serde.rs - "as_sat_per_kwu::opt::deserialize::.*", # Replaces return value with Ok(Default::default()), which is the same as Ok(None) - "as_sat_per_vb_floor::opt::deserialize::.*", # Replaces return value with Ok(Default::default()), which is the same as Ok(None) - "as_sat_per_vb_ceil::opt::deserialize::.*", # Replaces return value with Ok(Default::default()), which is the same as Ok(None) - # src/amount/serde.rs - "as_sat::opt::deserialize::>.*", # Replaces return value with Ok(Default::default()), which is the same as Ok(None) - "as_btc::opt::deserialize::>.*", # Replaces return value with Ok(Default::default()), which is the same as Ok(None) - "as_str::opt::deserialize::>.*", # Replaces return value with Ok(Default::default()), which is the same as Ok(None) # src/locktime/relative.rs - "Time::to_consensus_u32" # It will replace | with ^, which will return the same value since the XOR is always taken against the u16 and an all-zero bitmask + "Time::to_consensus_u32", # Mutant from replacing | with ^, this returns the same value since the XOR is taken against the u16 with an all-zero bitmask ]