mirror of
https://github.com/basicmachines-co/basic-memory
synced 2026-06-21 13:47:35 +00:00
020957cd76
Signed-off-by: phernandez <paul@basicmachines.co> Co-authored-by: phernandez <paul@basicmachines.co> Co-authored-by: Claude <noreply@anthropic.com>
221 lines
8.6 KiB
Python
221 lines
8.6 KiB
Python
"""Tests for context service."""
|
|
|
|
from datetime import datetime, timedelta, UTC
|
|
|
|
import pytest
|
|
import pytest_asyncio
|
|
|
|
from basic_memory.repository.search_repository import SearchIndexRow
|
|
from basic_memory.schemas.memory import memory_url, memory_url_path
|
|
from basic_memory.schemas.search import SearchItemType
|
|
from basic_memory.services.context_service import ContextService
|
|
|
|
|
|
@pytest_asyncio.fixture
|
|
async def context_service(search_repository, entity_repository, observation_repository):
|
|
"""Create context service for testing."""
|
|
return ContextService(search_repository, entity_repository, observation_repository)
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_find_connected_depth_limit(context_service, test_graph):
|
|
"""Test depth limiting works.
|
|
Our traversal path is:
|
|
- Depth 0: Root
|
|
- Depth 1: Relations + directly connected entities (Connected1, Connected2)
|
|
- Depth 2: Relations + next level entities (Deep)
|
|
"""
|
|
type_id_pairs = [("entity", test_graph["root"].id)]
|
|
|
|
# With depth=1, we get direct connections
|
|
# shallow_results = await context_service.find_related(type_id_pairs, max_depth=1)
|
|
# shallow_entities = {(r.id, r.type) for r in shallow_results if r.type == "entity"}
|
|
#
|
|
# assert (test_graph["deep"].id, "entity") not in shallow_entities
|
|
|
|
# search deeper
|
|
deep_results = await context_service.find_related(type_id_pairs, max_depth=3, max_results=100)
|
|
deep_entities = {(r.id, r.type) for r in deep_results if r.type == "entity"}
|
|
print(deep_entities)
|
|
# Should now include Deep entity
|
|
assert (test_graph["deep"].id, "entity") in deep_entities
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_find_connected_timeframe(
|
|
context_service, test_graph, search_repository, entity_repository
|
|
):
|
|
"""Test timeframe filtering.
|
|
This tests how traversal is affected by the item dates.
|
|
When we filter by date, items are only included if:
|
|
1. They match the timeframe
|
|
2. There is a valid path to them through other items in the timeframe
|
|
"""
|
|
now = datetime.now(UTC)
|
|
old_date = now - timedelta(days=10)
|
|
recent_date = now - timedelta(days=1)
|
|
|
|
# Update entity table timestamps directly
|
|
# Root entity uses old date
|
|
root_entity = test_graph["root"]
|
|
await entity_repository.update(root_entity.id, {"created_at": old_date, "updated_at": old_date})
|
|
|
|
# Connected entity uses recent date
|
|
connected_entity = test_graph["connected1"]
|
|
await entity_repository.update(
|
|
connected_entity.id, {"created_at": recent_date, "updated_at": recent_date}
|
|
)
|
|
|
|
# Also update search_index for test consistency
|
|
await search_repository.index_item(
|
|
SearchIndexRow(
|
|
project_id=entity_repository.project_id,
|
|
id=test_graph["root"].id,
|
|
title=test_graph["root"].title,
|
|
content_snippet="Root content",
|
|
permalink=test_graph["root"].permalink,
|
|
file_path=test_graph["root"].file_path,
|
|
type=SearchItemType.ENTITY,
|
|
metadata={"created_at": old_date.isoformat()},
|
|
created_at=old_date.isoformat(),
|
|
updated_at=old_date.isoformat(),
|
|
)
|
|
)
|
|
await search_repository.index_item(
|
|
SearchIndexRow(
|
|
project_id=entity_repository.project_id,
|
|
id=test_graph["relations"][0].id,
|
|
title="Root Entity → Connected Entity 1",
|
|
content_snippet="",
|
|
permalink=f"{test_graph['root'].permalink}/connects_to/{test_graph['connected1'].permalink}",
|
|
file_path=test_graph["root"].file_path,
|
|
type=SearchItemType.RELATION,
|
|
from_id=test_graph["root"].id,
|
|
to_id=test_graph["connected1"].id,
|
|
relation_type="connects_to",
|
|
metadata={"created_at": old_date.isoformat()},
|
|
created_at=old_date.isoformat(),
|
|
updated_at=old_date.isoformat(),
|
|
)
|
|
)
|
|
await search_repository.index_item(
|
|
SearchIndexRow(
|
|
project_id=entity_repository.project_id,
|
|
id=test_graph["connected1"].id,
|
|
title=test_graph["connected1"].title,
|
|
content_snippet="Connected 1 content",
|
|
permalink=test_graph["connected1"].permalink,
|
|
file_path=test_graph["connected1"].file_path,
|
|
type=SearchItemType.ENTITY,
|
|
metadata={"created_at": recent_date.isoformat()},
|
|
created_at=recent_date.isoformat(),
|
|
updated_at=recent_date.isoformat(),
|
|
)
|
|
)
|
|
|
|
type_id_pairs = [("entity", test_graph["root"].id)]
|
|
|
|
# Search with a 7-day cutoff
|
|
since_date = now - timedelta(days=7)
|
|
results = await context_service.find_related(type_id_pairs, since=since_date)
|
|
|
|
# Only connected1 is recent, but we can't get to it
|
|
# because its connecting relation is too old and is filtered out
|
|
# (we can only reach connected1 through a relation starting from root)
|
|
entity_ids = {r.id for r in results if r.type == "entity"}
|
|
assert len(entity_ids) == 0 # No accessible entities within timeframe
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_build_context(context_service, test_graph):
|
|
"""Test exact permalink lookup."""
|
|
url = memory_url.validate_strings("memory://test/root")
|
|
context_result = await context_service.build_context(url)
|
|
|
|
# Check metadata
|
|
assert context_result.metadata.uri == memory_url_path(url)
|
|
assert context_result.metadata.depth == 1
|
|
assert context_result.metadata.primary_count == 1
|
|
assert context_result.metadata.related_count > 0
|
|
assert context_result.metadata.generated_at is not None
|
|
|
|
# Check results
|
|
assert len(context_result.results) == 1
|
|
context_item = context_result.results[0]
|
|
|
|
# Check primary result
|
|
primary_result = context_item.primary_result
|
|
assert primary_result.id == test_graph["root"].id
|
|
assert primary_result.type == "entity"
|
|
assert primary_result.title == "Root"
|
|
assert primary_result.permalink == "test/root"
|
|
assert primary_result.file_path == "test/Root.md"
|
|
assert primary_result.created_at is not None
|
|
|
|
# Check related results
|
|
assert len(context_item.related_results) > 0
|
|
|
|
# Find related relation
|
|
relation = next((r for r in context_item.related_results if r.type == "relation"), None)
|
|
assert relation is not None
|
|
assert relation.relation_type == "connects_to"
|
|
assert relation.from_id == test_graph["root"].id
|
|
assert relation.to_id == test_graph["connected1"].id
|
|
|
|
# Find related entity
|
|
related_entity = next((r for r in context_item.related_results if r.type == "entity"), None)
|
|
assert related_entity is not None
|
|
assert related_entity.id == test_graph["connected1"].id
|
|
assert related_entity.title == test_graph["connected1"].title
|
|
assert related_entity.permalink == test_graph["connected1"].permalink
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_build_context_with_observations(context_service, test_graph):
|
|
"""Test context building with observations."""
|
|
# The test_graph fixture already creates observations for root entity
|
|
# Let's use those existing observations
|
|
|
|
# Build context
|
|
url = memory_url.validate_strings("memory://test/root")
|
|
context_result = await context_service.build_context(url, include_observations=True)
|
|
|
|
# Check the metadata
|
|
assert context_result.metadata.total_observations > 0
|
|
assert len(context_result.results) == 1
|
|
|
|
# Check that observations were included
|
|
context_item = context_result.results[0]
|
|
assert len(context_item.observations) > 0
|
|
|
|
# Check observation properties
|
|
for observation in context_item.observations:
|
|
assert observation.type == "observation"
|
|
assert observation.category in ["note", "tech"] # Categories from test_graph fixture
|
|
assert observation.entity_id == test_graph["root"].id
|
|
|
|
# Verify at least one observation has the correct category and content
|
|
note_observation = next((o for o in context_item.observations if o.category == "note"), None)
|
|
assert note_observation is not None
|
|
assert "Root note" in note_observation.content
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_build_context_not_found(context_service):
|
|
"""Test handling non-existent permalinks."""
|
|
context = await context_service.build_context("memory://does/not/exist")
|
|
assert len(context.results) == 0
|
|
assert context.metadata.primary_count == 0
|
|
assert context.metadata.related_count == 0
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_context_metadata(context_service, test_graph):
|
|
"""Test metadata is correctly populated."""
|
|
context = await context_service.build_context("memory://test/root", depth=2)
|
|
metadata = context.metadata
|
|
assert metadata.uri == "test/root"
|
|
assert metadata.depth == 2
|
|
assert metadata.generated_at is not None
|
|
assert metadata.primary_count > 0
|