581 lines
22 KiB
Python
581 lines
22 KiB
Python
from unittest.mock import AsyncMock, patch
|
|
|
|
import pytest
|
|
from honcho_core.types import DeriverStatus
|
|
|
|
from sdks.python.src.honcho.async_client.client import AsyncHoncho
|
|
from sdks.python.src.honcho.async_client.peer import AsyncPeer
|
|
from sdks.python.src.honcho.async_client.session import (
|
|
AsyncSession,
|
|
)
|
|
from sdks.python.src.honcho.async_client.session import (
|
|
SessionPeerConfig as AsyncSessionPeerConfig,
|
|
)
|
|
from sdks.python.src.honcho.client import Honcho
|
|
from sdks.python.src.honcho.peer import Peer
|
|
from sdks.python.src.honcho.session import Session, SessionPeerConfig
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_session_metadata(client_fixture: tuple[Honcho | AsyncHoncho, str]):
|
|
"""
|
|
Tests creation and metadata operations for sessions.
|
|
"""
|
|
honcho_client, client_type = client_fixture
|
|
|
|
if client_type == "async":
|
|
assert isinstance(honcho_client, AsyncHoncho)
|
|
session = await honcho_client.session(id="test-session-meta")
|
|
assert isinstance(session, AsyncSession)
|
|
|
|
metadata = await session.get_metadata()
|
|
assert metadata == {}
|
|
|
|
await session.set_metadata({"foo": "bar"})
|
|
metadata = await session.get_metadata()
|
|
assert metadata == {"foo": "bar"}
|
|
else:
|
|
assert isinstance(honcho_client, Honcho)
|
|
session = honcho_client.session(id="test-session-meta")
|
|
assert isinstance(session, Session)
|
|
|
|
metadata = session.get_metadata()
|
|
assert metadata == {}
|
|
|
|
session.set_metadata({"foo": "bar"})
|
|
metadata = session.get_metadata()
|
|
assert metadata == {"foo": "bar"}
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_session_peer_management(
|
|
client_fixture: tuple[Honcho | AsyncHoncho, str],
|
|
):
|
|
"""
|
|
Tests adding, setting, getting, and removing peers from a session.
|
|
"""
|
|
honcho_client, client_type = client_fixture
|
|
|
|
if client_type == "async":
|
|
assert isinstance(honcho_client, AsyncHoncho)
|
|
session = await honcho_client.session(id="test-session-peers")
|
|
assert isinstance(session, AsyncSession)
|
|
peer1 = await honcho_client.peer(id="p1")
|
|
assert isinstance(peer1, AsyncPeer)
|
|
peer2 = await honcho_client.peer(id="p2")
|
|
assert isinstance(peer2, AsyncPeer)
|
|
peer3 = await honcho_client.peer(id="p3")
|
|
assert isinstance(peer3, AsyncPeer)
|
|
|
|
await session.add_peers([peer1, peer2])
|
|
peers = await session.get_peers()
|
|
assert len(peers) == 2
|
|
peer_ids = {p.id for p in peers}
|
|
assert "p1" in peer_ids and "p2" in peer_ids
|
|
|
|
await session.set_peers([peer2, peer3])
|
|
peers = await session.get_peers()
|
|
assert len(peers) == 2
|
|
peer_ids = {p.id for p in peers}
|
|
assert "p2" in peer_ids and "p3" in peer_ids
|
|
|
|
await session.remove_peers([peer2])
|
|
peers = await session.get_peers()
|
|
assert len(peers) == 1
|
|
assert peers[0].id == "p3"
|
|
else:
|
|
assert isinstance(honcho_client, Honcho)
|
|
session = honcho_client.session(id="test-session-peers")
|
|
assert isinstance(session, Session)
|
|
peer1 = honcho_client.peer(id="p1")
|
|
assert isinstance(peer1, Peer)
|
|
peer2 = honcho_client.peer(id="p2")
|
|
assert isinstance(peer2, Peer)
|
|
peer3 = honcho_client.peer(id="p3")
|
|
assert isinstance(peer3, Peer)
|
|
|
|
session.add_peers([peer1, peer2])
|
|
peers = session.get_peers()
|
|
assert len(peers) == 2
|
|
peer_ids = {p.id for p in peers}
|
|
assert "p1" in peer_ids and "p2" in peer_ids
|
|
|
|
session.set_peers([peer2, peer3])
|
|
peers = session.get_peers()
|
|
assert len(peers) == 2
|
|
peer_ids = {p.id for p in peers}
|
|
assert "p2" in peer_ids and "p3" in peer_ids
|
|
|
|
session.remove_peers([peer2])
|
|
peers = session.get_peers()
|
|
assert len(peers) == 1
|
|
assert peers[0].id == "p3"
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_session_peer_config(client_fixture: tuple[Honcho | AsyncHoncho, str]):
|
|
"""
|
|
Tests getting and setting peer configurations in a session.
|
|
"""
|
|
honcho_client, client_type = client_fixture
|
|
|
|
if client_type == "async":
|
|
assert isinstance(honcho_client, AsyncHoncho)
|
|
config = AsyncSessionPeerConfig(observe_others=False, observe_me=False)
|
|
session = await honcho_client.session(id="test-session-config")
|
|
assert isinstance(session, AsyncSession)
|
|
peer = await honcho_client.peer(id="p-config")
|
|
assert isinstance(peer, AsyncPeer)
|
|
await session.add_peers([(peer, config)])
|
|
|
|
retrieved_config = await session.get_peer_config(peer)
|
|
assert retrieved_config.observe_me is False
|
|
assert retrieved_config.observe_others is False
|
|
|
|
await session.set_peer_config(
|
|
peer, AsyncSessionPeerConfig(observe_others=True, observe_me=True)
|
|
)
|
|
retrieved_config = await session.get_peer_config(peer)
|
|
assert retrieved_config.observe_me is True
|
|
assert retrieved_config.observe_others is True
|
|
else:
|
|
assert isinstance(honcho_client, Honcho)
|
|
config = SessionPeerConfig(observe_others=False, observe_me=False)
|
|
session = honcho_client.session(id="test-session-config")
|
|
assert isinstance(session, Session)
|
|
peer = honcho_client.peer(id="p-config")
|
|
assert isinstance(peer, Peer)
|
|
session.add_peers([(peer, config)])
|
|
|
|
retrieved_config = session.get_peer_config(peer)
|
|
assert retrieved_config.observe_me is False
|
|
assert retrieved_config.observe_others is False
|
|
|
|
session.set_peer_config(
|
|
peer, SessionPeerConfig(observe_others=True, observe_me=True)
|
|
)
|
|
retrieved_config = session.get_peer_config(peer)
|
|
assert retrieved_config.observe_me
|
|
assert retrieved_config.observe_others
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_session_messages(client_fixture: tuple[Honcho | AsyncHoncho, str]):
|
|
"""
|
|
Tests adding and getting messages from a session.
|
|
"""
|
|
honcho_client, client_type = client_fixture
|
|
|
|
if client_type == "async":
|
|
assert isinstance(honcho_client, AsyncHoncho)
|
|
session = await honcho_client.session(id="test-session-msg")
|
|
assert isinstance(session, AsyncSession)
|
|
user = await honcho_client.peer(id="user-msg")
|
|
assert isinstance(user, AsyncPeer)
|
|
assistant = await honcho_client.peer(id="assistant-msg")
|
|
assert isinstance(assistant, AsyncPeer)
|
|
|
|
await session.add_messages(
|
|
[
|
|
user.message("Hello assistant"),
|
|
assistant.message("Hello user"),
|
|
]
|
|
)
|
|
messages_page = await session.get_messages()
|
|
messages = messages_page.items
|
|
assert len(messages) == 2
|
|
|
|
messages_page = await session.get_messages(filters={"peer_id": user.id})
|
|
messages = messages_page.items
|
|
assert len(messages) == 1
|
|
assert messages[0].content == "Hello assistant"
|
|
else:
|
|
assert isinstance(honcho_client, Honcho)
|
|
session = honcho_client.session(id="test-session-msg")
|
|
assert isinstance(session, Session)
|
|
user = honcho_client.peer(id="user-msg")
|
|
assert isinstance(user, Peer)
|
|
assistant = honcho_client.peer(id="assistant-msg")
|
|
assert isinstance(assistant, Peer)
|
|
|
|
session.add_messages(
|
|
[
|
|
user.message("Hello assistant"),
|
|
assistant.message("Hello user"),
|
|
]
|
|
)
|
|
messages_page = session.get_messages()
|
|
messages = list(messages_page)
|
|
assert len(messages) == 2
|
|
|
|
messages_page = session.get_messages(filters={"peer_id": user.id})
|
|
messages = list(messages_page)
|
|
assert len(messages) == 1
|
|
assert messages[0].content == "Hello assistant"
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_session_get_context(client_fixture: tuple[Honcho | AsyncHoncho, str]):
|
|
"""
|
|
Tests getting the context of a session.
|
|
"""
|
|
honcho_client, client_type = client_fixture
|
|
|
|
if client_type == "async":
|
|
assert isinstance(honcho_client, AsyncHoncho)
|
|
session = await honcho_client.session(id="test-session-ctx")
|
|
assert isinstance(session, AsyncSession)
|
|
user = await honcho_client.peer(id="user-ctx")
|
|
assert isinstance(user, AsyncPeer)
|
|
await session.add_messages([user.message("This is a context test.")])
|
|
context = await session.get_context()
|
|
assert len(context.messages) == 1
|
|
assert "context test" in context.messages[0].content
|
|
else:
|
|
assert isinstance(honcho_client, Honcho)
|
|
session = honcho_client.session(id="test-session-ctx")
|
|
assert isinstance(session, Session)
|
|
user = honcho_client.peer(id="user-ctx")
|
|
assert isinstance(user, Peer)
|
|
session.add_messages([user.message("This is a context test.")])
|
|
context = session.get_context()
|
|
assert len(context.messages) == 1
|
|
assert "context test" in context.messages[0].content
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_session_search(client_fixture: tuple[Honcho | AsyncHoncho, str]):
|
|
"""
|
|
Tests searching for messages in a session.
|
|
"""
|
|
honcho_client, client_type = client_fixture
|
|
search_query = "a unique message for session search"
|
|
|
|
if client_type == "async":
|
|
assert isinstance(honcho_client, AsyncHoncho)
|
|
session = await honcho_client.session(id="search-session-s")
|
|
assert isinstance(session, AsyncSession)
|
|
user = await honcho_client.peer(id="search-user-s")
|
|
assert isinstance(user, AsyncPeer)
|
|
await session.add_messages([user.message(search_query)])
|
|
|
|
search_results = await session.search(search_query)
|
|
assert isinstance(search_results, list)
|
|
assert len(search_results) >= 1
|
|
assert search_query in search_results[0].content
|
|
else:
|
|
assert isinstance(honcho_client, Honcho)
|
|
session = honcho_client.session(id="search-session-s")
|
|
assert isinstance(session, Session)
|
|
user = honcho_client.peer(id="search-user-s")
|
|
assert isinstance(user, Peer)
|
|
session.add_messages([user.message(search_query)])
|
|
|
|
search_results = session.search(search_query)
|
|
assert isinstance(search_results, list)
|
|
assert len(search_results) >= 1
|
|
assert search_query in search_results[0].content
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_session_add_messages_return_value(
|
|
client_fixture: tuple[Honcho | AsyncHoncho, str],
|
|
):
|
|
"""
|
|
Tests that add_messages returns a list of Message objects.
|
|
"""
|
|
honcho_client, client_type = client_fixture
|
|
|
|
if client_type == "async":
|
|
assert isinstance(honcho_client, AsyncHoncho)
|
|
session = await honcho_client.session(id="test-session-add-msg-return")
|
|
assert isinstance(session, AsyncSession)
|
|
user = await honcho_client.peer(id="user-add-msg-return")
|
|
assert isinstance(user, AsyncPeer)
|
|
assistant = await honcho_client.peer(id="assistant-add-msg-return")
|
|
assert isinstance(assistant, AsyncPeer)
|
|
|
|
# Test single message return value
|
|
from honcho_core.types.workspaces.sessions.message import Message
|
|
|
|
result = await session.add_messages(user.message("Hello assistant"))
|
|
assert isinstance(result, list)
|
|
assert len(result) == 1
|
|
assert isinstance(result[0], Message)
|
|
assert result[0].content == "Hello assistant"
|
|
assert result[0].peer_id == user.id
|
|
|
|
# Test multiple messages return value
|
|
result = await session.add_messages(
|
|
[
|
|
user.message("How are you?"),
|
|
assistant.message("I'm doing well, thank you!"),
|
|
]
|
|
)
|
|
assert isinstance(result, list)
|
|
assert len(result) == 2
|
|
assert all(isinstance(msg, Message) for msg in result)
|
|
assert result[0].content == "How are you?"
|
|
assert result[0].peer_id == user.id
|
|
assert result[1].content == "I'm doing well, thank you!"
|
|
assert result[1].peer_id == assistant.id
|
|
else:
|
|
assert isinstance(honcho_client, Honcho)
|
|
session = honcho_client.session(id="test-session-add-msg-return")
|
|
assert isinstance(session, Session)
|
|
user = honcho_client.peer(id="user-add-msg-return")
|
|
assert isinstance(user, Peer)
|
|
assistant = honcho_client.peer(id="assistant-add-msg-return")
|
|
assert isinstance(assistant, Peer)
|
|
|
|
# Test single message return value
|
|
from honcho_core.types.workspaces.sessions.message import Message
|
|
|
|
result = session.add_messages(user.message("Hello assistant"))
|
|
assert isinstance(result, list)
|
|
assert len(result) == 1
|
|
assert isinstance(result[0], Message)
|
|
assert result[0].content == "Hello assistant"
|
|
assert result[0].peer_id == user.id
|
|
|
|
# Test multiple messages return value
|
|
result = session.add_messages(
|
|
[
|
|
user.message("How are you?"),
|
|
assistant.message("I'm doing well, thank you!"),
|
|
]
|
|
)
|
|
assert isinstance(result, list)
|
|
assert len(result) == 2
|
|
assert all(isinstance(msg, Message) for msg in result)
|
|
assert result[0].content == "How are you?"
|
|
assert result[0].peer_id == user.id
|
|
assert result[1].content == "I'm doing well, thank you!"
|
|
assert result[1].peer_id == assistant.id
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_session_working_rep(client_fixture: tuple[Honcho | AsyncHoncho, str]):
|
|
"""
|
|
Tests getting the working representation of a peer in a session.
|
|
"""
|
|
honcho_client, client_type = client_fixture
|
|
|
|
if client_type == "async":
|
|
assert isinstance(honcho_client, AsyncHoncho)
|
|
session = await honcho_client.session(id="test-session-wr")
|
|
assert isinstance(session, AsyncSession)
|
|
peer = await honcho_client.peer(id="peer-wr")
|
|
assert isinstance(peer, AsyncPeer)
|
|
await session.add_messages([peer.message("test message for working rep")])
|
|
_working_rep = await session.working_rep(peer)
|
|
else:
|
|
assert isinstance(honcho_client, Honcho)
|
|
session = honcho_client.session(id="test-session-wr")
|
|
assert isinstance(session, Session)
|
|
peer = honcho_client.peer(id="peer-wr")
|
|
assert isinstance(peer, Peer)
|
|
session.add_messages([peer.message("test message for working rep")])
|
|
_working_rep = session.working_rep(peer)
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_session_delete(client_fixture: tuple[Honcho | AsyncHoncho, str]) -> None:
|
|
"""
|
|
Tests deleting a session and verifying all associated data is removed.
|
|
"""
|
|
honcho_client, client_type = client_fixture
|
|
|
|
if client_type == "async":
|
|
assert isinstance(honcho_client, AsyncHoncho)
|
|
session = await honcho_client.session(id="test-session-delete")
|
|
assert isinstance(session, AsyncSession)
|
|
|
|
# Add a peer and messages to make the session have data
|
|
user = await honcho_client.peer(id="user-delete")
|
|
await session.add_peers([user])
|
|
await session.add_messages(
|
|
[user.message("Test message that should be deleted")]
|
|
)
|
|
|
|
# Verify messages exist before deletion
|
|
messages_page = await session.get_messages()
|
|
messages = messages_page.items
|
|
assert len(messages) == 1
|
|
|
|
# Delete should not raise an exception
|
|
await session.delete()
|
|
|
|
# Verify session is removed from active sessions list
|
|
all_sessions_page = await honcho_client.get_sessions({"is_active": True})
|
|
all_sessions = all_sessions_page.items
|
|
all_session_ids = [s.id for s in all_sessions]
|
|
assert "test-session-delete" not in all_session_ids
|
|
|
|
# Verify session is also removed from all sessions (hard delete, not soft)
|
|
all_sessions_page = await honcho_client.get_sessions()
|
|
all_sessions = all_sessions_page.items
|
|
all_session_ids = [s.id for s in all_sessions]
|
|
assert "test-session-delete" not in all_session_ids
|
|
else:
|
|
assert isinstance(honcho_client, Honcho)
|
|
session = honcho_client.session(id="test-session-delete")
|
|
assert isinstance(session, Session)
|
|
|
|
# Add a peer and messages to make the session have data
|
|
user = honcho_client.peer(id="user-delete")
|
|
session.add_peers([user])
|
|
session.add_messages([user.message("Test message that should be deleted")])
|
|
|
|
# Verify messages exist before deletion
|
|
messages_page = session.get_messages()
|
|
messages = list(messages_page)
|
|
assert len(messages) == 1
|
|
|
|
# Delete should not raise an exception
|
|
session.delete()
|
|
|
|
# Verify session is removed from active sessions list
|
|
all_sessions_page = honcho_client.get_sessions({"is_active": True})
|
|
all_sessions = list(all_sessions_page)
|
|
all_session_ids = [s.id for s in all_sessions]
|
|
assert "test-session-delete" not in all_session_ids
|
|
|
|
# Verify session is also removed from all sessions (hard delete, not soft)
|
|
all_sessions_page = honcho_client.get_sessions()
|
|
all_sessions = list(all_sessions_page)
|
|
all_session_ids = [s.id for s in all_sessions]
|
|
assert "test-session-delete" not in all_session_ids
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_session_get_deriver_status(
|
|
client_fixture: tuple[Honcho | AsyncHoncho, str],
|
|
):
|
|
"""
|
|
Tests getting deriver status with various parameter combinations for sessions.
|
|
"""
|
|
honcho_client, client_type = client_fixture
|
|
|
|
if client_type == "async":
|
|
assert isinstance(honcho_client, AsyncHoncho)
|
|
session = await honcho_client.session(id="test-session-deriver-status")
|
|
assert isinstance(session, AsyncSession)
|
|
|
|
status = await session.get_deriver_status()
|
|
assert isinstance(status, DeriverStatus)
|
|
assert hasattr(status, "total_work_units")
|
|
assert hasattr(status, "completed_work_units")
|
|
assert hasattr(status, "in_progress_work_units")
|
|
assert hasattr(status, "pending_work_units")
|
|
assert status.sessions is None
|
|
|
|
# Test with observer_id only
|
|
peer = await honcho_client.peer(id="test-peer-session-deriver")
|
|
await peer.get_metadata() # Create the peer
|
|
status = await session.get_deriver_status(observer_id=peer.id)
|
|
assert isinstance(status, DeriverStatus)
|
|
|
|
# Test with sender_id only
|
|
status = await session.get_deriver_status(sender_id=peer.id)
|
|
assert isinstance(status, DeriverStatus)
|
|
|
|
# Test with both observer_id and sender_id
|
|
status = await session.get_deriver_status(
|
|
observer_id=peer.id, sender_id=peer.id
|
|
)
|
|
assert isinstance(status, DeriverStatus)
|
|
else:
|
|
assert isinstance(honcho_client, Honcho)
|
|
session = honcho_client.session(id="test-session-deriver-status")
|
|
assert isinstance(session, Session)
|
|
|
|
# Test with no parameters
|
|
status = session.get_deriver_status()
|
|
assert isinstance(status, DeriverStatus)
|
|
assert hasattr(status, "total_work_units")
|
|
assert hasattr(status, "completed_work_units")
|
|
assert hasattr(status, "in_progress_work_units")
|
|
assert hasattr(status, "pending_work_units")
|
|
assert status.sessions is None
|
|
|
|
# Test with observer_id only
|
|
peer = honcho_client.peer(id="test-peer-session-deriver")
|
|
peer.get_metadata() # Create the peer
|
|
status = session.get_deriver_status(observer_id=peer.id)
|
|
assert isinstance(status, DeriverStatus)
|
|
|
|
# Test with sender_id only
|
|
status = session.get_deriver_status(sender_id=peer.id)
|
|
assert isinstance(status, DeriverStatus)
|
|
|
|
# Test with both observer_id and sender_id
|
|
status = session.get_deriver_status(observer_id=peer.id, sender_id=peer.id)
|
|
assert isinstance(status, DeriverStatus)
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_session_poll_deriver_status(
|
|
client_fixture: tuple[Honcho | AsyncHoncho, str],
|
|
):
|
|
"""
|
|
Tests polling deriver status until completion for sessions.
|
|
"""
|
|
honcho_client, client_type = client_fixture
|
|
|
|
# Mock the get_deriver_status method to return a "completed" status
|
|
# to avoid infinite polling in tests
|
|
completed_status = DeriverStatus(
|
|
total_work_units=0,
|
|
completed_work_units=0,
|
|
in_progress_work_units=0,
|
|
pending_work_units=0,
|
|
)
|
|
|
|
if client_type == "async":
|
|
assert isinstance(honcho_client, AsyncHoncho)
|
|
session = await honcho_client.session(id="test-session-poll-deriver")
|
|
assert isinstance(session, AsyncSession)
|
|
|
|
with patch.object(
|
|
session.__class__,
|
|
"get_deriver_status",
|
|
new=AsyncMock(return_value=completed_status),
|
|
):
|
|
status = await session.poll_deriver_status()
|
|
assert isinstance(status, DeriverStatus)
|
|
assert status.pending_work_units == 0
|
|
assert status.in_progress_work_units == 0
|
|
|
|
# Test with parameters
|
|
peer = await honcho_client.peer(id="test-peer-session-poll")
|
|
with patch.object(
|
|
session.__class__,
|
|
"get_deriver_status",
|
|
new=AsyncMock(return_value=completed_status),
|
|
):
|
|
status = await session.poll_deriver_status(
|
|
observer_id=peer.id, sender_id=peer.id
|
|
)
|
|
assert isinstance(status, DeriverStatus)
|
|
else:
|
|
assert isinstance(honcho_client, Honcho)
|
|
session = honcho_client.session(id="test-session-poll-deriver")
|
|
assert isinstance(session, Session)
|
|
|
|
with patch.object(
|
|
session.__class__, "get_deriver_status", return_value=completed_status
|
|
):
|
|
status = session.poll_deriver_status()
|
|
assert isinstance(status, DeriverStatus)
|
|
assert status.pending_work_units == 0
|
|
assert status.in_progress_work_units == 0
|
|
|
|
# Test with parameters
|
|
peer = honcho_client.peer(id="test-peer-session-poll")
|
|
with patch.object(
|
|
session.__class__, "get_deriver_status", return_value=completed_status
|
|
):
|
|
status = session.poll_deriver_status(observer_id=peer.id, sender_id=peer.id)
|
|
assert isinstance(status, DeriverStatus)
|