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
[WFCORE-6198] DelegatingModelControllerClient.close should handle a missing client #5350
Conversation
Core -> Full Integration Build 11942 outcome was FAILURE using a merge of 36bfd41 Failed tests
|
Core -> Full Integration Build 12140 outcome was FAILURE using a merge of 36bfd41 Failed tests
|
Core -> Full Integration Build 11943 outcome was FAILURE using a merge of 36bfd41 Failed tests
|
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.
@soul2zimate, one doubt, looking at the anonymous implementation:
public DelegatingModelControllerClient(final ModelControllerClient delegate) {
this(new DelegateProvider() {
@Override
public ModelControllerClient getDelegate() {
return delegate;
}
});
}
Since it doesn't throw an ISE, shouldn't at least this anonymous implementation check the delegate
argument is not null to fully satisfice the getDelegate() API?
|
Core -> WildFly Preview Integration Build 12079 outcome was FAILURE using a merge of 07801d8 Failed tests
|
Core -> Full Integration Build 12147 outcome was FAILURE using a merge of 07801d8 Failed tests
|
07801d8
to
4a1757d
Compare
Issue: https://issues.redhat.com/browse/WFCORE-6198