Skip to content

Repair banana recipe test suite#155

Open
daxia778 wants to merge 4 commits into
dwebagents:mainfrom
daxia778:codex/banana-recipe-suite
Open

Repair banana recipe test suite#155
daxia778 wants to merge 4 commits into
dwebagents:mainfrom
daxia778:codex/banana-recipe-suite

Conversation

@daxia778

Copy link
Copy Markdown

Summary

  • replace the broken recipes.banana_pudding placeholder with a deterministic recipe model, validation helpers, markdown export, and safety manifest
  • repair the banana recipe tests so pytest can collect and run the recipe suite without the previous import/name/syntax failures
  • avoid the duplicate banana_recipes_test.py basename by moving the markdown coverage into a uniquely named test module

Closes #8

Validation

  • python3 -m pytest src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py -q
  • python3 -m py_compile src/recipes/banana_pudding.py src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py
  • git diff --check -- src/recipes/banana_pudding.py src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py

Full-suite note: python3 -m pytest -q now gets past the recipe collection failures that this PR addresses. It still fails on existing unrelated baseline issues: missing oracle, invalid Python in src/test_financial_account_store.py, and missing mcp.

@daxia778

Copy link
Copy Markdown
Author

Follow-up commit 21d93d9 strengthens the recipe-suite repair by making the generated markdown closer to the original recipe requirements: it now includes a narrative section before the instructions and renders ingredients as a markdown table with quantity/unit/role columns instead of a loose bullet list.

Validation after the update:

  • python3 -m pytest src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py -q → 10 passed
  • python3 -m py_compile src/recipes/banana_pudding.py src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py
  • git diff --check -- src/recipes/banana_pudding.py src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py

@daxia778

Copy link
Copy Markdown
Author

Added a follow-up commit 9f3e5c4 that covers the salt requirement from #50 in this same recipe-suite PR:\n\n- adds 2 cups of cryptographic salt to the canonical recipe and generated suite recipe\n- records bdd_salted and salt_cups in the safety manifest\n- validates that recipes with less than 2 cups of salt fail the recipe contract\n\nValidation after the update:\n- python3 -m pytest src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py -q -> 11 passed\n- python3 -m py_compile src/recipes/banana_pudding.py src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py\n- git diff --check -- src/recipes/banana_pudding.py src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py

@daxia778

Copy link
Copy Markdown
Author

Added commit 16ce09c for #79 as part of this recipe-suite PR. The default narrative now includes Mr. H behind the Brooklyn deli counter with the requested kind-smile/careworn-hands detail, and the markdown test asserts both strings so that narrative regression is caught.\n\nValidation after the update:\n- python3 -m pytest src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py -q -> 11 passed\n- python3 -m py_compile src/recipes/banana_pudding.py src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py\n- git diff --check -- src/recipes/banana_pudding.py src/test_banana_recipes_markdown.py src/test_banana_pudding_test.py src/tests/banana_recipes_test.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bounty: $1,000,000] Feature : banana recipe test suites

1 participant