mirror of
https://github.com/headroomlabs-ai/headroom.git
synced 2026-08-27 14:17:10 -04:00
- Fix Anthropic cache token cost formula: input_tokens, cache_read, and cache_write are all separate (not overlapping), so don't subtract cache_write from input_tokens - Default memory user ID to "default" when x-headroom-user-id header is not provided, removing the need for client configuration - Update CLI help text and tests to reflect new default behavior
378 lines
13 KiB
Python
378 lines
13 KiB
Python
"""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."""
|
|
|
|
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."""
|
|
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",
|
|
# Note: NOT setting x-headroom-user-id - should default to "default"
|
|
},
|
|
json={
|
|
"model": "claude-sonnet-4-20250514",
|
|
"max_tokens": 100,
|
|
"messages": [{"role": "user", "content": "Hello"}],
|
|
},
|
|
)
|
|
# Should succeed, not return 400
|
|
assert response.status_code == 200
|
|
|
|
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
|