<AWT Dev> [9] Review request for 8014755: [TEST_BUG] frames didn't closed after execution some awt/dnd/ tests

[Petr Pchelko]

Hello, Oleg.

1. I'm not sure your about your new API in ProcessCommunicator. 
First of all, now the communicator can't be used twice in a single test to open and then close a bunch of processes. 
Also, you need to modify the tests themselves. 
I think it would be better to do the termination implicitly: you can get jtreg property with execution timeout, start a timer 
thread when you create a process and kill it after the timeout. In case doWaitFor finishes you kill the timer thread. 

2. We normally remove .html if it's possible to convert test to a standalone app.

3. NoFormatsCrashTest lacks .java file.

With best regards. Petr.

On 13 мая 2014 г., at 14:55, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:

> Hi. Oleg.
> A few notes.
> - Some tests contains empty/non-rethrow catch blocks. I guess we should rethrow an exception and the test should fail in this case.
> - Some tests use System.exit() which should be replaced by the exception.
> 
> On 5/9/14 7:32 PM, Oleg Pekhovskiy wrote:
>> Hi all,
>> 
>> please review the fix
>> http://cr.openjdk.java.net/~bagiras/9/8014755.1/
>> for
>> https://bugs.openjdk.java.net/browse/JDK-8014755
>> 
>> These tests were moved from the closed workspace.
>> 
>> The main idea of fix is to force termination of child JVM if it's not exited automatically after the test sequence.
>> 
>> Tests accuracy was improved to cover all error cases.
>> 
>> destroyProcess() method was added to
>> test.java.awt.regtesthelpers.process.ProcessCommunicator
>> for termination of child JVM when needed.
>> 
>> Thanks,
>> Oleg
>> 
> 
> 
> -- 
> Best regards, Sergey.
>