fix: off-by-one in block producer fees pallet benchmark (#1102)

Author: AmbientTea

toolkit/block-producer-fees/pallet/src/benchmarking.rs

@@ -24,7 +24,7 @@ mod benchmarks {
 	where
 		<T as crate::Config>::Moment: From<u64>,
 	{
-		let size = T::HistoricalChangesPerProducer::get();
+		let size = T::HistoricalChangesPerProducer::get() + 1;
 		// Pessimistic storage content for is full, because it requires additional removal from the vecdeque.
 		let data = (0..size)
 			.into_iter()

toolkit/block-producer-fees/pallet/src/weights.rs

@@ -1,10 +1,10 @@
 
 //! Autogenerated weights for pallet_block_producer_fees
 //!
-//! THIS FILE WAS AUTO-GENERATED USING THE SUBSTRATE BENCHMARK CLI VERSION 47.0.0
-//! DATE: 2025-05-05, STEPS: `5000`, REPEAT: `2000`, LOW RANGE: `[]`, HIGH RANGE: `[]`
+//! THIS FILE WAS AUTO-GENERATED USING THE SUBSTRATE BENCHMARK CLI VERSION 51.0.0
+//! DATE: 2025-11-18, STEPS: `5000`, REPEAT: `2000`, LOW RANGE: `[]`, HIGH RANGE: `[]`
 //! WORST CASE MAP SIZE: `1000000`
-//! HOSTNAME: `framework-13`, CPU: `AMD Ryzen 7 7840U w/ Radeon  780M Graphics`
+//! HOSTNAME: `Nikolaoss-MacBook-Pro.local`, CPU: `<UNKNOWN>`
 //! EXECUTION: , WASM-EXECUTION: Compiled, CHAIN: None, DB CACHE: 1024
 
 // Executed Command:
@@ -22,9 +22,9 @@
 // --repeat=2000
 // --wasm-execution=compiled
 // --output
-// toolkit/block-producer-fees/pallet/src/weights.rs
+// ./toolkit/block-producer-fees/pallet/src/weights.rs
 // --template
-// .maintain/frame-weight-template.hbs
+// ./.maintain/frame-weight-template.hbs
 
 #![cfg_attr(rustfmt, rustfmt_skip)]
 #![allow(unused_parens)]
@@ -48,10 +48,10 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> {
 	/// Proof: `Aura::CurrentSlot` (`max_values`: Some(1), `max_size`: Some(8), added: 503, mode: `MaxEncodedLen`)
 	fn set_fee() -> Weight {
 		// Proof Size summary in bytes:
-		//  Measured:  `1046`
-		//  Estimated: `4511`
-		// Minimum execution time: 11_562_000 picoseconds.
-		Weight::from_parts(12_423_000, 4511)
+		//  Measured:  `1056`
+		//  Estimated: `4521`
+		// Minimum execution time: 10_000_000 picoseconds.
+		Weight::from_parts(11_000_000, 4521)
 			.saturating_add(T::DbWeight::get().reads(2_u64))
 			.saturating_add(T::DbWeight::get().writes(1_u64))
 	}
@@ -65,10 +65,10 @@ impl WeightInfo for () {
 	/// Proof: `Aura::CurrentSlot` (`max_values`: Some(1), `max_size`: Some(8), added: 503, mode: `MaxEncodedLen`)
 	fn set_fee() -> Weight {
 		// Proof Size summary in bytes:
-		//  Measured:  `1046`
-		//  Estimated: `4511`
-		// Minimum execution time: 11_562_000 picoseconds.
-		Weight::from_parts(12_423_000, 4511)
+		//  Measured:  `1056`
+		//  Estimated: `4521`
+		// Minimum execution time: 10_000_000 picoseconds.
+		Weight::from_parts(11_000_000, 4521)
 			.saturating_add(RocksDbWeight::get().reads(2_u64))
 			.saturating_add(RocksDbWeight::get().writes(1_u64))
 	}