<Swing Dev> [10] RFR 6962725:Regtest javax/swing/JFileChooser/6738668/bug6738668.java fails under Linux

Prasanta Sadhukhan prasanta.sadhukhan at oracle.com
Fri Jun 2 05:30:22 UTC 2017 

Thanks Sergey. I confirm new test fails with 7b40 in jtreg and pass with 
9b170.

Regards
Prasanta
On 6/1/2017 9:12 PM, Sergey Bylokhov wrote:
> Hi, Prasanta.
> You should set a JT_JAVA environment variable to jdk8 or jdk9 before running jtreg:
> http://openjdk.java.net/jtreg/runtests.html
>
> This java will be used to start a jtreg itself. And for the test the java passed to "-jdk" will be used.
>
> ----- prasanta.sadhukhan at oracle.com wrote:
>
>> I could not run jdk7u40 build with my jtreg harness as it cites the
>> below problem. I do not have jtreg older than 4.2b03, nor I could find
>>
>> in jtreg site.
>>
>> PRSADHUK-IN+prsadhuk at PRSADHUK-IN /cygdrive/c/jdk9/jtreg-42b03
>> $ ./bin/jtreg -jdk:/cygdrive/d/Vbox_Shared/JDK/jdk1.7.0-b40/fastdebug/
>>
>> /cygdrive/d/jdk7u/jdk7u-dev/jdk/test/javax/swing/JFileChooser/6738668/bug6738668.java
>> java.lang.UnsupportedClassVersionError:
>> com/sun/javatest/regtest/JDK$Fault : Unsupported major.minor version
>> 52.0
>>           at java.lang.ClassLoader.defineClass1(Native Method)
>>           at java.lang.ClassLoader.defineClass(ClassLoader.java:642)
>>
>> Regards
>> Prasanta
>> On 5/30/2017 10:28 PM, Sergey Bylokhov wrote:
>>> The new version fails even if run via jtreg?
>>>
>>> ----- prasanta.sadhukhan at oracle.com wrote:
>>>
>>>> OK. With revised command line, it fails with new updated test in
>> 7b40
>>>> and passed in 9b170
>>>>
>>>> /cygdrive/d/Vbox_Shared/JDK/jdk1.7.0-b40/fastdebug/bin/java
>>>> -Djava.security.manager -Djava.security.policy=security.policy
>>>> bug6738668
>>>> tmp dir C:\cygwin64\tmp\
>>>> Exception in thread "main" java.security.AccessControlException:
>>>> access
>>>> denied (java.io.FilePermission
>>>> C:\Users\prsadhuk\AppData\Roaming\Microsoft\Windows\Recent read)
>>>>            at
>>>>
>> java.security.AccessControlContext.checkPermission(AccessControlContext.java:345)
>>>>            at
>>>>
>> java.security.AccessController.checkPermission(AccessController.java:555)
>>>>            at
>>>>
>> java.lang.SecurityManager.checkPermission(SecurityManager.java:549)
>>>>            at
>>>> java.lang.SecurityManager.checkRead(SecurityManager.java:888)
>>>>
>>>> /cygdrive/d/Vbox_Shared/JDK/jdk9-b170/fastdebug/bin/java
>>>> -Djava.security.manager -Djava.security.policy=security.policy
>>>> bug6738668
>>>> tmp dir C:\cygwin64\tmp\
>>>> Test passed for LookAndFeel
>> javax.swing.plaf.metal.MetalLookAndFeel
>>>> tmp dir C:\cygwin64\tmp\
>>>> Test passed for LookAndFeel
>> javax.swing.plaf.nimbus.NimbusLookAndFeel
>>>> tmp dir C:\cygwin64\tmp\
>>>> Test passed for LookAndFeel
>>>> com.sun.java.swing.plaf.motif.MotifLookAndFeel
>>>> tmp dir C:\cygwin64\tmp\
>>>> Test passed for LookAndFeel
>>>> com.sun.java.swing.plaf.windows.WindowsLookAndFeel
>>>> tmp dir C:\cygwin64\tmp\
>>>> Test passed for LookAndFeel
>>>> com.sun.java.swing.plaf.windows.WindowsClassicLookAndFeel
>>>>
>>>> Regards
>>>> Prasanta
>>>> On 5/30/2017 2:59 AM, Sergey Bylokhov wrote:
>>>>> I guess you will need to set security manager which will use your
>>>> policy file:
>>>>
>> https://docs.oracle.com/cd/E13222_01/wls/docs81b/secmanage/java.html
>>>>> ----- prasanta.sadhukhan at oracle.com wrote:
>>>>>
>>>>>> I am not sure why it is passing in my case. I thought it might
>> be
>>>>>> taking
>>>>>> in some other security policy so I renamed and use a different
>>>> named
>>>>>> policy but it still passed in my windows 7.
>>>>>>
>>>>>> $ /cygdrive/d/Vbox_Shared/JDK/jdk1.7.0-b40/fastdebug/bin/java
>>>>>> -Djava.security.policy=my.policy bug6738668
>>>>>> Test passed for LookAndFeel
>>>> javax.swing.plaf.metal.MetalLookAndFeel
>>>>>> Test passed for LookAndFeel
>>>>>> com.sun.java.swing.plaf.motif.MotifLookAndFeel
>>>>>> Test passed for LookAndFeel
>>>>>> com.sun.java.swing.plaf.windows.WindowsLookAndFeel
>>>>>> Test passed for LookAndFeel
>>>>>> com.sun.java.swing.plaf.windows.WindowsClassicLookAndFeel
>>>>>>
>>>>>> PRSADHUK-IN+prsadhuk at PRSADHUK-IN
>>>>>>
>> /cygdrive/d/jdk10/client/jdk/test/javax/swing/JFileChooser/6738668
>>>>>> $ cat my.policy
>>>>>> grant {
>>>>>>         permission java.io.FilePermission "C:\\temp\\*", "read";
>>>>>>         permission java.io.FilePermission "C:\\temp", "read";
>>>>>>         permission java.util.PropertyPermission "*", "read";
>>>>>> };
>>>>>>
>>>>>> PRSADHUK-IN+prsadhuk at PRSADHUK-IN
>>>>>>
>> /cygdrive/d/jdk10/client/jdk/test/javax/swing/JFileChooser/6738668
>>>>>> $ /cygdrive/d/Vbox_Shared/JDK/jdk1.7.0-b40/fastdebug/bin/java
>>>>>> -version
>>>>>> java version "1.7.0-ea-fastdebug"
>>>>>> Java(TM) SE Runtime Environment (build 1.7.0-ea-fastdebug-b40)
>>>>>> Java HotSpot(TM) 64-Bit Server VM (build 14.0-b07-fastdebug,
>> mixed
>>>>>> mode)
>>>>>>
>>>>>> BTW, does my updated test fail in your environment with b40?
>>>>>>
>>>>>> Regards
>>>>>> Prasanta
>>>>>> On 5/28/2017 4:10 AM, Sergey Bylokhov wrote:
>>>>>>> There is a comment that JDK-6738668 a regression of JDK-6484091
>>>>>> which was pushed to b20.
>>>>>>> I just checked the test from the command line on b40 and it
>>>> fails,
>>>>>> but passed on b55.
>>>>>>> Can you please check why it was passed in your case.
>>>>>>>
>>>>>>> ----- prasanta.sadhukhan at oracle.com wrote:
>>>>>>>
>>>>>>>> Hi Sergey,
>>>>>>>>
>>>>>>>> I do not see any comment mention about failure in 7b20 but
>>>> anyways,
>>>>>> I
>>>>>>>> tried with 7b20,7b30,7b40,7b50 and all of them passed with
>>>>>> original
>>>>>>>> and
>>>>>>>> updated test in windows.
>>>>>>>>
>>>>>>>> Regards
>>>>>>>> Prasanta
>>>>>>>> On 5/25/2017 2:30 AM, Sergey Bylokhov wrote:
>>>>>>>>> I am not sure but according to the comments of JDK-6738668 it
>>>> was
>>>>>> a
>>>>>>>> regression in 7b20.
>>>>>>>>> So I suggest to check a few build between b20 and b54
>>>>>>>>>
>>>>>>>>> ----- prasanta.sadhukhan at oracle.com wrote:
>>>>>>>>>
>>>>>>>>>> Hi Sergey,
>>>>>>>>>>
>>>>>>>>>> No, it is passing. Actually, the original test is also
>> passing
>>>>>>>> with
>>>>>>>>>> 1.7.0 b05 (6738668 is supposedly fixed in 7b55), there's no
>>>>>>>>>> SecurityException
>>>>>>>>>>
>>>>>>>>>> jdk1.7.0/bin/java -Djava.security.policy=security.policy
>>>>>>>> bug6738668
>>>>>>>>>> Test passed for LookAndFeel
>>>>>>>> javax.swing.plaf.metal.MetalLookAndFeel
>>>>>>>>>> Test passed for LookAndFeel
>>>>>>>>>> com.sun.java.swing.plaf.motif.MotifLookAndFeel
>>>>>>>>>> Test passed for LookAndFeel
>>>>>>>>>> com.sun.java.swing.plaf.windows.WindowsLookAndFeel
>>>>>>>>>> Test passed for LookAndFeel
>>>>>>>>>> com.sun.java.swing.plaf.windows.WindowsClassicLookAndFeel
>>>>>>>>>>
>>>>>>>>>> jdk1.7.0/bin/java -version
>>>>>>>>>> java version "1.7.0-ea"
>>>>>>>>>> Java(TM) SE Runtime Environment (build 1.7.0-ea-b05)
>>>>>>>>>> Java HotSpot(TM) 64-Bit Server VM (build 1.7.0-ea-b05, mixed
>>>>>> mode)
>>>>>>>>>> Regards
>>>>>>>>>> Prasanta
>>>>>>>>>> On 5/24/2017 11:48 AM, Sergey Bylokhov wrote:
>>>>>>>>>>> Hi, Prasanta.
>>>>>>>>>>> Please confirm that the updated test fails before
>> JDK-6738668
>>>>>> was
>>>>>>>>>> fixed.
>>>>>>>>>>>> Hi ALl,
>>>>>>>>>>>>
>>>>>>>>>>>> Please review a testbug fix for an issue where this
>>>> regression
>>>>>>>> test
>>>>>>>>>> is failing in linux because
>>>>>>>>>>>> JFileChooser was trying to access C:/temp path which does
>>>> not
>>>>>>>> exist
>>>>>>>>>> in linux.
>>>>>>>>>>>> Proposed fix is to use java.io.tmpdir instead of hardcoded
>>>>>>>> windows
>>>>>>>>>> path.
>>>>>>>>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-6962725
>>>>>>>>>>>> webrev:
>>>>>>>> http://cr.openjdk.java.net/~psadhukhan/6962725/webrev.00/
>>>>>>>>>>>> Regards
>>>>>>>>>>>> Prasanta

More information about the swing-dev mailing list