RFR: 8328629: JUnit test without a timeout value can hang indefinitely [v2]

Fri Sep 20 18:21:05 UTC 2024

On Fri, 20 Sep 2024 16:31:58 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> On closer inspection, I don't think this should be removed as part of PR #1569, since that file is already using JUnit 5 and is not touched by your PR.
>> 
>> Either I can remove the method with this PR or we just leave it for a follow-up. Removing it with this PR seems easiest, since I am already modifying that method as part of this PR (along with other methods that catch and ignore InterruptedException from sleep).
>
> I agree with removing the method in this PR.  Thanks!

Done. Thanks.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1575#discussion_r1769017784