Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add the @Default annotation back as a qualifier for the RestClientDelegateBean #199

Closed
jamezp opened this issue Aug 23, 2023 · 0 comments · Fixed by #198
Closed

Add the @Default annotation back as a qualifier for the RestClientDelegateBean #199

jamezp opened this issue Aug 23, 2023 · 0 comments · Fixed by #198

Comments

@jamezp
Copy link
Contributor

jamezp commented Aug 23, 2023

In #182 the @Default annotation was inadvertently removed from the RestClientDelegateBean.getQualifiers(). There also seems to be no test for checking the default injection point. The @Default annotation should be added back and a test should be added to test that both @Inject RestClient and @Inject @RestClient RestClient work.

See https://issues.redhat.com/browse/WFLY-18366
jamezp added a commit to jamezp/resteasy-microprofile that referenced this issue Aug 23, 2023
@jamezp
[resteasy#199] ]Add back the @default annotation as a qualifier in th…
34554a6 
…e RestClientDelegateBean.

Signed-off-by: James R. Perkins <jperkins@redhat.com>
@jamezp jamezp mentioned this issue Aug 23, 2023
Merged
jamezp added a commit that referenced this issue Aug 23, 2023
@jamezp
Merge pull request #198 from jamezp/fix-default-injection
c703e9d 
[#199] Add back the @default annotation as a qualifier in the RestClientD…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@jamezp