From 7757f72859343472cf5ac48b118c97f06d73798b Mon Sep 17 00:00:00 2001 From: Will Miao <13051207myq@gmail.com> Date: Sun, 5 Oct 2025 22:40:43 +0800 Subject: [PATCH] Enhance test for saving paths to ensure cross-platform compatibility in folder paths --- tests/config/test_config_save_paths.py | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/tests/config/test_config_save_paths.py b/tests/config/test_config_save_paths.py index 1f3a59b0..f0756e36 100644 --- a/tests/config/test_config_save_paths.py +++ b/tests/config/test_config_save_paths.py @@ -76,10 +76,17 @@ def test_save_paths_renames_default_library(monkeypatch: pytest.MonkeyPatch, tmp name, payload = fake_settings.upsert_calls[0] assert name == "comfyui" - assert payload["folder_paths"] == folder_paths - assert payload["default_lora_root"] == folder_paths["loras"][0] - assert payload["default_checkpoint_root"] == folder_paths["checkpoints"][0] - assert payload["default_embedding_root"] == folder_paths["embeddings"][0] + + # The Config class normalizes paths to use forward slashes for cross-platform compatibility + # Convert expected paths to the same format for comparison + expected_folder_paths = { + key: [path.replace("\\", "/") for path in paths] + for key, paths in folder_paths.items() + } + assert payload["folder_paths"] == expected_folder_paths + assert payload["default_lora_root"] == folder_paths["loras"][0].replace("\\", "/") + assert payload["default_checkpoint_root"] == folder_paths["checkpoints"][0].replace("\\", "/") + assert payload["default_embedding_root"] == folder_paths["embeddings"][0].replace("\\", "/") assert payload["metadata"] == {"display_name": "ComfyUI", "source": "comfyui"} assert payload["activate"] is True