Remove deprecated constructors from fuzz target
What the developer wrote
Authored by Shing Him Ng
Remove deprecated constructors from fuzz target
These were causing the daily fuzz cron job to fail
What changed, and why it matters
This is a routine maintenance change to a fuzz-testing file. It removes two deprecated test helper functions from a list so that an automated daily fuzzing job stops failing. There is no change to production code, no user-facing behavior, and no security fix.
No security action needed. Treat as normal cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit edits fuzz/fuzz_targets/units/arbitrary_weight.rs, reducing the constructor list from [Weight::from_wu, Weight::from_witness_data_size, Weight::from_non_witness_data_size] to [Weight::from_wu]. The removed constructors are deprecated and were triggering failures in the daily fuzz CI cron job. The actual Weight implementation is untouched; only the fuzz target’s test coverage is narrowed.
Changed components
fuzz/fuzz_targets/units/arbitrary_weight.rsInspect captured patch +1 / −1
diff --git a/fuzz/fuzz_targets/units/arbitrary_weight.rs b/fuzz/fuzz_targets/units/arbitrary_weight.rs
index 96426075..b50c917f 100644
--- a/fuzz/fuzz_targets/units/arbitrary_weight.rs
+++ b/fuzz/fuzz_targets/units/arbitrary_weight.rs
@@ -34,7 +34,7 @@ fn do_test(data: &[u8]) {
// Constructors that return a Weight
for constructor in
- [Weight::from_wu, Weight::from_witness_data_size, Weight::from_non_witness_data_size]
+ [Weight::from_wu]
{
if let Ok(val) = u.arbitrary() {
constructor(val);
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.