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

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