[Increase Test Coverage] Batch 126
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 15.7k
- Forks
- 5.6k
- Avg merge
- 2d 44m
- Merged PRs (30d)
- 80
Description
Increase the code coverage percent on the following files to at least 80%.
Please be aware that currently the percentage might be inaccurate if the module uses salt due to #64696
File | Percent
salt/modules/out.py 40
salt/modules/pam.py 64
salt/modules/rest_pkg.py 36
salt/modules/rest_sample_utils.py 60
salt/modules/rsync.py 53
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
Review the five listed modules under salt/modules/ and check their existing test coverage before choosing a starting point. Increase coverage for out.py, pam.py, rest_pkg.py, rest_sample_utils.py, and rsync.py to at least 80%, while considering the warning that salt may make coverage inaccurate due to #64696. Verify the resulting coverage percentages for each file.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100