1) Getting rid of the deleteOnExit - and the *need* for it is a good thing here as it is run when the VM exits. This is not a friendly thing to do when an agentvm is being used as it means that only after several hours when the whole test run is done will the files be deleted. Perhaps this test should be run in othervm mode because of what it is doing.

2) It is possible that this test needs more than the default timeout even after the proposed change.
It may have time to complete on some systems .. but not others.

-phil.

On 02/20/2018 08:24 AM, Jayathirth D V wrote:
Hi Prasanta,

I ran the test 3 times and it still times out and throws the error.
I have attached timeout log for more info.

Thanks,
Jay

-----Original Message-----
From: Prasanta Sadhukhan
Sent: Tuesday, February 20, 2018 9:08 PM
To: swing-dev@openjdk.java.net
Subject: <Swing Dev> [11] 
JDK-8198005:javax/swing/JFileChooser/7199708/bug7199708.java throws error

Hi All,

Please review a test fix for an issue whereby it is seen that the test times 
out on windows 7.
Issue seems to stem from calling File.deleteOnExit() for large number of files, 
around 30000, which is causing the test to timeout.
Fix is to call File.delete(path) which causes the test to pass. It is also 
verified that the new test also crashes if 7199708 fix is backed out.

Bug: https://bugs.openjdk.java.net/browse/JDK-8198005
webrev: http://cr.openjdk.java.net/~psadhukhan/8198005/webrev.00/

Regards
Prasanta

Reply via email to