Merge bitcoin/bitcoin#32310: test: Run all benchmarks in the sanity check

fanquake · fanquake · commit 82d1e94838ee · 2025-04-23T10:16:05.000+01:00
faca46b test: Run all benchmarks in the sanity check (MarcoFalke) Pull request description: It is unclear why not all benchmarks are run, given that: * they only run as a sanity check (fastest version) * no one otherwise runs them, not even CI * issues have been missed due to this ACKs for top commit: l0rinc: ACK faca46b BrandonOdiwuor: Code Review ACK faca46b Tree-SHA512: 866f1ccff0313017dd313d5a218d7ee088b823601a129b9ed4c5819b0d57fd808d78e3ea28ca00714ae6b209df5312b7b9dea091b2b028821ff46b8ba263c48a
diff --git a/src/bench/CMakeLists.txt b/src/bench/CMakeLists.txt
@@ -79,8 +79,8 @@ if(ENABLE_WALLET)
   target_link_libraries(bench_bitcoin bitcoin_wallet)
 endif()
 
-add_test(NAME bench_sanity_check_high_priority
-  COMMAND bench_bitcoin -sanity-check -priority-level=high
+add_test(NAME bench_sanity_check
+  COMMAND bench_bitcoin -sanity-check
 )
 
 install_binary_component(bench_bitcoin)

Original file line number	Diff line number	Diff line change
`@@ -79,8 +79,8 @@ if(ENABLE_WALLET)`
`79`	`79`	`target_link_libraries(bench_bitcoin bitcoin_wallet)`
`80`	`80`	`endif()`
`81`	`81`
`82`		`-add_test(NAME bench_sanity_check_high_priority`
`83`		`- COMMAND bench_bitcoin -sanity-check -priority-level=high`
	`82`	`+add_test(NAME bench_sanity_check`
	`83`	`+ COMMAND bench_bitcoin -sanity-check`
`84`	`84`	`)`
`85`	`85`
`86`	`86`	`install_binary_component(bench_bitcoin)`