RFR: 8285503: SwingNodeDnDMemoryLeakTest::testSwingNodeMemoryLeak fails sometimes [v2]

Mon Nov 14 14:03:03 UTC 2022

On Mon, 14 Nov 2022 07:04:36 GMT, Ajit Ghaisas <aghaisas at openjdk.org> wrote:

>> Root cause: The test unreliability comes from the 250ms sleep between `System.gc()` calls.
>> Other system tests such as `TabPaneHeaderLeakTest`, `AccordionTitlePaneLeakTest` and `ShapeViewOrderLeakTest` etc sleep for 500ms between `System.gc()` calls.
>> 
>> Fix:
>> - Increased the sleep time to 500ms between `System.gc()` calls.
>> 
>> Testing: This test used to fail on my macBook all the time if ran as part of a full system test run. I executed full test runs 5 times with this change and no failure was observed.
>
> Ajit Ghaisas has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Revert total swingnode count to 10 in test

Marked as reviewed by kcr (Lead).

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

PR: https://git.openjdk.org/jfx/pull/946