From b49698b3e5fc2e61507be6e4e8982448adc6002a Mon Sep 17 00:00:00 2001 From: claire <28279548+claiwe@users.noreply.github.com> Date: Sat, 21 Feb 2026 03:17:13 -0600 Subject: [PATCH] visual_tests: Fix MultiWorkspace panics (#49784) After commit ee3f40f, the settings-related visual tests started panicking. The settings UI code depends on MultiWorkspaces, so changing these tests appropriately fixes the panics. Closes #49659 Release Notes: - N/A --- crates/zed/src/visual_test_runner.rs | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/crates/zed/src/visual_test_runner.rs b/crates/zed/src/visual_test_runner.rs index 89e3e5d3b05010931ba78d263dae543519f9b77a..6b1aca5939bf9a7874dd7a590d64a133f96a9dea 100644 --- a/crates/zed/src/visual_test_runner.rs +++ b/crates/zed/src/visual_test_runner.rs @@ -1288,7 +1288,7 @@ fn run_settings_ui_subpage_visual_tests( ) }); - let workspace_window: WindowHandle = cx + let workspace_window: WindowHandle = cx .update(|cx| { cx.open_window( WindowOptions { @@ -1298,9 +1298,10 @@ fn run_settings_ui_subpage_visual_tests( ..Default::default() }, |window, cx| { - cx.new(|cx| { + let workspace = cx.new(|cx| { Workspace::new(None, project.clone(), app_state.clone(), window, cx) - }) + }); + cx.new(|cx| MultiWorkspace::new(workspace, window, cx)) }, ) }) @@ -2343,7 +2344,7 @@ fn run_tool_permissions_visual_tests( ) }); - let workspace_window: WindowHandle = cx + let workspace_window: WindowHandle = cx .update(|cx| { cx.open_window( WindowOptions { @@ -2353,9 +2354,10 @@ fn run_tool_permissions_visual_tests( ..Default::default() }, |window, cx| { - cx.new(|cx| { + let workspace = cx.new(|cx| { Workspace::new(None, project.clone(), app_state.clone(), window, cx) - }) + }); + cx.new(|cx| MultiWorkspace::new(workspace, window, cx)) }, ) })