Add a pure integration test to `configure_for`/`test_update_recipe.py`
Open
@chrisbillowsMO is already working on this.
Since Feb 27, 2024.
enhancement
quality assurance
technical debt
testing
- Dominant language
- Python
- Stars
- 5
- Forks
- 3
- Avg merge
- 5d 6h
- Merged PRs (30d)
- 8
Description
During #18 / #227 an "integration test" was added to `configure_for`/`test_update_recipe.py` as `test_main`.
A pure integration test should be added. This should test the two functions `create_recipe` and `write_recipe` working together.
`test_main` should then be rebadged as a functional test.
See this comment on #227 from @ehogan:
> We agreed when we spoke earlier that it made sense to add the integration test as a separate test, and keep the main test as a "functional" test. Happy for this to be done via a separate issue
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.