-
Notifications
You must be signed in to change notification settings - Fork 74
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
feat(jans-auth-server):get up tests updating arquillian and jetty-embedded issue 1247 #2017
feat(jans-auth-server):get up tests updating arquillian and jetty-embedded issue 1247 #2017
Conversation
…s-updating-arquillian-and-jetty
…s-updating-arquillian-and-jetty
} | ||
|
||
public ClientService getClientService() { | ||
return (ClientService) HM_TEST_INJECTOR.get("ClientService"); |
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.
Can you check whether CdiUtil.bean(ClientService.class)
will work instead of alien static holder? If it will work we have to drop this map.
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.
I have applied CdiUtil.bean, it is working thanks.
[Jans authentication server parent] Kudos, SonarCloud Quality Gate passed! |
Prepare
Description
Updating arquillian version, to allow running embedded jetty tests.
Target issue
#1247
closes #1247
Implementation Details
upgrade version arquillian-jetty-embedded-11 1.0.0.CR4
upgrade version arquillian test 1.7.0.Alpha10
allow using injection inside tests
Test and Document the changes