Review request: JDK-8004928 TEST_BUG: Reduce dependence of CoreLib tests from the AWT subsystem.
Daniel D. Daugherty
daniel.daugherty at oracle.com
Wed Dec 12 16:52:31 UTC 2012
On 12/12/12 9:47 AM, Alan Bateman wrote:
> 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 thought the deadlock had to do with locks grabbed on the way
to getting into the underlying PlatformLogger, but my memory is
hazy and I don't have the cycles to research this.
> I hope it demonstrates the same issue with a JDK that doesn't have the
> fix.
That would be the right way to see if the test still "works".
> (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).
Thanks for the background.
Dan
More information about the core-libs-dev
mailing list