--- old/test/hotspot/jtreg/runtime/appcds/jigsaw/modulepath/AddOpens.java 2018-04-06 15:40:42.522085848 -0700 +++ new/test/hotspot/jtreg/runtime/appcds/jigsaw/modulepath/AddOpens.java 2018-04-06 15:40:42.253060479 -0700 @@ -90,21 +90,21 @@ "-Xlog:class+load=trace", "-XX:+PrintSystemDictionaryAtExit", "--module-path", moduleDir.toString(), "-m", TEST_MODULE1); - TestCommon.checkExecReturn(output, 0, true, "Loading classes to share"); + TestCommon.checkDump(output); // run with the archive using the same command line as in dump time // plus the "--add-opens java.base/java.lang=com.simple" option. // The main class should be loaded from the archive. // The setaccessible(true) on the ClassLoader.defineClass method should // be successful. - output = TestCommon.execCommon( "-Xlog:class+load=trace", - "-cp", destJar.toString(), - "--add-opens", "java.base/java.lang=" + TEST_MODULE1, - "--module-path", moduleDir.toString(), - "-m", TEST_MODULE1, "with_add_opens"); - TestCommon.checkExecReturn(output, 0, true, - "[class,load] com.simple.Main source: shared objects file", - "method.setAccessible succeeded!"); + TestCommon.run( "-Xlog:class+load=trace", + "-cp", destJar.toString(), + "--add-opens", "java.base/java.lang=" + TEST_MODULE1, + "--module-path", moduleDir.toString(), + "-m", TEST_MODULE1, "with_add_opens") + .assertNormalExit( + "[class,load] com.simple.Main source: shared objects file", + "method.setAccessible succeeded!"); } }