Skip to content

Commit ae96971

Browse files
Trecekclaude
andcommitted
fix(review): rename vacuous test to reflect actual behavior
The test loaded a recipe with capability gates but claimed to test recipes without them — rename and fix docstring to match reality. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 924fa9a commit ae96971

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

tests/recipe/test_bundled_recipes_dispatch_ready.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -293,9 +293,9 @@ def test_claude_implementation_dispatch_feasible() -> None:
293293
assert result.get("dispatch_feasible") is True
294294

295295

296-
def test_dispatch_feasible_defaults_true_for_recipes_without_capability_gates() -> None:
297-
"""Recipes that do not use capability-gated run_python steps should
298-
default to dispatch_feasible=True regardless of backend overrides."""
296+
def test_dispatch_feasible_true_when_capability_overrides_are_truthy() -> None:
297+
"""Capability-gated recipes report dispatch_feasible=True when all
298+
capability ingredient overrides evaluate as truthy."""
299299
result = load_and_validate(
300300
"implementation",
301301
project_dir=_PROJECT_ROOT,

0 commit comments

Comments
 (0)