249 lines
9.2 KiB
Python
249 lines
9.2 KiB
Python
import os
|
|
import time
|
|
import unittest
|
|
import xml.etree.ElementTree as ET
|
|
from unittest.mock import patch
|
|
|
|
from sia.entry.background_entry import BackgroundEntry
|
|
from sia.entry import Entry
|
|
from sia.entry.parse_error_entry import ParseErrorEntry
|
|
from sia.entry.read_entry import ReadEntry
|
|
from sia.entry.reasoning_entry import ReasoningEntry
|
|
from sia.web_io_buffer import WebIOBuffer
|
|
from sia.working_memory import WorkingMemory
|
|
from sia.entry.write_entry import WriteEntry
|
|
|
|
|
|
class MockEntry(Entry):
|
|
"""Mock entry class for testing"""
|
|
def __init__(self, id: str):
|
|
super().__init__(id)
|
|
self.updated = False
|
|
self.cleaned_up = False
|
|
|
|
def update(self) -> None:
|
|
self.updated = True
|
|
|
|
def cleanup(self) -> None:
|
|
self.cleaned_up = True
|
|
|
|
def generate_context(self) -> ET.Element:
|
|
elem = ET.Element("mock", {"id": self.id})
|
|
elem.text = "<![CDATA[mock content]]>"
|
|
return elem
|
|
|
|
class WorkingMemoryTest(unittest.TestCase):
|
|
def setUp(self):
|
|
"""Set up test cases with a fresh working memory"""
|
|
self.memory = WorkingMemory()
|
|
self.work_dir = os.getcwd() # Use current directory for tests
|
|
|
|
def wait_for_process_start(self, entry: BackgroundEntry, timeout=1.0):
|
|
"""Wait for process to start, checking context pid"""
|
|
start_time = time.time()
|
|
while time.time() - start_time < timeout:
|
|
entry.update()
|
|
context = entry.generate_context()
|
|
if context.get("pid") is not None:
|
|
return True
|
|
time.sleep(0.1)
|
|
return False
|
|
|
|
def test_initialization(self):
|
|
"""Test initial state of working memory"""
|
|
self.assertEqual(self.memory.get_entries_count(), 0)
|
|
self.assertEqual(self.memory.generate_context(), [])
|
|
|
|
def test_add_entry(self):
|
|
"""Test adding entries"""
|
|
entry = MockEntry("test-id-1")
|
|
self.memory.add_entry(entry)
|
|
|
|
self.assertEqual(self.memory.get_entries_count(), 1)
|
|
self.assertEqual(self.memory.get_entry("test-id-1"), entry)
|
|
|
|
def test_add_invalid_entry(self):
|
|
"""Test adding invalid entry type"""
|
|
with self.assertRaises(TypeError):
|
|
self.memory.add_entry("not an entry")
|
|
|
|
def test_remove_entry(self):
|
|
"""Test removing entries"""
|
|
entry1 = MockEntry("test-id-1")
|
|
entry2 = MockEntry("test-id-2")
|
|
|
|
self.memory.add_entry(entry1)
|
|
self.memory.add_entry(entry2)
|
|
self.memory.remove_entry("test-id-1")
|
|
|
|
self.assertEqual(self.memory.get_entries_count(), 1)
|
|
self.assertIsNone(self.memory.get_entry("test-id-1"))
|
|
self.assertEqual(self.memory.get_entry("test-id-2"), entry2)
|
|
|
|
def test_remove_nonexistent_entry(self):
|
|
"""Test removing entry that doesn't exist"""
|
|
entry = MockEntry("test-id-1")
|
|
self.memory.add_entry(entry)
|
|
self.memory.remove_entry("nonexistent-id")
|
|
|
|
self.assertEqual(self.memory.get_entries_count(), 1)
|
|
self.assertEqual(self.memory.get_entry("test-id-1"), entry)
|
|
|
|
def test_update_entries(self):
|
|
"""Test updating all entries"""
|
|
entries = [
|
|
MockEntry(f"test-id-{i}")
|
|
for i in range(3)
|
|
]
|
|
|
|
for entry in entries:
|
|
self.memory.add_entry(entry)
|
|
|
|
self.memory.update()
|
|
|
|
for entry in entries:
|
|
self.assertTrue(entry.updated)
|
|
|
|
def test_generate_context(self):
|
|
"""Test generating XML context"""
|
|
entry1 = MockEntry("test-id-1")
|
|
entry2 = MockEntry("test-id-2")
|
|
|
|
self.memory.add_entry(entry1)
|
|
self.memory.add_entry(entry2)
|
|
|
|
context = self.memory.generate_context()
|
|
|
|
self.assertEqual(len(context), 2)
|
|
self.assertEqual(context[0].tag, "mock")
|
|
self.assertEqual(context[0].get("id"), "test-id-1")
|
|
self.assertEqual(context[1].tag, "mock")
|
|
self.assertEqual(context[1].get("id"), "test-id-2")
|
|
|
|
def test_get_entries_by_type(self):
|
|
"""Test retrieving entries by type"""
|
|
# Create IO buffer for IO entries
|
|
io_buffer = WebIOBuffer()
|
|
|
|
# Add different types of entries
|
|
entries = [
|
|
ReasoningEntry("reasoning-1", "test reasoning"),
|
|
ParseErrorEntry("error-1", "bad content", "error msg"),
|
|
ReadEntry("read-1", io_buffer),
|
|
WriteEntry("write-1", "test output", io_buffer)
|
|
]
|
|
|
|
for entry in entries:
|
|
self.memory.add_entry(entry)
|
|
|
|
# Test filtering by each type
|
|
reasoning_entries = [e for e in self.memory.get_entries() if isinstance(e, ReasoningEntry)]
|
|
self.assertEqual(len(reasoning_entries), 1)
|
|
self.assertIsInstance(reasoning_entries[0], ReasoningEntry)
|
|
|
|
error_entries = [e for e in self.memory.get_entries() if isinstance(e, ParseErrorEntry)]
|
|
self.assertEqual(len(error_entries), 1)
|
|
self.assertIsInstance(error_entries[0], ParseErrorEntry)
|
|
|
|
read_entries = [e for e in self.memory.get_entries() if isinstance(e, ReadEntry)]
|
|
self.assertEqual(len(read_entries), 1)
|
|
self.assertIsInstance(read_entries[0], ReadEntry)
|
|
|
|
write_entries = [e for e in self.memory.get_entries() if isinstance(e, WriteEntry)]
|
|
self.assertEqual(len(write_entries), 1)
|
|
self.assertIsInstance(write_entries[0], WriteEntry)
|
|
|
|
def test_empty_context_generation(self):
|
|
"""Test context generation with no entries"""
|
|
context = self.memory.generate_context()
|
|
self.assertEqual(context, [])
|
|
|
|
def test_get_nonexistent_entry(self):
|
|
"""Test retrieving entry that doesn't exist"""
|
|
self.assertIsNone(self.memory.get_entry("nonexistent-id"))
|
|
|
|
def test_remove_entry_cleanup(self):
|
|
"""Test that removing an entry triggers cleanup"""
|
|
# Create and start background process
|
|
entry = BackgroundEntry("test-id", self.work_dir, "sleep 10")
|
|
self.memory.add_entry(entry)
|
|
|
|
# Wait for process to start and get PID
|
|
self.assertTrue(self.wait_for_process_start(entry))
|
|
context = entry.generate_context()
|
|
pid = int(context.get("pid"))
|
|
|
|
# Mock os.kill to verify it's called with the correct signals
|
|
with patch('os.kill') as mock_kill:
|
|
# Remove entry
|
|
self.memory.remove_entry(entry.id)
|
|
|
|
# Verify kill was called
|
|
mock_kill.assert_called()
|
|
|
|
def test_cleanup_on_memory_clear(self):
|
|
"""Test that clearing memory properly cleans up all entries"""
|
|
# Use MockEntry with cleanup tracking
|
|
entries = [MockEntry(f"id-{i}") for i in range(3)]
|
|
|
|
for entry in entries:
|
|
self.memory.add_entry(entry)
|
|
|
|
# Clear memory
|
|
self.memory.clear()
|
|
|
|
# Verify all entries were cleaned up
|
|
for entry in entries:
|
|
self.assertTrue(entry.cleaned_up)
|
|
|
|
self.assertEqual(self.memory.get_entries_count(), 0)
|
|
|
|
def test_cleanup_on_memory_deletion(self):
|
|
"""Test that deleting memory properly cleans up all entries"""
|
|
# Use a mock entry that tracks cleanup
|
|
entry = MockEntry("test-id")
|
|
|
|
# Patch the __del__ method of WorkingMemory to ensure it calls clear()
|
|
with patch.object(WorkingMemory, '__del__', lambda self: self.clear()):
|
|
# Create a new memory instance to delete
|
|
test_memory = WorkingMemory()
|
|
test_memory.add_entry(entry)
|
|
|
|
# Explicitly call del to trigger the patched __del__ method
|
|
test_memory.__del__()
|
|
|
|
# Verify entry was cleaned up
|
|
self.assertTrue(entry.cleaned_up)
|
|
|
|
def test_get_entries(self):
|
|
"""Test getting all entries"""
|
|
# Add multiple entries
|
|
entries = [
|
|
ReasoningEntry(f"id-{i}", f"content {i}")
|
|
for i in range(3)
|
|
]
|
|
|
|
for entry in entries:
|
|
self.memory.add_entry(entry)
|
|
|
|
# Get all entries
|
|
retrieved_entries = self.memory.get_entries()
|
|
|
|
# Verify count and contents
|
|
self.assertEqual(len(retrieved_entries), len(entries))
|
|
for entry in entries:
|
|
self.assertIn(entry, retrieved_entries)
|
|
|
|
def test_get_entries_returns_copy(self):
|
|
"""Test that get_entries returns a copy of the list"""
|
|
# Add an entry
|
|
entry = ReasoningEntry("test-id", "test content")
|
|
self.memory.add_entry(entry)
|
|
|
|
# Get entries and modify the returned list
|
|
entries = self.memory.get_entries()
|
|
entries.clear()
|
|
|
|
# Verify original memory is unchanged
|
|
self.assertEqual(self.memory.get_entries_count(), 1)
|
|
self.assertIsNotNone(self.memory.get_entry("test-id")) |