<AWT Dev>  Review Request: 8205153 Delete "standard instructions" machinery in the open automated tests
Sergey.Bylokhov at oracle.com
Fri Jun 22 21:08:52 UTC 2018
On 22/06/2018 13:56, Phil Race wrote:
> 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 ?
Only the tests which previously sometimes fail because of instructions
windows which were open on top of the test window. I did not change any
other logic in the tests, other than mechanical removing of part of the
code just to simplify webrev. The next step will be drop of applet
usage(also w/o altered the logic of the tests).
> On 06/22/2018 01:49 PM, Sergey Bylokhov wrote:
>> Any volunteers for review?
>> On 18/06/2018 12:09, Sergey Bylokhov wrote:
>>> 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.
Best regards, Sergey.
More information about the awt-dev