RFR: 8348423: [TestBug] stress test Nodes initialization from a background thread [v8]

Kevin Rushforth kcr at openjdk.org
Wed Feb 5 23:36:18 UTC 2025


On Wed, 5 Feb 2025 21:33:52 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> tests/system/src/test/java/test/robot/javafx/scene/NodeInitializationStressTest.java line 782:
>> 
>>> 780:             new Thread(() -> {
>>> 781:                 try {
>>> 782:                     while (running.get()) {
>> 
>> Maybe exit early if there is a failure, e.g., `while (running.get() && !failed.get())`?
>> 
>> In order to be effective, the `UncaughtExceptionHandler` would need to interrupt the main thread (so that `sleep(DURATION)` terminates early), so we would need to register the main thread with the `UncaughtExceptionHandler`.
>> 
>> This could be a follow-on if we decide it is helpful (I'm not sure it's worth doing).
>
> Probably not worth doing it - the test is relatively quick (even after we add all other  Nodes).

Also, this would only bail out of a single test method, so very likely not worth it.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1690#discussion_r1943827541


More information about the openjfx-dev mailing list