Review request: JDK-7162111 TEST_BUG: change tests run in headless mode [macosx] (open part)

Alexey Utkin alexey.utkin at oracle.com
Mon Nov 19 14:54:42 UTC 2012


Here is the updated version:
     http://cr.openjdk.java.net/~uta/openjdk-webrevs/JDK-7162111/webrev.01/

On 19.11.2012 16:26, Alan Bateman wrote:
> On 19/11/2012 09:57, Alexey Utkin wrote:
>> Bug description:
>>     https://jbs.oracle.com/bugs/browse/JDK-7162111
>>
>> Here is the suggested fix:
>>     http://cr.openjdk.java.net/~uta/openjdk-webrevs/JDK-7162111/
> Thanks for taking this one on.
>
> The change to JFrameCreateTime.java looks fine to me.
>
> For test/java/io/Serializable/resolveClass/deserializeButton/Foo.java 
> then I suggest removing the "was java.awt.Button" references from the 
> comments. The reason is that it will likely confuse future 
> maintainers. Also I think Adapter should be renamed, perhaps Element? 
> Finally the Error message still includes "Button" in the message and 
> we should change that. Otherwise it's great to have this test running 
> headless.
Done.
>
> One thing you'll need to do is remove these tests from the exclude 
> list (jdk/test/ProblemList.txt), otherwise they will not be run on the 
> Mac as they are currently excluded for that platform.
Done.
>
> I realize you've run the javax.script and jrunscript tests and they 
> pass for you but it may be that they aren't run (because they excluded 
> and ProblemList.txt has not been updated) or maybe you just didn't run 
> into the conditions that cause AWT to hang. I think these tests should 
> have their @run tag changed so that they run with 
> -Djava.awt.headless=true. That will allow them to be removed from the 
> exclude list.
>
> I looked at java/util/ResourceBundle/Control/Bug6530694.java and it 
> appears that -Djava.awt.headless=true has added as part of the 
> forward-port of the Mac port so I think this means it can be removed 
> from the exclude list now.
Here is the JPRT respond with fixed test/ProblemList.txt:
http://prt-web.us.oracle.com//archive/2012/11/2012-11-19-123835.fritz.jdk/JobStatus.txt

Run time:           01H 16m 33s
Build Stats:        18 pass, 0 fail, 0 killed, 0 working, 0 initializing, 0 not started
Test Stats:         30 pass, 0 fail, 0 killed, 0 working, 0 initializing, 0 not started

Looks good without "-Djava.awt.headless=true".

Regards,
-uta



More information about the core-libs-dev mailing list