<AWT Dev> [11] Review Request: 8205153 Delete "standard instructions" machinery in the open automated tests

Phil Race philip.race at oracle.com
Fri Jun 22 20:56:46 UTC 2018


I dozed off trying to review this a couple of days ago ;-)
I really just want to know if the tests all pass after the update ?

-phil.


On 06/22/2018 01:49 PM, Sergey Bylokhov wrote:
> Any volunteers for review?
> =)
>
> On 18/06/2018 12:09, Sergey Bylokhov wrote:
>> Hello.
>> Please review the fix for jdk11.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8205153
>> Webrev: http://cr.openjdk.java.net/~serb/8205153/webrev.00
>>
>> This fix is a part of the effort to cleanup and stabilize our tests. 
>> This change will delete "standard instructions" machinery in the open 
>> automated tests, which is just a copy-pasted code to to show this 
>> message to the user:
>> "This is an AUTOMATIC test, simply wait until it is done."
>>
>> In most cases it doubles the amount of code in the test, and make it 
>> difficult to update such tests. Also not all tests assume that the 
>> instruction frame could be opened on top of the test frame, which 
>> make the test unstable.
>>
>
>



More information about the awt-dev mailing list