From 1847cbd3e3b41d126f6de957cd20fe7c6eab1fa0 Mon Sep 17 00:00:00 2001 From: aiongg Date: Thu, 22 Oct 2026 03:28:46 +0000 Subject: [PATCH] Integration: #1997 test picks the 3:4 crop by its named ratio, which follows the photo's orientation --- crates/ui-egui/src/tests_crop_tool.rs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/crates/ui-egui/src/tests_crop_tool.rs b/crates/ui-egui/src/tests_crop_tool.rs index 75010d1f5..16e0bf658 100644 --- a/crates/ui-egui/src/tests_crop_tool.rs +++ b/crates/ui-egui/src/tests_crop_tool.rs @@ -75,7 +75,7 @@ fn x_in_the_crop_tool_swaps_the_aspect_once() { h.request("engine.execute", json!({"command": "library.select", "params": {"ids": [id.0], "active": id.0}}), T); h.app.ui.view = ViewMode::Detail; h.request("engine.execute", json!({"command": "panel.crop"}), T); - let r = h.request("engine.execute", json!({"command": "crop.aspect", "params": {"aspect": [3, 4]}}), T); + let r = h.request("engine.execute", json!({"command": "crop.aspect", "params": {"aspect": "3x4"}}), T); assert_eq!(r["ok"], true, "{r}"); h.settle(SETTLE); assert_eq!(h.app.ui.right, RightPanel::Crop); @@ -84,14 +84,14 @@ fn x_in_the_crop_tool_swaps_the_aspect_once() { let r = p.develop.crop.geometry.rect_px(p.width as f64, p.height as f64); (r.width() > r.height(), p.develop.crop.aspect) }; - assert_eq!(crop(&h), (true, Some((300, 400))), "a 3:4 crop follows the landscape photo"); + assert_eq!(crop(&h), (true, Some((400, 300))), "a 3:4 crop follows the landscape photo"); let undo = h.app.session.undo.len(); h.request("ui.key", json!({"key": "x"}), T); h.settle(SETTLE); - assert_eq!(crop(&h), (false, Some((400, 300))), "X turns the crop portrait"); + assert_eq!(crop(&h), (false, Some((300, 400))), "X turns the crop portrait"); assert_eq!(h.app.session.undo.len(), undo + 1, "one swap, one undo step"); assert_eq!(h.app.session.catalog.photo(id).unwrap().flag, lightcraft_catalog::Flag::None, "X doesn't reject while cropping"); h.request("ui.key", json!({"key": "x"}), T); h.settle(SETTLE); - assert_eq!(crop(&h), (true, Some((300, 400))), "X again turns it back"); + assert_eq!(crop(&h), (true, Some((400, 300))), "X again turns it back"); }