From ced173e95ac5a5027cee63d9aa26d582f9763b90 Mon Sep 17 00:00:00 2001 From: Chuck Date: Sun, 12 Jul 2026 10:31:52 -0400 Subject: [PATCH] test: sync mock cache manager signature with CacheManager.get MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit test_circuit_breaker has been failing on main: plugin_health passes memory_ttl= to cache_manager.get(), and the conftest mock's signature was never updated — the same component/double drift class as the monitored_update bug (#392). Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_01FqzC1nzTWL4kaqgMaQZFam --- test/conftest.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/test/conftest.py b/test/conftest.py index 5c764b7b..d18a49dd 100644 --- a/test/conftest.py +++ b/test/conftest.py @@ -38,7 +38,11 @@ def mock_cache_manager(): mock._memory_cache_timestamps = {} mock.cache_dir = "/tmp/test_cache" - def mock_get(key: str, max_age: int = 300) -> Optional[Dict]: + def mock_get(key: str, max_age: Optional[int] = 300, + memory_ttl: Optional[int] = None) -> Optional[Dict]: + # Signature mirrors CacheManager.get — keep in sync or callers + # passing keyword args (health tracker, resource monitor) break + # only in tests, hiding real-API compatibility. return mock._memory_cache.get(key) def mock_set(key: str, data: Dict, ttl: Optional[int] = None) -> None: