Skip to content

Commit 7c4f8f3

Browse files
Trecekclaude
andcommitted
fix(review): restore error-free assertion in test_get_recipe_uses_project_dir
The assertion 'assert "\"error\"" not in result' was removed in 05bda5b without replacement. The new test_recipe_resource_returns_error_for_invalid_recipe covers the error path via mock but not the happy-path real-recipe assertion. Restore the error-free check to prevent regressions where get_recipe returns both the recipe name and an error key simultaneously. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 5a67beb commit 7c4f8f3

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

tests/server/test_tools_kitchen_gate_features.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -373,6 +373,9 @@ def test_get_recipe_uses_project_dir(tmp_path, monkeypatch):
373373

374374
result = get_recipe("test-get-recipe-project-dir")
375375

376+
assert '"error"' not in result, (
377+
f"get_recipe failed to find recipe in project_dir={different_dir}. Result: {result}"
378+
)
376379
assert "test-get-recipe-project-dir" in result, (
377380
f"get_recipe did not use project_dir for recipe lookup. Result: {result}"
378381
)

0 commit comments

Comments
 (0)