circuitforge-core/tests/test_llm_router.py
pyr0ball fb3a4c697d
Some checks failed
CI / test (push) Waiting to run
Mirror / mirror (push) Has been cancelled
Release — PyPI / release (push) Has been cancelled
feat(llm): v0.20.0 — LLMRouter dict init + Ollama embed preflight (closes #59, #60)
- LLMRouter.__init__ now accepts a Path | dict; pagepiper ingest scripts
  pass a runtime-constructed config dict instead of a temp file
- _check_ollama_model_pulled() preflight on embed(): checks /api/tags once
  per backend URL and raises RuntimeError("...Fix: ollama pull <model>")
  when the configured embedding model is not pulled; silently skips for
  non-Ollama backends (vLLM, etc.) that don't expose /api/tags
- 6 new tests: dict init paths (x2) + preflight scenarios (x4)
- Existing embed tests updated to mock requests.get to avoid live Ollama calls
2026-05-05 14:59:49 -07:00

336 lines
12 KiB
Python

from unittest.mock import MagicMock, patch
import pytest
from circuitforge_core.llm import LLMRouter
def _make_router(config: dict) -> LLMRouter:
"""Build a router from an in-memory config dict (bypass file loading)."""
router = object.__new__(LLMRouter)
router.config = config
return router
def test_complete_uses_first_reachable_backend():
router = _make_router(
{
"fallback_order": ["local"],
"backends": {
"local": {
"type": "openai_compat",
"base_url": "http://localhost:11434/v1",
"model": "llama3",
"supports_images": False,
}
},
}
)
mock_client = MagicMock()
mock_client.chat.completions.create.return_value = MagicMock(
choices=[MagicMock(message=MagicMock(content="hello"))]
)
with (
patch.object(router, "_is_reachable", return_value=True),
patch("circuitforge_core.llm.router.OpenAI", return_value=mock_client),
):
result = router.complete("say hello")
assert result == "hello"
def test_complete_falls_back_on_unreachable_backend():
router = _make_router(
{
"fallback_order": ["unreachable", "working"],
"backends": {
"unreachable": {
"type": "openai_compat",
"base_url": "http://nowhere:1/v1",
"model": "x",
"supports_images": False,
},
"working": {
"type": "openai_compat",
"base_url": "http://localhost:11434/v1",
"model": "llama3",
"supports_images": False,
},
},
}
)
mock_client = MagicMock()
mock_client.chat.completions.create.return_value = MagicMock(
choices=[MagicMock(message=MagicMock(content="fallback"))]
)
def reachable(url):
return "nowhere" not in url
with (
patch.object(router, "_is_reachable", side_effect=reachable),
patch("circuitforge_core.llm.router.OpenAI", return_value=mock_client),
):
result = router.complete("test")
assert result == "fallback"
def test_complete_raises_when_all_backends_exhausted():
router = _make_router(
{
"fallback_order": ["dead"],
"backends": {
"dead": {
"type": "openai_compat",
"base_url": "http://nowhere:1/v1",
"model": "x",
"supports_images": False,
}
},
}
)
with patch.object(router, "_is_reachable", return_value=False):
with pytest.raises(RuntimeError, match="exhausted"):
router.complete("test")
def test_try_cf_orch_alloc_import_path():
"""Verify lazy import points to circuitforge_orch, not circuitforge_core.resources."""
import inspect
from circuitforge_core.llm import router as router_module
src = inspect.getsource(router_module.LLMRouter._try_cf_orch_alloc)
assert "circuitforge_orch.client" in src
assert "circuitforge_core.resources.client" not in src
def test_embed_returns_vectors_from_openai_compat_backend():
router = _make_router(
{
"fallback_order": ["ollama"],
"backends": {
"ollama": {
"type": "openai_compat",
"base_url": "http://localhost:11434/v1",
"model": "mistral:7b",
"embedding_model": "nomic-embed-text",
"supports_images": False,
}
},
}
)
mock_client = MagicMock()
mock_client.embeddings.create.return_value = MagicMock(
data=[
MagicMock(embedding=[0.1, 0.2, 0.3]),
MagicMock(embedding=[0.4, 0.5, 0.6]),
]
)
with (
patch.object(router, "_is_reachable", return_value=True),
patch("circuitforge_core.llm.router.requests.get", return_value=MagicMock(status_code=404)),
patch("circuitforge_core.llm.router.OpenAI", return_value=mock_client),
):
result = router.embed(["hello world", "fireball rules"])
assert result == [[0.1, 0.2, 0.3], [0.4, 0.5, 0.6]]
mock_client.embeddings.create.assert_called_once_with(
model="nomic-embed-text",
input=["hello world", "fireball rules"],
)
def test_embed_uses_chat_model_when_no_embedding_model_configured():
router = _make_router(
{
"fallback_order": ["ollama"],
"backends": {
"ollama": {
"type": "openai_compat",
"base_url": "http://localhost:11434/v1",
"model": "llama3",
"supports_images": False,
}
},
}
)
mock_client = MagicMock()
mock_client.embeddings.create.return_value = MagicMock(
data=[MagicMock(embedding=[0.9, 0.8])]
)
with (
patch.object(router, "_is_reachable", return_value=True),
patch("circuitforge_core.llm.router.requests.get", return_value=MagicMock(status_code=404)),
patch("circuitforge_core.llm.router.OpenAI", return_value=mock_client),
):
router.embed(["test"])
call_kwargs = mock_client.embeddings.create.call_args
assert call_kwargs.kwargs["model"] == "llama3"
def test_embed_skips_non_openai_compat_backends():
router = _make_router(
{
"fallback_order": ["anthropic", "ollama"],
"backends": {
"anthropic": {
"type": "anthropic",
"enabled": True,
"model": "claude-haiku-4-5-20251001",
"api_key_env": "ANTHROPIC_API_KEY",
"supports_images": True,
},
"ollama": {
"type": "openai_compat",
"base_url": "http://localhost:11434/v1",
"model": "nomic-embed-text",
"supports_images": False,
},
},
}
)
mock_client = MagicMock()
mock_client.embeddings.create.return_value = MagicMock(
data=[MagicMock(embedding=[0.1])]
)
mock_openai = MagicMock(return_value=mock_client)
with (
patch.object(router, "_is_reachable", return_value=True),
patch("circuitforge_core.llm.router.requests.get", return_value=MagicMock(status_code=404)),
patch("circuitforge_core.llm.router.OpenAI", mock_openai),
):
result = router.embed(["hello"])
assert result == [[0.1]]
# Only ollama reached the OpenAI constructor; anthropic was skipped by type check
mock_openai.assert_called_once()
def test_embed_raises_when_all_backends_exhausted():
router = _make_router(
{
"fallback_order": ["dead"],
"backends": {
"dead": {
"type": "openai_compat",
"base_url": "http://nowhere:1/v1",
"model": "x",
"supports_images": False,
}
},
}
)
with patch.object(router, "_is_reachable", return_value=False):
with pytest.raises(RuntimeError, match="exhausted"):
router.embed(["test"])
# ── #59: LLMRouter dict init ──────────────────────────────────────────────────
def test_init_accepts_inline_dict():
config = {
"fallback_order": ["local"],
"backends": {
"local": {
"type": "openai_compat",
"base_url": "http://localhost:11434/v1",
"model": "llama3",
"supports_images": False,
}
},
}
router = LLMRouter(config)
assert router.config["fallback_order"] == ["local"]
assert "local" in router.config["backends"]
def test_init_dict_is_used_directly():
config = {"fallback_order": [], "backends": {}}
router = LLMRouter(config)
assert router.config is config
# ── #60: Ollama embedding model preflight ─────────────────────────────────────
def _ollama_backend(model: str = "nomic-embed-text") -> dict:
return {
"fallback_order": ["ollama"],
"backends": {
"ollama": {
"type": "openai_compat",
"base_url": "http://localhost:11434/v1",
"embedding_model": model,
"model": "mistral:7b",
"supports_images": False,
}
},
}
def test_embed_raises_actionable_error_when_model_not_pulled():
router = _make_router(_ollama_backend("nomic-embed-text"))
tags_resp = MagicMock(status_code=200)
tags_resp.json.return_value = {"models": [{"name": "mistral:latest"}]}
with (
patch.object(router, "_is_reachable", return_value=True),
patch("circuitforge_core.llm.router.requests.get", return_value=tags_resp),
):
with pytest.raises(RuntimeError, match='ollama pull nomic-embed-text'):
router.embed(["hello"])
def test_embed_proceeds_when_model_is_pulled():
router = _make_router(_ollama_backend("nomic-embed-text"))
tags_resp = MagicMock(status_code=200)
tags_resp.json.return_value = {
"models": [{"name": "nomic-embed-text:latest"}, {"name": "mistral:latest"}]
}
mock_client = MagicMock()
mock_client.embeddings.create.return_value = MagicMock(
data=[MagicMock(embedding=[0.1, 0.2])]
)
with (
patch.object(router, "_is_reachable", return_value=True),
patch("circuitforge_core.llm.router.requests.get", return_value=tags_resp),
patch("circuitforge_core.llm.router.OpenAI", return_value=mock_client),
):
result = router.embed(["hello"])
assert result == [[0.1, 0.2]]
def test_embed_skips_preflight_when_tags_endpoint_unavailable():
"""Non-Ollama backends (vLLM, etc.) don't expose /api/tags — check must be silent."""
router = _make_router(_ollama_backend("custom-embed"))
tags_resp = MagicMock(status_code=404)
mock_client = MagicMock()
mock_client.embeddings.create.return_value = MagicMock(
data=[MagicMock(embedding=[0.5])]
)
with (
patch.object(router, "_is_reachable", return_value=True),
patch("circuitforge_core.llm.router.requests.get", return_value=tags_resp),
patch("circuitforge_core.llm.router.OpenAI", return_value=mock_client),
):
result = router.embed(["hello"])
assert result == [[0.5]]
def test_ollama_tags_cache_is_hit_only_once():
router = _make_router(_ollama_backend("nomic-embed-text"))
tags_resp = MagicMock(status_code=200)
tags_resp.json.return_value = {"models": [{"name": "nomic-embed-text:latest"}]}
mock_client = MagicMock()
mock_client.embeddings.create.return_value = MagicMock(
data=[MagicMock(embedding=[0.1])]
)
with (
patch.object(router, "_is_reachable", return_value=True),
patch("circuitforge_core.llm.router.requests.get", return_value=tags_resp) as mock_get,
patch("circuitforge_core.llm.router.OpenAI", return_value=mock_client),
):
router.embed(["first"])
router.embed(["second"])
# /api/tags is called once (cache hit on second embed)
tags_calls = [c for c in mock_get.call_args_list if "api/tags" in str(c)]
assert len(tags_calls) == 1