openrewrite / openrewrite/rewrite-testing-frameworks
Remove `@VisibleForTesting` if variable is accessed from production code
Open
Nobody has claimed this yet.
- #689 by @JohannisK — closed without merging
recipe
- 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
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 issue context and closed pull request #689 first, then locate the Java variable that is accessed from production code. Confirm whether the annotation is misleading in that usage and remove it where appropriate. Done means the annotation no longer marks production-accessed code and the relevant tests still pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- testing-qa
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100