-
Notifications
You must be signed in to change notification settings - Fork 2
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
Bug fix for TEF exclusion not honored #17
base: main
Are you sure you want to change the base?
Conversation
@@ -107,9 +107,9 @@ | |||
</build> | |||
|
|||
<properties> | |||
<revision>0.1.5</revision> | |||
<revision>0.2.0</revision> |
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.
This should be a minor version bump, since its a bugfix. Refer to semver for guidance.
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.
This is also JDK17 compliant change.
Hence the bump to major version seemed relevant.
// The type itself is parameterized | ||
return ((ParameterizedTypeImpl) genericSuperClass.getActualTypeArguments()[0]).getRawType(); | ||
return (Class<?>)((ParameterizedType) genericSuperClass.getActualTypeArguments()[0]).getRawType(); |
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.
Why is this additional cast to Class<?>
needed?
|
||
assertNotNull(flow); | ||
|
||
assertEquals(4, flow.getBizlogics().size()); |
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 also assert on the order of bizlogics?
|
||
@Override | ||
public List<Class<? extends IBizlogic>> exclusions() { | ||
return ImmutableList.of(DataAdapterBizlogic2.class); |
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.
Given adapter2 was never added to the flow from any place, this exclusion does not test things properly. I think you should add DataAdapterBizlogic2
in adapters
No description provided.