Skip to content

Remove @InjectMocks from initialized fields#758

Merged
timtebeek merged 5 commits into
mainfrom
remove-injectmocks-from-initialized-fields
Jul 5, 2025
Merged

Remove @InjectMocks from initialized fields#758
timtebeek merged 5 commits into
mainfrom
remove-injectmocks-from-initialized-fields

Conversation

@timtebeek

Copy link
Copy Markdown
Member

@timtebeek

Copy link
Copy Markdown
Member Author

Now failing the easy mock migration, so likely we need to adjust still.

@timtebeek timtebeek self-assigned this Jul 5, 2025
@timtebeek timtebeek added enhancement New feature or request mockito labels Jul 5, 2025
@timtebeek timtebeek moved this from In Progress to Ready to Review in OpenRewrite Jul 5, 2025
@timtebeek timtebeek merged commit 79e3b3f into main Jul 5, 2025
2 checks passed
@timtebeek timtebeek deleted the remove-injectmocks-from-initialized-fields branch July 5, 2025 18:20
@github-project-automation github-project-automation Bot moved this from Ready to Review to Done in OpenRewrite Jul 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request mockito

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

MockitoBestPractices removes @InjectMocks instance creation for interfaces

1 participant