Using `winreg.DeleteTree` in `test_winreg`
Open
Nobody has claimed this yet.
OS-windows
tests
type-refactor
- Dominant language
- Python
- Stars
- 77.2k
- Forks
- 35.9k
- PR merge metrics
- PR metrics pending
Description
There is a recursive function delete_tree in test_winreg to delete the entire key. We can use the newly added winreg.DeleteTree to do this for more robust code and to simplify our code base.
Linked PRs
- gh-142104
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.
Research direction
Start in test_winreg by reading the recursive delete_tree helper and the nearby registry tests. Check the linked gh-142104 work before proceeding, then confirm the tests still remove entire registry keys robustly using winreg.DeleteTree and that the redundant helper is gone.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- operating-systems, testing-qa
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 25/100