From abefb94daae656789143318170a604106618ead4 Mon Sep 17 00:00:00 2001 From: Duncan Tourolle Date: Fri, 2 Oct 2026 22:44:17 -0400 Subject: [PATCH] Give the export-ignores-the-viewport tests the viewport zoom now takes MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 0b06e31b ("Let a zoomed view fill the viewport...") added the viewport's size to DevelopSession::zoom_about and left this integration test calling it with three arguments, so dr-ui's tests did not compile. The photograph here is 64×64; a 64×64 viewport keeps the zoomed view the tests were written against. --- ui/dr-ui/tests/export_ignores_the_viewport.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ui/dr-ui/tests/export_ignores_the_viewport.rs b/ui/dr-ui/tests/export_ignores_the_viewport.rs index 14cac1c..f801989 100644 --- a/ui/dr-ui/tests/export_ignores_the_viewport.rs +++ b/ui/dr-ui/tests/export_ignores_the_viewport.rs @@ -75,7 +75,7 @@ fn an_export_is_the_same_frame_whether_or_not_the_canvas_is_zoomed() { let before = top_row_span(&unzoomed.rgba, unzoomed.width); // What a scroll wheel over the middle of the canvas does. - session.zoom_about(4.0, 0.5, 0.5); + session.zoom_about(4.0, 0.5, 0.5, 64, 64); assert!(session.is_zoomed(), "the session did not take the zoom"); let zoomed = session @@ -108,7 +108,7 @@ fn a_thumbnail_is_the_same_frame_whether_or_not_the_canvas_is_zoomed() { let (w, _, unzoomed) = session.render_thumbnail(32).expect("thumbnail"); let before = top_row_span(&unzoomed, w); - session.zoom_about(4.0, 0.5, 0.5); + session.zoom_about(4.0, 0.5, 0.5, 64, 64); let (_, _, zoomed) = session.render_thumbnail(32).expect("thumbnail"); let after = top_row_span(&zoomed, w);