<Swing Dev> [13] RFR 8194944:Regression automated test 'open/test/jdk/javax/swing/JInternalFrame/8145896/TestJInternalFrameMaximize.java' fails

Tejpal Rebari tejpal.rebari at oracle.com
Fri Jul 19 07:41:30 UTC 2019


Hi All,
Added the blockTillDisplayed() method instead of 1000ms delay.

here is the latest changes.
webrev : http://cr.openjdk.java.net/~pkbalakr/Tejpal/8194944/webrev1 <http://cr.openjdk.java.net/~pkbalakr/Tejpal/8194944/webrev1>

Thanks and regards
Tejpal Rebari


> On 08-Jul-2019, at 1:43 PM, Prasanta Sadhukhan <prasanta.sadhukhan at oracle.com> wrote:
> 
> Adding delay is indeterministic.  I guess it will be better if you use blockTillDisplayed() method which you can find in many of the testcase(s) like javax/swing/JComboBox/8182031/ComboPopupTest.java, javax/swing/JEditorPane/6917744/bug6917744.java
> Regards
> Prasanta
> On 08-Jul-19 1:02 PM, Tejpal Rebari wrote:
>> Hi All,
>> Please review a test fix for the issue where the test fails on Mac  and Linux intermittently.
>> 
>> The test fails because of “component must be showing on the screen to determine its location” error.
>> Which is because the test tries to get the location of the component and the component is not displayed on the screen yet.
>> 
>> The test tries to get the location of menu in the executeTest method so By adding some delay before the executeTest call solves the problem.
>> 
>> Tested in our internal build and test system and test passes on all the three Platforms.
>> 
>> Bug 		: https://bugs.openjdk.java.net/browse/JDK-8194944 <https://bugs.openjdk.java.net/browse/JDK-8194944>
>> webrev	: http://cr.openjdk.java.net/~pkbalakr/Tejpal/8194944/webrev0 <http://cr.openjdk.java.net/~pkbalakr/Tejpal/8194944/webrev0>
>> 
>> Thanks and regards
>> Tejpal 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/swing-dev/attachments/20190719/67d9439f/attachment.html>


More information about the swing-dev mailing list