Review request: JDK-8004928 TEST_BUG: Reduce dependence of CoreLib tests from the AWT subsystem.
Alan Bateman
Alan.Bateman at oracle.com
Wed Dec 12 16:47:10 UTC 2012
On 12/12/2012 16:36, Daniel D. Daugherty wrote:
> For this item:
>
> > test/java/util/logging/LoggingDeadlock4.java
> > Test case was simplified to avoid AWT class loading.
> Negative test
> > result was tested on early JDK7 build.
>
> if I remember correctly, the whole point of that test was to
> check for a logging deadlock relative to AWT's usage of logging.
> If you avoid loading AWT classes, doesn't that make the test
> rather useless?
>
> Dan
java.awt.Container:
private static final PlatformLogger log =
PlatformLogger.getLogger("java.awt.Container");
private static final PlatformLogger eventLog =
PlatformLogger.getLogger("java.awt.event.Container");
and the updated test is just using PlatformLogger directly, I hope it
demonstrates the same issue with a JDK that doesn't have the fix.
(BTW: Just as background, with compact profiles coming then we need to
beat our tests into shape so that the tests for the APIs supported in
each profile can be run. Alexey is addressing some of the low-hanging
fruit, clearly it won't be possible to remove the dependency from all
tests. Also care is required to ensure that the test continues to test
what it was created to test. Expect efforts like this ^10 once modules
comes).
-Alan
More information about the core-libs-dev
mailing list