-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
QuarkusComponentTest: support test method parameter injection #38379
QuarkusComponentTest: support test method parameter injection #38379
Conversation
mkouba
commented
Jan 24, 2024
- params annotated with SkipInject are never injected
3823a0f
to
40b7514
Compare
40b7514
to
8507b22
Compare
@manovotn I know that you have some experience with test method param injection from |
test-framework/common/src/main/java/io/quarkus/test/InjectMock.java
Outdated
Show resolved
Hide resolved
🙈 The PR is closed and the preview is expired. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just a few suggestions that would improve documentation somewhat IMHO.
1cf5af6
to
85bd285
Compare
...it5-component/src/main/java/io/quarkus/test/component/QuarkusComponentTestConfiguration.java
Outdated
Show resolved
Hide resolved
test-framework/junit5-component/src/main/java/io/quarkus/test/component/SkipInject.java
Outdated
Show resolved
Hide resolved
.../junit5-component/src/main/java/io/quarkus/test/component/QuarkusComponentTestExtension.java
Outdated
Show resolved
Hide resolved
85bd285
to
3cc46ef
Compare
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM now!
...it5-component/src/main/java/io/quarkus/test/component/QuarkusComponentTestConfiguration.java
Outdated
Show resolved
Hide resolved
.../junit5-component/src/main/java/io/quarkus/test/component/QuarkusComponentTestExtension.java
Outdated
Show resolved
Hide resolved
.../junit5-component/src/main/java/io/quarkus/test/component/QuarkusComponentTestExtension.java
Outdated
Show resolved
Hide resolved
LGTM otherwise. |
- params annotated with SkipInject are never injected Co-authored-by: Ladislav Thon <[email protected]>
3cc46ef
to
f853e4c
Compare
This comment has been minimized.
This comment has been minimized.
✔️ The latest workflow run for the pull request has completed successfully. It should be safe to merge provided you have a look at the other checks in the summary. You can consult the Develocity build scans. |