On Fri, 5 May 2023 12:41:03 GMT, Lukasz Kostyra <lkost...@openjdk.org> wrote:
>> tests/system/src/test/java/test/javafx/embed/swing/SwingNodeMemoryLeakTest.java >> line 80: >> >>> 78: }); >>> 79: >>> 80: JMemoryBuddy.assertCollectable(weakRefArrSN); >> >> On my Mac (MacBook Pro x64 running macOS 13.2.1) I saw a one-time test >> failure here. So there might be a need to wait for the EDT to at least do >> something, possibly followed by a sleep (on the test thread, not on the EDT). > > Interesting, I will add a `runAndWait()` noop and a test thread sleep then. I think you mean `SwingUtilities::invokeAndWait`, right? Btw, I also got a similar one-time failure in `SwingNodeDnDMemoryLeakTest`, so perhaps the same could be added there? ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1121#discussion_r1186070622
