theupdateframework / theupdateframework/python-tuf
Revise test_updater_ng.py
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 1.7k
- Forks
- 304
- Avg merge
- 1d 2h
- Merged PRs (30d)
- 17
Description
Description of issue or feature request:
Currently test_updater_ng.py is the single place where ngclient/updater.py is tested with an HTTP server instance serving files from the file system and not with the in-memory files served by RepositorySimulator. The tests included are not well thought out and were added with the goal of achieving acceptable test coverage.
Revise the existing tests and determine what is worth keeping and/or what should be added to build a meaningful set of integration tests of Updater + HTTP server.
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 with test_updater_ng.py and compare its filesystem-backed HTTP server setup with the in-memory files served by RepositorySimulator. Read ngclient/updater.py to understand the Updater and HTTP-server integration covered by the existing tests. Done means retaining worthwhile coverage and adding or revising tests into a meaningful integration-test set.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- testing-qa
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100