From 9dd9b3eb8d1478fb715acb9661c9bf825b3a1ad8 Mon Sep 17 00:00:00 2001 From: Hakan Seven Date: Thu, 1 Oct 2026 00:27:42 +0300 Subject: [PATCH] test(icons): run the themed-handle timing benchmark only on request It asserted a wall-clock race between two loops, which the parallel suite's load lost now and then. The cache's behaviour stays pinned by the entry-count tests. Co-Authored-By: Claude Opus 5.5 --- src/ui/icons.rs | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/ui/icons.rs b/src/ui/icons.rs index 9d1450987..7a2385fb4 100644 --- a/src/ui/icons.rs +++ b/src/ui/icons.rs @@ -1030,8 +1030,13 @@ mod themed_cache_tests { /// on a different static. /// /// Run with `--nocapture` to see the numbers: - /// `cargo test --lib themed_cache_speedup_over_uncached_handle_creation -- --nocapture`. + /// `cargo test --lib themed_cache_speedup_over_uncached_handle_creation -- --ignored --nocapture`. + /// + /// Ignored by default: it compares wall-clock times, and under the + /// parallel suite's load the uncached loop can finish first. The cache's + /// behaviour is pinned by the entry-count tests above. #[test] + #[ignore = "wall-clock benchmark; run with --ignored"] fn themed_cache_speedup_over_uncached_handle_creation() { use std::time::Instant;