Hello,

Please review the proposed fix for

8081547: Prepare client libs regression tests for running in a concurrent, headless jtreg environment
    http://cr.openjdk.java.net/~darcy/8081547.1/

This fix takes advantage of the recently added "headful" jtreg keyword (JDK-8129822: Define "headful" jtreg keyword). Patch below.

Some long-standing test failures are added to the problem list, including one in sound; hence the cc to sound-dev.

With the modifications, the the follow test run now passes for me:

jtreg [...] -agentvm -conc:2 -exclude:ProblemList.txt -Djava.awt.headless=true -k:'!headful' :jdk_beans

Likewise, the test invocation

    jtreg [...] -agentvm -conc:2 -exclude:ProblemList.txt :jdk_beans

also succeeds when a headful environment is indeed present with the additional 11 tests passing.

Thanks,

-Joe

--- old/test/ProblemList.txt    2015-06-29 15:57:08.822250552 -0700
+++ new/test/ProblemList.txt    2015-06-29 15:57:08.654250549 -0700
@@ -116,6 +116,13 @@

 # jdk_beans

+# 8060027
+
+java/beans/XMLEncoder/Test4903007.java generic-all
+java/beans/XMLEncoder/java_awt_GridBagLayout.java generic-all
+java/beans/XMLDecoder/8028054/TestConstructorFinder.java generic-all
+java/beans/XMLDecoder/8028054/TestMethodFinder.java generic-all
+
 ############################################################################

 # jdk_lang
@@ -293,6 +300,10 @@

 # jdk_sound

+# 8059743
+
+javax/sound/midi/Gervill/SoftProvider/GetDevice.java generic-all
+
 ############################################################################

 # jdk_swing
--- old/test/java/beans/PropertyEditor/6380849/TestPropertyEditor.java 2015-06-29 15:57:09.298250559 -0700 +++ new/test/java/beans/PropertyEditor/6380849/TestPropertyEditor.java 2015-06-29 15:57:09.130250556 -0700
@@ -1,5 +1,5 @@
-/**
- * Copyright (c) 2009, 2012, Oracle and/or its affiliates. All rights reserved.
+/*
+ * Copyright (c) 2009, 2015, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -28,6 +28,7 @@
  * @author Sergey Malenkov
  * @compile -XDignore.symbol.file TestPropertyEditor.java
  * @run main TestPropertyEditor
+ * @key headful
  */

 import editors.SecondBeanEditor;
--- old/test/java/beans/PropertyEditor/TestColorClass.java 2015-06-29 15:57:09.770250566 -0700 +++ new/test/java/beans/PropertyEditor/TestColorClass.java 2015-06-29 15:57:09.598250563 -0700
@@ -26,6 +26,7 @@
  * @bug 4506596 6258510
* @summary Tests PropertyEditor for value of type Color with security manager
  * @author Sergey Malenkov
+ * @key headful
  */

 import java.awt.Color;
--- old/test/java/beans/PropertyEditor/TestColorClassJava.java 2015-06-29 15:57:10.230250573 -0700 +++ new/test/java/beans/PropertyEditor/TestColorClassJava.java 2015-06-29 15:57:10.062250570 -0700
@@ -26,6 +26,7 @@
  * @bug 4506596
  * @summary Tests PropertyEditor for value of type Color
  * @author Sergey Malenkov
+ * @key headful
  */

 import java.awt.Color;
--- old/test/java/beans/PropertyEditor/TestColorClassNull.java 2015-06-29 15:57:10.694250580 -0700 +++ new/test/java/beans/PropertyEditor/TestColorClassNull.java 2015-06-29 15:57:10.522250577 -0700
@@ -26,6 +26,7 @@
  * @bug 4506596 6498171
  * @summary Tests PropertyEditor for null value of type Color
  * @author Sergey Malenkov
+ * @key headful
  */

 import java.awt.Color;
--- old/test/java/beans/PropertyEditor/TestColorClassValue.java 2015-06-29 15:57:11.162250587 -0700 +++ new/test/java/beans/PropertyEditor/TestColorClassValue.java 2015-06-29 15:57:10.994250584 -0700
@@ -26,6 +26,7 @@
  * @bug 4222827 4506596
  * @summary Tests PropertyEditor for value of type Color
  * @author Sergey Malenkov
+ * @key headful
  */

 import java.awt.Color;
--- old/test/java/beans/PropertyEditor/TestFontClass.java 2015-06-29 15:57:11.630250594 -0700 +++ new/test/java/beans/PropertyEditor/TestFontClass.java 2015-06-29 15:57:11.458250591 -0700
@@ -26,6 +26,7 @@
  * @bug 4506596 6258510 6538853
* @summary Tests PropertyEditor for value of type Font with security manager
  * @author Sergey Malenkov
+ * @key headful
  */

 import java.awt.Font;
--- old/test/java/beans/PropertyEditor/TestFontClassJava.java 2015-06-29 15:57:12.098250601 -0700 +++ new/test/java/beans/PropertyEditor/TestFontClassJava.java 2015-06-29 15:57:11.930250598 -0700
@@ -26,6 +26,7 @@
  * @bug 4506596 6538853
  * @summary Tests PropertyEditor for value of type Font
  * @author Sergey Malenkov
+ * @key headful
  */

 import java.awt.Font;
--- old/test/java/beans/PropertyEditor/TestFontClassNull.java 2015-06-29 15:57:12.566250608 -0700 +++ new/test/java/beans/PropertyEditor/TestFontClassNull.java 2015-06-29 15:57:12.394250605 -0700
@@ -26,6 +26,7 @@
  * @bug 4506596 6498171 6538853
  * @summary Tests PropertyEditor for null value of type Font
  * @author Sergey Malenkov
+ * @key headful
  */

 import java.awt.Font;
--- old/test/java/beans/PropertyEditor/TestFontClassValue.java 2015-06-29 15:57:13.030250615 -0700 +++ new/test/java/beans/PropertyEditor/TestFontClassValue.java 2015-06-29 15:57:12.862250612 -0700
@@ -26,6 +26,7 @@
  * @bug 4222827 4506596 6538853
  * @summary Tests PropertyEditor for value of type Font
  * @author Sergey Malenkov
+ * @key headful
  */

 import java.awt.Font;
--- old/test/java/beans/XMLEncoder/java_awt_ScrollPane.java 2015-06-29 15:57:13.486250622 -0700 +++ new/test/java/beans/XMLEncoder/java_awt_ScrollPane.java 2015-06-29 15:57:13.322250619 -0700
@@ -25,6 +25,7 @@
  * @test
  * @bug 6402062 6487891
  * @summary Tests ScrollPane encoding
+ * @key headful
  * @author Sergey Malenkov
  */


Reply via email to