Alan Bateman wrote:
Mandy Chung wrote:
Another patch of test fixes - with these fixes, there are now 5 remaining tests in ProblemList.txt excluded for jdk_lang test target.

6961894: TEST_BUG: jdk_lang tests fail in samevm mode

Webrev at:
  http://cr.openjdk.java.net/~mchung/6961894/webrev.00/

- Some tests are marked to run in othervm mode
- Some java/lang/management tests are cleaned up to restore the state before it finishes. - java/lang/reflect/Proxy and a few tests have its custom classloader. Fix them to use the proper parent class loader.

Thanks
Mandy
This mostly look good to me.

One comment on the M&M tests that still run in samevm mode, is that it might be nicer to restore the settings in a finally block. That will ensure they are restored in the event that the test fails.

Good suggestion.  I'll make the change.
That could work in test/java/lang/management/ThreadMXBean/{Enable,Disable}Test.java for example. Also in test/java/lang/management/MemoryMXBean/CollectionUsageThreshold.java.

Thanks for removing sun/management/jmxremote/bootstrap/JvmstatCountersTest.java from the list - I didn't realize it was the problem list when fixing 6950927. I notice you've also removed a few java/util/ResourceBundle and java/util/Locale tests from the problem - is this list problem list maintenance?

Yes, some tests were fixed but the problem list was not updated.

Thanks
Mandy

Reply via email to