2026-01-26 21:45:08 -08:00
|
|
|
"""Integration tests for proxy memory system with real API calls.
|
|
|
|
|
|
|
|
|
|
These tests require:
|
|
|
|
|
- ANTHROPIC_API_KEY environment variable set
|
|
|
|
|
|
|
|
|
|
Run with:
|
|
|
|
|
ANTHROPIC_API_KEY=... uv run pytest tests/test_proxy_memory_integration.py -v
|
|
|
|
|
|
|
|
|
|
Test categories:
|
|
|
|
|
- TestMemoryHeaderValidation: User ID header validation
|
|
|
|
|
- TestMemoryToolInjection: Memory tools are injected
|
|
|
|
|
- TestMemorySaveAndSearch: End-to-end save/recall flow
|
|
|
|
|
- TestMemoryUserIsolation: User memory isolation
|
|
|
|
|
"""
|
|
|
|
|
|
|
|
|
|
import os
|
|
|
|
|
import tempfile
|
|
|
|
|
import time
|
|
|
|
|
from pathlib import Path
|
|
|
|
|
|
|
|
|
|
import pytest
|
|
|
|
|
|
|
|
|
|
# Set tokenizer parallelism before importing transformers
|
|
|
|
|
os.environ["TOKENIZERS_PARALLELISM"] = "false"
|
|
|
|
|
|
|
|
|
|
pytest.importorskip("fastapi")
|
|
|
|
|
pytest.importorskip("httpx")
|
|
|
|
|
|
|
|
|
|
from fastapi.testclient import TestClient
|
|
|
|
|
|
|
|
|
|
from headroom.proxy.server import ProxyConfig, create_app
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.fixture
|
|
|
|
|
def temp_memory_db():
|
|
|
|
|
"""Create temporary memory database."""
|
|
|
|
|
with tempfile.NamedTemporaryFile(suffix=".db", delete=False) as f:
|
|
|
|
|
yield f.name
|
|
|
|
|
# Cleanup
|
|
|
|
|
Path(f.name).unlink(missing_ok=True)
|
|
|
|
|
# Also cleanup related files (HNSW index, etc.)
|
|
|
|
|
for suffix in ["-shm", "-wal", ".hnsw"]:
|
|
|
|
|
Path(f.name + suffix).unlink(missing_ok=True)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.fixture
|
|
|
|
|
def memory_client(temp_memory_db):
|
|
|
|
|
"""Create test client with memory enabled."""
|
|
|
|
|
config = ProxyConfig(
|
|
|
|
|
optimize=False, # Disable optimization for simpler tests
|
|
|
|
|
cache_enabled=False,
|
|
|
|
|
rate_limit_enabled=False,
|
|
|
|
|
cost_tracking_enabled=False,
|
|
|
|
|
memory_enabled=True,
|
|
|
|
|
memory_backend="local",
|
|
|
|
|
memory_db_path=temp_memory_db,
|
|
|
|
|
memory_inject_tools=True,
|
|
|
|
|
memory_inject_context=True,
|
|
|
|
|
memory_top_k=5,
|
|
|
|
|
)
|
|
|
|
|
app = create_app(config)
|
|
|
|
|
with TestClient(app) as client:
|
|
|
|
|
yield client
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.fixture
|
|
|
|
|
def no_memory_client():
|
|
|
|
|
"""Create test client with memory disabled."""
|
|
|
|
|
config = ProxyConfig(
|
|
|
|
|
optimize=False,
|
|
|
|
|
cache_enabled=False,
|
|
|
|
|
rate_limit_enabled=False,
|
|
|
|
|
cost_tracking_enabled=False,
|
|
|
|
|
memory_enabled=False,
|
|
|
|
|
)
|
|
|
|
|
app = create_app(config)
|
|
|
|
|
with TestClient(app) as client:
|
|
|
|
|
yield client
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.fixture
|
|
|
|
|
def anthropic_api_key():
|
|
|
|
|
"""Get Anthropic API key from environment."""
|
|
|
|
|
return os.environ.get("ANTHROPIC_API_KEY")
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
class TestMemoryHeaderValidation:
|
|
|
|
|
"""Test user ID header validation."""
|
|
|
|
|
|
2026-01-26 22:49:31 -08:00
|
|
|
def test_missing_user_id_uses_default(self, memory_client, anthropic_api_key):
|
|
|
|
|
"""Request without x-headroom-user-id should use 'default' user for simple DevEx."""
|
2026-01-26 21:45:08 -08:00
|
|
|
if not anthropic_api_key:
|
|
|
|
|
pytest.skip("ANTHROPIC_API_KEY not set")
|
|
|
|
|
|
|
|
|
|
response = memory_client.post(
|
|
|
|
|
"/v1/messages",
|
|
|
|
|
headers={
|
|
|
|
|
"x-api-key": anthropic_api_key,
|
|
|
|
|
"anthropic-version": "2023-06-01",
|
2026-01-26 22:49:31 -08:00
|
|
|
# Note: NOT setting x-headroom-user-id - should default to "default"
|
2026-01-26 21:45:08 -08:00
|
|
|
},
|
|
|
|
|
json={
|
|
|
|
|
"model": "claude-sonnet-4-20250514",
|
|
|
|
|
"max_tokens": 100,
|
|
|
|
|
"messages": [{"role": "user", "content": "Hello"}],
|
|
|
|
|
},
|
|
|
|
|
)
|
2026-01-26 22:49:31 -08:00
|
|
|
# Should succeed, not return 400
|
|
|
|
|
assert response.status_code == 200
|
2026-01-26 21:45:08 -08:00
|
|
|
|
|
|
|
|
def test_with_user_id_succeeds(self, memory_client, anthropic_api_key):
|
|
|
|
|
"""Request with x-headroom-user-id should succeed."""
|
|
|
|
|
if not anthropic_api_key:
|
|
|
|
|
pytest.skip("ANTHROPIC_API_KEY not set")
|
|
|
|
|
|
|
|
|
|
response = memory_client.post(
|
|
|
|
|
"/v1/messages",
|
|
|
|
|
headers={
|
|
|
|
|
"x-api-key": anthropic_api_key,
|
|
|
|
|
"anthropic-version": "2023-06-01",
|
|
|
|
|
"x-headroom-user-id": "test-user-123",
|
|
|
|
|
},
|
|
|
|
|
json={
|
|
|
|
|
"model": "claude-sonnet-4-20250514",
|
|
|
|
|
"max_tokens": 100,
|
|
|
|
|
"messages": [{"role": "user", "content": "Hello, just say hi back."}],
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
assert response.status_code == 200
|
|
|
|
|
|
|
|
|
|
def test_no_memory_client_doesnt_require_user_id(self, no_memory_client, anthropic_api_key):
|
|
|
|
|
"""When memory is disabled, user ID header should not be required."""
|
|
|
|
|
if not anthropic_api_key:
|
|
|
|
|
pytest.skip("ANTHROPIC_API_KEY not set")
|
|
|
|
|
|
|
|
|
|
response = no_memory_client.post(
|
|
|
|
|
"/v1/messages",
|
|
|
|
|
headers={
|
|
|
|
|
"x-api-key": anthropic_api_key,
|
|
|
|
|
"anthropic-version": "2023-06-01",
|
|
|
|
|
# No x-headroom-user-id
|
|
|
|
|
},
|
|
|
|
|
json={
|
|
|
|
|
"model": "claude-sonnet-4-20250514",
|
|
|
|
|
"max_tokens": 100,
|
|
|
|
|
"messages": [{"role": "user", "content": "Hello, just say hi."}],
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
assert response.status_code == 200
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.skipif(not os.environ.get("ANTHROPIC_API_KEY"), reason="ANTHROPIC_API_KEY not set")
|
|
|
|
|
class TestMemoryToolInjection:
|
|
|
|
|
"""Test memory tool injection."""
|
|
|
|
|
|
|
|
|
|
def test_memory_tools_are_available(self, memory_client, anthropic_api_key):
|
|
|
|
|
"""Memory tools should be available to the LLM."""
|
|
|
|
|
response = memory_client.post(
|
|
|
|
|
"/v1/messages",
|
|
|
|
|
headers={
|
|
|
|
|
"x-api-key": anthropic_api_key,
|
|
|
|
|
"anthropic-version": "2023-06-01",
|
|
|
|
|
"x-headroom-user-id": "test-user-tool-check",
|
|
|
|
|
},
|
|
|
|
|
json={
|
|
|
|
|
"model": "claude-sonnet-4-20250514",
|
|
|
|
|
"max_tokens": 500,
|
|
|
|
|
"messages": [
|
|
|
|
|
{
|
|
|
|
|
"role": "user",
|
|
|
|
|
"content": "List the tools available to you. Just list the tool names.",
|
|
|
|
|
}
|
|
|
|
|
],
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
assert response.status_code == 200
|
|
|
|
|
|
|
|
|
|
# The response should mention memory tools
|
|
|
|
|
content = response.json().get("content", [])
|
|
|
|
|
text = ""
|
|
|
|
|
for block in content:
|
|
|
|
|
if isinstance(block, dict) and block.get("type") == "text":
|
|
|
|
|
text += block.get("text", "")
|
|
|
|
|
|
|
|
|
|
# At least one memory tool should be mentioned
|
|
|
|
|
assert any(tool in text.lower() for tool in ["memory_save", "memory_search", "memory"]), (
|
|
|
|
|
f"Memory tools not found in response: {text}"
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.skipif(not os.environ.get("ANTHROPIC_API_KEY"), reason="ANTHROPIC_API_KEY not set")
|
|
|
|
|
class TestMemorySaveAndSearch:
|
|
|
|
|
"""Test memory save and search flow."""
|
|
|
|
|
|
|
|
|
|
def test_save_memory_via_explicit_instruction(self, memory_client, anthropic_api_key):
|
|
|
|
|
"""LLM should be able to save memories when instructed."""
|
|
|
|
|
user_id = f"test-user-save-{int(time.time())}"
|
|
|
|
|
|
|
|
|
|
# Request that explicitly asks to save
|
|
|
|
|
response = memory_client.post(
|
|
|
|
|
"/v1/messages",
|
|
|
|
|
headers={
|
|
|
|
|
"x-api-key": anthropic_api_key,
|
|
|
|
|
"anthropic-version": "2023-06-01",
|
|
|
|
|
"x-headroom-user-id": user_id,
|
|
|
|
|
},
|
|
|
|
|
json={
|
|
|
|
|
"model": "claude-sonnet-4-20250514",
|
|
|
|
|
"max_tokens": 1000,
|
|
|
|
|
"messages": [
|
|
|
|
|
{
|
|
|
|
|
"role": "user",
|
|
|
|
|
"content": "Please save this to memory: My favorite programming language is Rust. "
|
|
|
|
|
"Use the memory_save tool to save this information.",
|
|
|
|
|
}
|
|
|
|
|
],
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
assert response.status_code == 200
|
|
|
|
|
|
|
|
|
|
# Check if response indicates tool was used
|
|
|
|
|
resp_json = response.json()
|
|
|
|
|
content = resp_json.get("content", [])
|
|
|
|
|
|
|
|
|
|
# Response could be tool_use (if not handled) or text (if handled)
|
|
|
|
|
# Either way, it should complete successfully
|
|
|
|
|
assert content, "Response should have content"
|
|
|
|
|
|
|
|
|
|
def test_save_and_recall_memory(self, memory_client, anthropic_api_key):
|
|
|
|
|
"""Save a memory and recall it in subsequent request."""
|
|
|
|
|
user_id = f"test-user-recall-{int(time.time())}"
|
|
|
|
|
|
|
|
|
|
# First request: save a memory with explicit instruction
|
|
|
|
|
save_response = memory_client.post(
|
|
|
|
|
"/v1/messages",
|
|
|
|
|
headers={
|
|
|
|
|
"x-api-key": anthropic_api_key,
|
|
|
|
|
"anthropic-version": "2023-06-01",
|
|
|
|
|
"x-headroom-user-id": user_id,
|
|
|
|
|
},
|
|
|
|
|
json={
|
|
|
|
|
"model": "claude-sonnet-4-20250514",
|
|
|
|
|
"max_tokens": 1000,
|
|
|
|
|
"messages": [
|
|
|
|
|
{
|
|
|
|
|
"role": "user",
|
|
|
|
|
"content": "Please remember this: My name is TestUser and I work at AcmeCorp. "
|
|
|
|
|
"Save this information using the memory_save tool.",
|
|
|
|
|
}
|
|
|
|
|
],
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
assert save_response.status_code == 200
|
|
|
|
|
|
|
|
|
|
# Wait a moment for memory to be indexed
|
|
|
|
|
time.sleep(1)
|
|
|
|
|
|
|
|
|
|
# Second request: ask about saved info
|
|
|
|
|
# Memory context should be injected automatically
|
|
|
|
|
recall_response = memory_client.post(
|
|
|
|
|
"/v1/messages",
|
|
|
|
|
headers={
|
|
|
|
|
"x-api-key": anthropic_api_key,
|
|
|
|
|
"anthropic-version": "2023-06-01",
|
|
|
|
|
"x-headroom-user-id": user_id,
|
|
|
|
|
},
|
|
|
|
|
json={
|
|
|
|
|
"model": "claude-sonnet-4-20250514",
|
|
|
|
|
"max_tokens": 300,
|
|
|
|
|
"messages": [
|
|
|
|
|
{
|
|
|
|
|
"role": "user",
|
|
|
|
|
"content": "What is my name and where do I work? "
|
|
|
|
|
"Answer based on what you know about me.",
|
|
|
|
|
}
|
|
|
|
|
],
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
assert recall_response.status_code == 200
|
|
|
|
|
|
|
|
|
|
# Check if response mentions the saved info
|
|
|
|
|
content = recall_response.json().get("content", [])
|
|
|
|
|
text = ""
|
|
|
|
|
for block in content:
|
|
|
|
|
if isinstance(block, dict) and block.get("type") == "text":
|
|
|
|
|
text += block.get("text", "")
|
|
|
|
|
|
|
|
|
|
# Should mention at least one of the saved facts
|
|
|
|
|
text_lower = text.lower()
|
|
|
|
|
assert "testuser" in text_lower or "acmecorp" in text_lower or "acme" in text_lower, (
|
|
|
|
|
f"Saved info not recalled: {text}"
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.skipif(not os.environ.get("ANTHROPIC_API_KEY"), reason="ANTHROPIC_API_KEY not set")
|
|
|
|
|
class TestMemoryUserIsolation:
|
|
|
|
|
"""Test that memories are isolated per user."""
|
|
|
|
|
|
|
|
|
|
def test_different_users_have_isolated_memories(self, memory_client, anthropic_api_key):
|
|
|
|
|
"""User A's memories should not appear for User B."""
|
|
|
|
|
timestamp = int(time.time())
|
|
|
|
|
user_a = f"user-a-isolation-{timestamp}"
|
|
|
|
|
user_b = f"user-b-isolation-{timestamp}"
|
|
|
|
|
secret_code = f"SECRETCODE{timestamp}"
|
|
|
|
|
|
|
|
|
|
# Save memory for user A
|
|
|
|
|
save_response = memory_client.post(
|
|
|
|
|
"/v1/messages",
|
|
|
|
|
headers={
|
|
|
|
|
"x-api-key": anthropic_api_key,
|
|
|
|
|
"anthropic-version": "2023-06-01",
|
|
|
|
|
"x-headroom-user-id": user_a,
|
|
|
|
|
},
|
|
|
|
|
json={
|
|
|
|
|
"model": "claude-sonnet-4-20250514",
|
|
|
|
|
"max_tokens": 1000,
|
|
|
|
|
"messages": [
|
|
|
|
|
{
|
|
|
|
|
"role": "user",
|
|
|
|
|
"content": f"Remember my secret code: {secret_code}. "
|
|
|
|
|
"Save this using the memory_save tool.",
|
|
|
|
|
}
|
|
|
|
|
],
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
assert save_response.status_code == 200
|
|
|
|
|
|
|
|
|
|
# Wait for memory to be indexed
|
|
|
|
|
time.sleep(1)
|
|
|
|
|
|
|
|
|
|
# Query as user B - should NOT have access to user A's memory
|
|
|
|
|
response_b = memory_client.post(
|
|
|
|
|
"/v1/messages",
|
|
|
|
|
headers={
|
|
|
|
|
"x-api-key": anthropic_api_key,
|
|
|
|
|
"anthropic-version": "2023-06-01",
|
|
|
|
|
"x-headroom-user-id": user_b,
|
|
|
|
|
},
|
|
|
|
|
json={
|
|
|
|
|
"model": "claude-sonnet-4-20250514",
|
|
|
|
|
"max_tokens": 300,
|
|
|
|
|
"messages": [
|
|
|
|
|
{
|
|
|
|
|
"role": "user",
|
|
|
|
|
"content": "What is my secret code? Search your memory for it.",
|
|
|
|
|
}
|
|
|
|
|
],
|
|
|
|
|
},
|
|
|
|
|
)
|
|
|
|
|
assert response_b.status_code == 200
|
|
|
|
|
|
|
|
|
|
# User B should NOT see user A's secret code
|
|
|
|
|
content = response_b.json().get("content", [])
|
|
|
|
|
text = ""
|
|
|
|
|
for block in content:
|
|
|
|
|
if isinstance(block, dict) and block.get("type") == "text":
|
|
|
|
|
text += block.get("text", "")
|
|
|
|
|
|
|
|
|
|
assert secret_code not in text, f"User B should not see User A's secret: {text}"
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.mark.skipif(not os.environ.get("ANTHROPIC_API_KEY"), reason="ANTHROPIC_API_KEY not set")
|
|
|
|
|
class TestMemoryStats:
|
|
|
|
|
"""Test memory-related stats and health."""
|
|
|
|
|
|
|
|
|
|
def test_health_endpoint_works_with_memory(self, memory_client):
|
|
|
|
|
"""Health endpoint should work when memory is enabled."""
|
|
|
|
|
response = memory_client.get("/health")
|
|
|
|
|
assert response.status_code == 200
|
|
|
|
|
data = response.json()
|
|
|
|
|
assert data.get("status") == "healthy"
|
|
|
|
|
|
|
|
|
|
def test_stats_endpoint_works_with_memory(self, memory_client):
|
|
|
|
|
"""Stats endpoint should work when memory is enabled."""
|
|
|
|
|
response = memory_client.get("/stats")
|
|
|
|
|
assert response.status_code == 200
|
|
|
|
|
data = response.json()
|
|
|
|
|
assert "requests" in data
|