Fix Kotlin, JUnit , Mockito: @MockBean does not inject a mocked bean (#1275)#1369
Open
MariusVolkhart wants to merge 2 commits into
Open
Fix Kotlin, JUnit , Mockito: @MockBean does not inject a mocked bean (#1275)#1369MariusVolkhart wants to merge 2 commits into
MariusVolkhart wants to merge 2 commits into
Conversation
Contributor
Author
|
The Java CI failure appears to be due to missing credentials for Sonatype Scan. I'm afraid I don't understand the GraalVM 17 failure. It's in a module I didn't modify, and it appears to have failed several times in the past, though recent builds on the destination build have succeeded. |
Closed
Contributor
|
@MariusVolkhart mege branch 4.10.x, please |
…icronaut-projects#1275) MicronautJunit5Extension was not handling MethodInjectionPoints. Kotlin `lateinit var`s get turned into MethodInjectionPoints by KSP. Accordingly, these fields were not having their mocks aligned. The MicronautSpockExtension already handled MethodInjectionPoint and was the blueprint for the JUnit implementation. Fixes micronaut-projects#1275 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
MariusVolkhart
force-pushed
the
mv/methodInjectionPoint
branch
from
4cdf263 to
969d137
Compare
Contributor
Author
|
Rebase complete |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
MicronautJunit5Extension was not handling MethodInjectionPoints. Kotlin
lateinit vars get turned into MethodInjectionPoints by KSP. Accordingly, these fields were not having their mocks aligned.The MicronautSpockExtension already handled MethodInjectionPoint and was the blueprint for the JUnit implementation.
Fixes #1275