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

Prasanta Sadhukhan prasanta.sadhukhan at oracle.com
Thu Jul 25 10:13:19 UTC 2019


+1

Regards
Prasanta
On 25-Jul-19 3:40 PM, Tejpal Rebari wrote:
> Hi All,
> I have verified that the test fails without the changes of the fix 
> 8145896.
>
> Regards
> Tejpal
>> On 25-Jul-2019, at 12:40 PM, Prasanta Sadhukhan 
>> <prasanta.sadhukhan at oracle.com 
>> <mailto:prasanta.sadhukhan at oracle.com>> wrote:
>>
>> OK. Last thing is, please confirm if the modified test still fails if 
>> we remove 8145896 fix 
>> [http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/ef8fd276efe1].
>>
>> Regards
>> Prasanta
>> On 25-Jul-19 12:33 PM, Tejpal Rebari wrote:
>>> Hi Prasanta,
>>> Agreed that the two points should be different, so retained the 
>>> point variable for executeTest().
>>>
>>> Here is the latest changes.
>>> webrev : http://cr.openjdk.java.net/~trebari/swing/8194944/webrev2/
>>>
>>> Thanks and regards
>>> Tejpal
>>>
>>>> On 22-Jul-2019, at 4:08 PM, Prasanta Sadhukhan 
>>>> <prasanta.sadhukhan at oracle.com 
>>>> <mailto:prasanta.sadhukhan at oracle.com>> wrote:
>>>>
>>>> Looks better but I guess frame's locationOnScreen point would be 
>>>> different from Menu's centrepoint so we should not use same "point" 
>>>> instance variable for both. Guess local "point" variable for 
>>>> executeTest() should be retained.
>>>>
>>>> Regards
>>>> Prasanta
>>>> On 19-Jul-19 1:11 PM, Tejpal Rebari wrote:
>>>>> 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
>>>>>
>>>>> Thanks and regards
>>>>> Tejpal Rebari
>>>>>
>>>>>
>>>>>> On 08-Jul-2019, at 1:43 PM, Prasanta Sadhukhan 
>>>>>> <prasanta.sadhukhan at oracle.com 
>>>>>> <mailto: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
>>>>>>> webrev: 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/20190725/0f1cc0af/attachment.html>


More information about the swing-dev mailing list