On Thu, 10 Jun 2021 10:07:48 GMT, Igor Ignatyev <[email protected]> wrote:
> Hi all, > > could you please review this _test-only_ patch that switches the execution > mode of several `serviceability/sa` tests to `driver`? all these tests seem > not to perform actual testing, and just organize other JDK executions and > validates their outputs. > > `ClhsdbDumpclass.java` had to be updated to have access to > `./jdk/test/lib/apps/LingeredApp.class` file. Although this also can be fixed > by propagating the classpath setting to `javap`, using the path to `.class` > file is more clear and relable, and I actually suspect that before `javap` > read not the dumped `.class` file, but the one from the test's classpath. > > Thanks, > -- Igor Marked as reviewed by sspitsyn (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/4460
