8343889: Test runtime/cds/appcds/redefineClass/RedefineBasicTest.java failed

8344046: Tests under cds/appcds/jvmti/redefineClasses should have @requires vm.cds

Reviewed-by: iklam, matsaave

Author: calvinccheung

test/hotspot/jtreg/runtime/cds/appcds/jvmti/redefineClasses/OldClassAndRedefineClass.java

@@ -26,15 +26,15 @@
  * @test
  * @bug 8342303
  * @summary Test loading of shared old class when another class has been redefined.
- * @library /test/lib /test/hotspot/jtreg/runtime/cds/appcds /test/hotspot/jtreg/runtime/cds/appcds/test-classes /test/hotspot/jtreg/runtime/cds/appcds/jvmti
- * @requires vm.cds.write.archived.java.heap
+ * @library /test/lib /test/hotspot/jtreg/runtime/cds/appcds /test/hotspot/jtreg/runtime/cds/appcds/test-classes
+ * @requires vm.cds
  * @requires vm.jvmti
+ * @run driver RedefineClassHelper
  * @build jdk.test.whitebox.WhiteBox
  *        OldClassAndRedefineClassApp
  * @compile ../../test-classes/OldSuper.jasm
  *          ../../test-classes/ChildOldSuper.java
  *          ../../test-classes/Hello.java
- * @run driver RedefineClassHelper
  * @run driver OldClassAndRedefineClass
  */
 

test/hotspot/jtreg/runtime/cds/appcds/jvmti/redefineClasses/RedefineBootClassTest.java

@@ -29,12 +29,12 @@
  * @library /test/lib
  *          /test/hotspot/jtreg/runtime/cds/appcds
  *          /test/hotspot/jtreg/runtime/cds/appcds/test-classes
- *          /test/hotspot/jtreg/runtime/cds/appcds/jvmti
+ * @requires vm.cds
  * @requires vm.jvmti
+ * @run driver RedefineClassHelper
  * @build RedefineBootClassTest
  *        RedefineBootClassApp
  *        BootSuper BootChild
- * @run driver RedefineClassHelper
  * @run driver RedefineBootClassTest
  */
 

test/hotspot/jtreg/runtime/cds/appcds/jvmti/redefineClasses/RedefineOldSuperTest.java

@@ -29,13 +29,13 @@
  * @library /test/lib
  *          /test/hotspot/jtreg/runtime/cds/appcds
  *          /test/hotspot/jtreg/runtime/cds/appcds/test-classes
- *          /test/hotspot/jtreg/runtime/cds/appcds/jvmti
+ * @requires vm.cds
  * @requires vm.jvmti
  * @compile ../../test-classes/OldSuper.jasm
+ * @run driver RedefineClassHelper
  * @build RedefineOldSuperTest
  *        RedefineOldSuperApp
  *        NewChild
- * @run driver RedefineClassHelper
  * @run driver RedefineOldSuperTest
  */
 

test/hotspot/jtreg/runtime/cds/appcds/redefineClass/RedefineBasicTest.java

@@ -30,7 +30,7 @@
  * @requires vm.jvmti
  * @library /test/lib /test/hotspot/jtreg/serviceability/jvmti/RedefineClasses /test/hotspot/jtreg/runtime/cds/appcds
  * @run driver RedefineClassHelper
- * @build jdk.test.whitebox.WhiteBox jdk.test.lib.compiler.InMemoryJavaCompiler  RedefineBasic
+ * @build jdk.test.whitebox.WhiteBox RedefineBasic
  * @run driver RedefineBasicTest
  */
 

test/hotspot/jtreg/runtime/cds/appcds/redefineClass/RedefineRunningMethods_Shared.java

@@ -30,7 +30,7 @@
  * @requires vm.jvmti
  * @library /test/lib /test/hotspot/jtreg/serviceability/jvmti/RedefineClasses /test/hotspot/jtreg/runtime/cds/appcds
  * @run driver RedefineClassHelper
- * @build jdk.test.whitebox.WhiteBox jdk.test.lib.compiler.InMemoryJavaCompiler
+ * @build jdk.test.whitebox.WhiteBox
  * @compile RedefineRunningMethods_SharedHelper.java
  * @run driver RedefineRunningMethods_Shared
  */

test/hotspot/jtreg/runtime/logging/RedefineClasses.java

@@ -30,7 +30,6 @@
  * @modules java.compiler
  *          java.instrument
  * @requires vm.jvmti
- * @build jdk.test.lib.compiler.InMemoryJavaCompiler
  * @run main RedefineClassHelper
  * @run main/othervm -Xmx256m -XX:MaxMetaspaceSize=64m -javaagent:redefineagent.jar -Xlog:all=trace:file=all.log RedefineClasses
  */