<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 07:10:31 UTC 2019
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/7752eb8f/attachment.html>
More information about the swing-dev
mailing list