[
https://issues.apache.org/jira/browse/FLINK-7102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16092684#comment-16092684
]
ASF GitHub Bot commented on FLINK-7102:
---------------------------------------
Github user pnowojski commented on the issue:
https://github.com/apache/flink/pull/4255
ok, I missed that you didn't fail if the expected exception didn't happen
:) Now it's LGTM
> improve ClassLoaderITCase
> -------------------------
>
> Key: FLINK-7102
> URL: https://issues.apache.org/jira/browse/FLINK-7102
> Project: Flink
> Issue Type: Bug
> Components: Tests
> Affects Versions: 1.4.0
> Reporter: Nico Kruber
> Assignee: Nico Kruber
>
> {{ClassLoaderITCase}}...
> * unnecessarily runs multiple tests in a single test case
> * {{#testDisposeSavepointWithCustomKvState()}} does not cancel its job (thus
> the order of execution of test cases defines the outcome)
> * uses {{e.getCause().getCause()}} which may cause {{NullPointerException}}s
> hiding the original error
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
