From 482abb45626999efdf52d63aa5f6b8d6e64a4edb Mon Sep 17 00:00:00 2001
From: Mateusz Sych <151755914+SychPL@users.noreply.github.com>
Date: Wed, 14 Oct 2026 07:26:51 +0200
Subject: [PATCH] test(mcp): compare the scratch folder by its canonical path
 (#965)

PathScope stores its roots canonicalised (\\?\ on Windows, /private/var on macOS), so the default-folders test now canonicalises scratch_dir() before looking it up, like the recovery folder on the line above.

The same fix came in from @Karl-VonRichter in #969, found on macOS.

Co-authored-by: Karl-VonRichter <20079846+Karl-VonRichter@users.noreply.github.com>
---
 crates/mcp/src/tests.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/crates/mcp/src/tests.rs b/crates/mcp/src/tests.rs
index b205f294..71179266 100644
--- a/crates/mcp/src/tests.rs
+++ b/crates/mcp/src/tests.rs
@@ -473,7 +473,7 @@ fn the_default_folders_are_the_designs_the_recovery_folder_and_the_scratch_folde
     let cwd = std::fs::canonicalize(std::env::current_dir().unwrap()).unwrap();
     assert!(!scope.roots().contains(&cwd), "the working folder is not a default folder: {scope:?}");
     assert!(scope.roots().contains(&std::fs::canonicalize(&recovery).unwrap()), "{scope:?}");
-    assert!(scope.roots().contains(&crate::scratch_dir().unwrap()), "{scope:?}");
+    assert!(scope.roots().contains(&std::fs::canonicalize(crate::scratch_dir().unwrap()).unwrap()), "{scope:?}");
 
     let mut session = solidcraft_engine::Session::default();
     session.execute("solid.box", &json!({"length": 10, "width": 10, "height": 10})).unwrap();