b4fbd6fe9f
Deploy Site / deploy-vercel (push) Has been skipped
Deploy Site / deploy-docs (push) Has been skipped
Build Skills Index / build-index (push) Has been skipped
CI / Deny unrelated histories (push) Has been skipped
CI / Detect affected areas (push) Successful in 27m35s
CI / OSV scan (push) Failing after 4s
CI / Build&Test Docker image (push) Successful in 9s
CI / Supply-chain scan (push) Has been skipped
CI / Lint Docker scripts (push) Failing after 5m13s
CI / Check contributors (push) Failing after 12m8s
CI / Docs Site (push) Failing after 12m8s
CI / TypeScript (push) Failing after 12m8s
CI / Python lints (push) Failing after 12m9s
CI / Python tests (push) Failing after 12m9s
CI / Check uv.lock (push) Failing after 23m22s
CI / CI timing report (push) Has been cancelled
Build Skills Index / trigger-deploy (push) Has been cancelled
CI / All required checks pass (push) Has been cancelled
146 lines
4.7 KiB
Python
146 lines
4.7 KiB
Python
"""Tests for the dynamic schema builder."""
|
|
|
|
from __future__ import annotations
|
|
|
|
from typing import Any, Dict, List, Optional
|
|
|
|
import pytest
|
|
import yaml
|
|
|
|
from agent import video_gen_registry
|
|
from agent.video_gen_provider import VideoGenProvider
|
|
|
|
|
|
@pytest.fixture(autouse=True)
|
|
def _reset_registry():
|
|
video_gen_registry._reset_for_tests()
|
|
yield
|
|
video_gen_registry._reset_for_tests()
|
|
|
|
|
|
@pytest.fixture
|
|
def cfg_home(tmp_path, monkeypatch):
|
|
monkeypatch.setenv("HERMES_HOME", str(tmp_path))
|
|
return tmp_path
|
|
|
|
|
|
def _write_cfg(home, cfg: dict):
|
|
(home / "config.yaml").write_text(yaml.safe_dump(cfg))
|
|
|
|
|
|
class _BothModalitiesProvider(VideoGenProvider):
|
|
"""Supports both text-to-video AND image-to-video (the common case)."""
|
|
|
|
@property
|
|
def name(self) -> str:
|
|
return "both"
|
|
|
|
def is_available(self) -> bool:
|
|
return True
|
|
|
|
def list_models(self) -> List[Dict[str, Any]]:
|
|
return [{"id": "family-a", "modalities": ["text", "image"]}]
|
|
|
|
def default_model(self) -> Optional[str]:
|
|
return "family-a"
|
|
|
|
def capabilities(self) -> Dict[str, Any]:
|
|
return {
|
|
"modalities": ["text", "image"],
|
|
"aspect_ratios": ["16:9", "9:16"],
|
|
"resolutions": ["720p", "1080p"],
|
|
"min_duration": 1,
|
|
"max_duration": 15,
|
|
"supports_audio": True,
|
|
"supports_negative_prompt": True,
|
|
"max_reference_images": 0,
|
|
}
|
|
|
|
def generate(self, prompt, **kwargs):
|
|
return {"success": True}
|
|
|
|
|
|
class _ImageOnlyProvider(VideoGenProvider):
|
|
"""Backend with only image-to-video support (rare but possible)."""
|
|
|
|
@property
|
|
def name(self) -> str:
|
|
return "img-only"
|
|
|
|
def is_available(self) -> bool:
|
|
return True
|
|
|
|
def list_models(self) -> List[Dict[str, Any]]:
|
|
return [{"id": "img-only-v1", "modalities": ["image"]}]
|
|
|
|
def default_model(self) -> Optional[str]:
|
|
return "img-only-v1"
|
|
|
|
def capabilities(self) -> Dict[str, Any]:
|
|
return {"modalities": ["image"], "min_duration": 1, "max_duration": 10}
|
|
|
|
def generate(self, prompt, **kwargs):
|
|
return {"success": True}
|
|
|
|
|
|
class TestDynamicSchemaBuilder:
|
|
def test_no_config_says_so(self, cfg_home):
|
|
from tools.video_generation_tool import _build_dynamic_video_schema
|
|
|
|
desc = _build_dynamic_video_schema()["description"]
|
|
assert "No video backend is configured" in desc
|
|
assert "hermes tools" in desc
|
|
|
|
def test_generic_description_keeps_edit_extend_out_of_surface(self, cfg_home):
|
|
from tools.video_generation_tool import _build_dynamic_video_schema, _GENERIC_DESCRIPTION
|
|
|
|
desc = _build_dynamic_video_schema()["description"]
|
|
assert "Video edit/extend workflows are not part of this unified surface" in desc
|
|
assert "operation='edit'" not in _GENERIC_DESCRIPTION
|
|
assert "operation='extend'" not in _GENERIC_DESCRIPTION
|
|
|
|
def test_both_modalities_advertises_auto_routing(self, cfg_home):
|
|
from tools.video_generation_tool import _build_dynamic_video_schema
|
|
|
|
_write_cfg(cfg_home, {"video_gen": {"provider": "both"}})
|
|
video_gen_registry.register_provider(_BothModalitiesProvider())
|
|
|
|
import hermes_cli.plugins as plugins_module
|
|
saved = plugins_module._ensure_plugins_discovered
|
|
plugins_module._ensure_plugins_discovered = lambda *a, **k: None
|
|
try:
|
|
desc = _build_dynamic_video_schema()["description"]
|
|
finally:
|
|
plugins_module._ensure_plugins_discovered = saved
|
|
|
|
assert "Active backend: Both" in desc
|
|
assert "text-to-video" in desc and "image-to-video" in desc
|
|
assert "routes automatically" in desc
|
|
assert "operations supported" not in desc
|
|
|
|
def test_image_only_model_warns_about_required_image_url(self, cfg_home):
|
|
from tools.video_generation_tool import _build_dynamic_video_schema
|
|
|
|
_write_cfg(cfg_home, {"video_gen": {"provider": "img-only"}})
|
|
video_gen_registry.register_provider(_ImageOnlyProvider())
|
|
|
|
import hermes_cli.plugins as plugins_module
|
|
saved = plugins_module._ensure_plugins_discovered
|
|
plugins_module._ensure_plugins_discovered = lambda *a, **k: None
|
|
try:
|
|
desc = _build_dynamic_video_schema()["description"]
|
|
finally:
|
|
plugins_module._ensure_plugins_discovered = saved
|
|
|
|
assert "image-to-video only" in desc
|
|
assert "image_url is REQUIRED" in desc
|
|
|
|
def test_builder_wired_into_registry(self):
|
|
from tools.registry import discover_builtin_tools, registry
|
|
|
|
discover_builtin_tools()
|
|
entry = registry._tools["video_generate"]
|
|
assert entry.dynamic_schema_overrides is not None
|
|
out = entry.dynamic_schema_overrides()
|
|
assert "description" in out
|