nextcloud / nextcloud/cookbook
Remove all covering annotations on method level
Open
Nobody has claimed this yet.
Backend
enhancement
meta and development
- Dominant language
- HTML
- Stars
- 641
- Forks
- 113
- Avg merge
- 21h 31m
- Merged PRs (30d)
- 26
Description
It is considered a bad habit to cover only part of a class. Better would be to cover the complete class. The current code base should be corrected regarding this practice.
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
No files, tests, or entry points are named. Start by locating method-level covering annotations across the codebase and compare them with the enclosing classes. Done means the current code no longer uses partial class coverage annotations and the relevant test or coverage checks still pass.
Written by the indexing model from the issue text.
Assessment
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100