From a2ab774eb4f3d77992246963e4b8e2183e90aa56 Mon Sep 17 00:00:00 2001 From: David Mann Date: Sun, 25 Oct 2026 22:59:53 +0100 Subject: [PATCH] Tests: the font-menu hover preview test lets the menu settle Opening the menu on the current family (#9197) paints rows whose font samples load over a few frames; on CI's font set that takes 6 steps, past egui_kittest's default of 4. Raise max_steps as other harnesses do. Co-Authored-By: Claude Opus 5.5 (1M context) --- crates/ui-egui/src/type_font_picker_tests.rs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/crates/ui-egui/src/type_font_picker_tests.rs b/crates/ui-egui/src/type_font_picker_tests.rs index 876992e9f7..02c745d3f0 100644 --- a/crates/ui-egui/src/type_font_picker_tests.rs +++ b/crates/ui-egui/src/type_font_picker_tests.rs @@ -531,7 +531,9 @@ fn options_bar_hover_previews_on_canvas_without_committing() { } } - let mut h = Harness::builder().with_size(vec2(900.0, 700.0)).build_eframe(|cc| { + // Opening the menu on the current family (#9197) paints rows that load their font samples + // over a few frames, more than the default four steps. + let mut h = Harness::builder().with_size(vec2(900.0, 700.0)).with_max_steps(64).build_eframe(|cc| { PhotocraftApp::setup_context(&cc.egui_ctx, crate::theme::ThemeKind::ALL[0]); PreviewApp { app: PhotocraftApp::new(photocraft_engine::Session::new(), Services::default()) } });