impl BenchmarkHelper for () (#997)

Author: zjb0807

oracle/src/benchmarking.rs

@@ -5,14 +5,13 @@ use frame_benchmarking::v2::*;
 
 use frame_support::assert_ok;
 use frame_system::{Pallet as System, RawOrigin};
-use sp_std::vec;
 
 #[instance_benchmarks]
 mod benchmarks {
 	use super::*;
 
 	#[benchmark]
-	fn feed_values(x: Linear<1, { T::BenchmarkHelper::get_currency_id_value_pairs().len() as u32 }>) {
+	fn feed_values(x: Linear<0, { T::BenchmarkHelper::get_currency_id_value_pairs().len() as u32 }>) {
 		// Register the caller
 		let caller: T::AccountId = whitelisted_caller();
 		T::Members::add(&caller);

oracle/src/lib.rs

@@ -63,6 +63,13 @@ pub trait BenchmarkHelper<OracleKey, OracleValue, L: Get<u32>> {
 	fn get_currency_id_value_pairs() -> BoundedVec<(OracleKey, OracleValue), L>;
 }
 
+#[cfg(feature = "runtime-benchmarks")]
+impl<OracleKey, OracleValue, L: Get<u32>> BenchmarkHelper<OracleKey, OracleValue, L> for () {
+	fn get_currency_id_value_pairs() -> BoundedVec<(OracleKey, OracleValue), L> {
+		BoundedVec::default()
+	}
+}
+
 #[frame_support::pallet]
 pub mod module {
 	use super::*;

oracle/src/mock.rs

@@ -63,15 +63,6 @@ impl SortedMembers<AccountId> for Members {
 	}
 }
 
-#[cfg(feature = "runtime-benchmarks")]
-pub struct BenchmarkHelper;
-#[cfg(feature = "runtime-benchmarks")]
-impl crate::BenchmarkHelper<Key, Value, MaxFeedValues> for BenchmarkHelper {
-	fn get_currency_id_value_pairs() -> BoundedVec<(Key, Value), MaxFeedValues> {
-		vec![(1, 1), (2, 2), (3, 3)].try_into().unwrap()
-	}
-}
-
 impl Config for Test {
 	type RuntimeEvent = RuntimeEvent;
 	type OnNewData = ();
@@ -85,7 +76,7 @@ impl Config for Test {
 	type MaxHasDispatchedSize = ConstU32<100>;
 	type MaxFeedValues = MaxFeedValues;
 	#[cfg(feature = "runtime-benchmarks")]
-	type BenchmarkHelper = BenchmarkHelper;
+	type BenchmarkHelper = ();
 }
 
 type Block = frame_system::mocking::MockBlock<Test>;