Update as suggested

substrate/frame/scheduler/src/benchmarking.rs

@@ -297,7 +297,7 @@ mod benchmarks {
 		#[extrinsic_call]
 		_(RawOrigin::Root, when, periodic, priority, call);
 
-		assert_eq!(Agenda::<T>::get(when).len(), s as usize + 1, "didn't add to schedule");
+		ensure!(Agenda::<T>::get(when).len() == s as usize + 1, "didn't add to schedule");
 
 		Ok(())
 	}
@@ -314,16 +314,16 @@ mod benchmarks {
 		#[extrinsic_call]
 		_(schedule_origin as SystemOrigin<T>, when, 0);
 
-		assert!(
+		ensure!(
 			s == 1 || Lookup::<T>::get(u32_to_name(0)).is_none(),
 			"didn't remove from lookup if more than 1 task scheduled for `when`"
 		);
 		// Removed schedule is NONE
-		assert!(
+		ensure!(
 			s == 1 || Agenda::<T>::get(when)[0].is_none(),
 			"didn't remove from schedule if more than 1 task scheduled for `when`"
 		);
-		assert!(
+		ensure!(
 			s > 1 || Agenda::<T>::get(when).len() == 0,
 			"remove from schedule if only 1 task scheduled for `when`"
 		);
@@ -347,7 +347,7 @@ mod benchmarks {
 		#[extrinsic_call]
 		_(RawOrigin::Root, id, when, periodic, priority, call);
 
-		assert_eq!(Agenda::<T>::get(when).len(), s as usize + 1, "didn't add to schedule");
+		ensure!(Agenda::<T>::get(when).len() == s as usize + 1, "didn't add to schedule");
 
 		Ok(())
 	}
@@ -358,21 +358,21 @@ mod benchmarks {
 	) -> Result<(), BenchmarkError> {
 		let when = BLOCK_NUMBER.into();
 
-		fill_schedule::<T>(when, s).unwrap();
+		fill_schedule::<T>(when, s)?;
 
 		#[extrinsic_call]
 		_(RawOrigin::Root, u32_to_name(0));
 
-		assert!(
+		ensure!(
 			s == 1 || Lookup::<T>::get(u32_to_name(0)).is_none(),
 			"didn't remove from lookup if more than 1 task scheduled for `when`"
 		);
 		// Removed schedule is NONE
-		assert!(
+		ensure!(
 			s == 1 || Agenda::<T>::get(when)[0].is_none(),
 			"didn't remove from schedule if more than 1 task scheduled for `when`"
 		);
-		assert!(
+		ensure!(
 			s > 1 || Agenda::<T>::get(when).len() == 0,
 			"remove from schedule if only 1 task scheduled for `when`"
 		);