From e7bb1e9c54bf0bc98182f712519f2d73796a3505 Mon Sep 17 00:00:00 2001 From: greatmengqi Date: Thu, 16 Apr 2026 11:22:00 +0800 Subject: [PATCH] fix(test): enable memory config in test_memory_queue_user_isolation The queue.add() method early-returns when config.memory.enabled is false. With the refactored AppConfig.current() default, memory is disabled, so these tests have to mock it explicitly. --- backend/tests/test_memory_queue_user_isolation.py | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/backend/tests/test_memory_queue_user_isolation.py b/backend/tests/test_memory_queue_user_isolation.py index 1a209d659..5b5c822dc 100644 --- a/backend/tests/test_memory_queue_user_isolation.py +++ b/backend/tests/test_memory_queue_user_isolation.py @@ -1,7 +1,19 @@ """Tests for user_id propagation through memory queue.""" from unittest.mock import MagicMock, patch +import pytest + from deerflow.agents.memory.queue import ConversationContext, MemoryUpdateQueue +from deerflow.config.app_config import AppConfig +from deerflow.config.memory_config import MemoryConfig + + +@pytest.fixture(autouse=True) +def _enable_memory(monkeypatch): + """Ensure MemoryUpdateQueue.add() doesn't early-return on disabled memory.""" + config = MagicMock(spec=AppConfig) + config.memory = MemoryConfig(enabled=True) + monkeypatch.setattr(AppConfig, "current", staticmethod(lambda: config)) def test_conversation_context_has_user_id():