From 40d998a026a96056c7e82b511004424dfd72bea1 Mon Sep 17 00:00:00 2001 From: Will Miao <13051207myq@gmail.com> Date: Fri, 3 Oct 2025 20:59:47 +0800 Subject: [PATCH] fix(settings): use timezone-aware datetime for current timestamp fix(tests): normalize stored paths in library upsert test --- py/services/settings_manager.py | 4 ++-- tests/services/test_settings_manager.py | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/py/services/settings_manager.py b/py/services/settings_manager.py index be660c99..09ceefdf 100644 --- a/py/services/settings_manager.py +++ b/py/services/settings_manager.py @@ -2,7 +2,7 @@ import copy import json import os import logging -from datetime import datetime +from datetime import datetime, timezone from typing import Any, Dict, Iterable, List, Mapping, Optional from ..utils.settings_paths import ensure_settings_file @@ -143,7 +143,7 @@ class SettingsManager: self._save_settings() def _current_timestamp(self) -> str: - return datetime.utcnow().replace(microsecond=0).isoformat() + "Z" + return datetime.now(timezone.utc).replace(microsecond=0).isoformat() def _build_library_payload( self, diff --git a/tests/services/test_settings_manager.py b/tests/services/test_settings_manager.py index 56eef7b9..3101b938 100644 --- a/tests/services/test_settings_manager.py +++ b/tests/services/test_settings_manager.py @@ -111,7 +111,8 @@ def test_upsert_library_creates_entry_and_activates(manager, tmp_path): assert manager.get_active_library_name() == "studio" libraries = manager.get_libraries() stored_paths = libraries["studio"]["folder_paths"]["loras"] - assert str(lora_dir).replace(os.sep, "/") in stored_paths + normalized_stored_paths = [p.replace(os.sep, "/") for p in stored_paths] + assert str(lora_dir).replace(os.sep, "/") in normalized_stored_paths def test_delete_library_switches_active(manager, tmp_path):