From c86b633695299ed27053940d5ea5c5a2392964b3 Mon Sep 17 00:00:00 2001
From: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Date: Wed, 25 Oct 2023 14:46:49 +0200
Subject: [PATCH] Sort the benchmarks before listing them (#2026)
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
---
.../utils/frame/benchmarking-cli/src/pallet/command.rs | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/substrate/utils/frame/benchmarking-cli/src/pallet/command.rs b/substrate/utils/frame/benchmarking-cli/src/pallet/command.rs
index 84da3aaa02c..99f77866f8d 100644
--- a/substrate/utils/frame/benchmarking-cli/src/pallet/command.rs
+++ b/substrate/utils/frame/benchmarking-cli/src/pallet/command.rs
@@ -748,13 +748,17 @@ impl CliConfiguration for PalletCmd {
/// List the benchmarks available in the runtime, in a CSV friendly format.
fn list_benchmark(
- benchmarks_to_run: Vec<(
+ mut benchmarks_to_run: Vec<(
Vec<u8>,
Vec<u8>,
Vec<(BenchmarkParameter, u32, u32)>,
Vec<(String, String)>,
)>,
) {
+ // Sort and de-dub by pallet and function name.
+ benchmarks_to_run.sort_by(|(pa, sa, _, _), (pb, sb, _, _)| (pa, sa).cmp(&(pb, sb)));
+ benchmarks_to_run.dedup_by(|(pa, sa, _, _), (pb, sb, _, _)| (pa, sa) == (pb, sb));
+
println!("pallet, benchmark");
for (pallet, extrinsic, _, _) in benchmarks_to_run {
println!("{}, {}", String::from_utf8_lossy(&pallet), String::from_utf8_lossy(&extrinsic));
--
GitLab