OK, I will do that, Mandy. Should I create a separate bug (subtask) for this test fixes?
Shura > On May 10, 2016, at 12:49 PM, Mandy Chung <mandy.ch...@oracle.com> wrote: > > >> On May 10, 2016, at 12:00 PM, Alexandre (Shura) Iline >> <alexandre.il...@oracle.com> wrote: >> >> Hi. >> >> Please review the fix: >> http://cr.openjdk.java.net/~shurailine/8155993/webrev.00/ >> >> In the failing tests I have replaced java.management with jdk.management in >> @modules. >> >> Besides the failed tests I have scanned all the tests which call >> ManagementFactory.getPlatformMBeanServer(…). Those which were not failing >> were already declaring dependency on jdk.management, sometimes together with >> java.management. Declaring java.management in such case is not needed, >> because java.management is a dependency of jdk.management. > > In fact JDK-8155993 reveals a potential bug in the VM support for > java.lang.management when jdk.management module is not present. > > I have no objection to change these tests to run with jdk.management and this > patch looks fine. But we should keep JDK-8155993 open and provide a small > test to reproduce NCDFE when running with an image with only java.management > but no jdk.management module. > > Mandy >
