RFR: 8346929: runtime/ClassUnload/DictionaryDependsTest.java fails with "Test failed: should be unloaded" [v2]

[David Holmes]

On Mon, 13 Jan 2025 04:27:15 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> I was trying to explain why interfaceName is not in the List.  I could have added it and tested that it's not unloaded but then the test would take too long.  It may be more correct to add it and make triggerUnloading try to unload it and fail to prove it cannot be unloaded.  But like I said in the comment, it would make the test take too long.
>
> So `interfaceName` is not expected to be unloaded so `triggerUnloading` would run the full 2 seconds failing to detect it is unloaded, if it was passed in. But surely you would just use WB to test if a class is still loaded as expected?

Seems pretty evident from the `failIf` calls what is expected. The comment is just confusing to me.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/22974#discussion_r1912653924