Skip to content

Commit 4aeb7d9

Browse files
committed
fix(review): replace vacuous pytest.skip with explicit backend setup
1 parent eb6076d commit 4aeb7d9

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

tests/server/test_tools_recipe.py

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -651,6 +651,7 @@ async def test_validate_recipe_codex_backend_injects_overrides(tool_ctx_kitchen_
651651
requires backend_supports_git_write=false, which drives pruning of
652652
codex-incompatible steps.
653653
"""
654+
from types import SimpleNamespace
654655
from unittest.mock import MagicMock, patch
655656

656657
script = tmp_path / "codex_recipe.yaml"
@@ -660,7 +661,13 @@ async def test_validate_recipe_codex_backend_injects_overrides(tool_ctx_kitchen_
660661
tool_ctx_kitchen_open.recipes.validate_from_path.return_value = {"valid": True}
661662

662663
if tool_ctx_kitchen_open.backend is None:
663-
pytest.skip("tool_ctx has no backend configured")
664+
backend = MagicMock()
665+
backend.name = "codex"
666+
backend.capabilities = SimpleNamespace(
667+
git_metadata_writable=False,
668+
supports_tool_list_changed=True,
669+
)
670+
tool_ctx_kitchen_open.backend = backend
664671

665672
with patch(
666673
"autoskillit.server.tools.tools_recipe._get_ctx_or_none",

0 commit comments

Comments
 (0)