openrewrite / openrewrite/rewrite-testing-frameworks
Remove `@VisibleForTesting` if code is not used in tests
Open
Nobody has claimed this yet.
recipe
uses-dataflow
- Dominant language
- Java
- Stars
- 100
- Forks
- 105
- Avg merge
- 2h 25m
- Merged PRs (30d)
- 9
Description
Leaving the VisibleForTesting is misleading in this scenario
Similar SPEC : RSPEC-5803
Similar issue https://github.com/openrewrite/rewrite-testing-frameworks/issues/224
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 by searching the repository for @VisibleForTesting and inspect each usage alongside its tests; no specific file or test is named in the issue. Determine which annotated code is not used by tests, then verify that the unnecessary annotations are removed without changing behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100