mirror of
https://github.com/ZhuLinsen/daily_stock_analysis
synced 2026-09-20 10:53:33 +08:00
ed848da6f0
* feat: persist Agent Chat skill selection by session * test: satisfy ChatPage mock immutability lint * fix: preserve implicit Skill state for legacy sessions * fix: make top-level Skill selection authoritative * fix: preserve session skills for invalid requests --------- Co-authored-by: zhulinsen <42829555+ZhuLinsen@users.noreply.github.com>
60 lines
1.8 KiB
Python
60 lines
1.8 KiB
Python
# -*- coding: utf-8 -*-
|
|
"""Thread-safety regression tests for ConversationManager."""
|
|
|
|
import threading
|
|
import unittest
|
|
from unittest.mock import patch
|
|
|
|
from src.agent.conversation import ConversationManager
|
|
|
|
|
|
class ConversationManagerThreadSafetyTestCase(unittest.TestCase):
|
|
@patch("src.agent.conversation.get_db")
|
|
def test_add_user_message_uses_session_state_transaction(self, get_db):
|
|
get_db.return_value.save_conversation_user_turn.return_value = 42
|
|
manager = ConversationManager()
|
|
|
|
message_id = manager.add_user_message(
|
|
"skill-session",
|
|
"hello",
|
|
[],
|
|
)
|
|
|
|
self.assertEqual(message_id, 42)
|
|
get_db.return_value.save_conversation_user_turn.assert_called_once_with(
|
|
"skill-session",
|
|
"hello",
|
|
[],
|
|
)
|
|
|
|
def test_add_message_is_safe_under_parallel_session_creation(self):
|
|
manager = ConversationManager()
|
|
errors = []
|
|
start = threading.Event()
|
|
|
|
def _worker(worker_id: int) -> None:
|
|
start.wait()
|
|
try:
|
|
for message_id in range(1000):
|
|
manager.add_message(f"session-{worker_id}-{message_id}", "user", "hello")
|
|
except Exception as exc: # pragma: no cover - failures are asserted below
|
|
errors.append(exc)
|
|
|
|
threads = [
|
|
threading.Thread(target=_worker, args=(idx,), daemon=True)
|
|
for idx in range(6)
|
|
]
|
|
|
|
with patch("src.agent.conversation.ConversationSession.add_message", autospec=True):
|
|
for thread in threads:
|
|
thread.start()
|
|
start.set()
|
|
for thread in threads:
|
|
thread.join()
|
|
|
|
self.assertEqual(errors, [])
|
|
|
|
|
|
if __name__ == "__main__":
|
|
unittest.main()
|