On Thu, 27 Apr 2023 20:51:53 GMT, Maurizio Cimadamore <mcimadam...@openjdk.org> wrote:
>> Jim Laskey has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 14 commits: >> >> - Merge branch 'master' into 8306112 >> - PreviewFeatures.isEnabled() >> - Clean up isPreview >> - Missing exception >> - Corrections >> - Update VM.java >> - Clean up testing >> - Update TestJavacTaskScanner.java >> - Merge branch 'master' into 8306112 >> - Clean up >> - ... and 4 more: https://git.openjdk.org/jdk/compare/96cdf93b...53a5321d > > test/jdk/tools/launcher/OnrampMainTest.java line 31: > >> 29: * @run main OnrampMainTest >> 30: */ >> 31: public class OnrampMainTest extends TestHelper { > > Should more tests be added for inherited methods? Test added ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13689#discussion_r1186003544