On Tue, 28 Nov 2023 14:43:36 GMT, Alan Bateman <al...@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 40 commits: >> >> - Merge remote-tracking branch 'upstream/master' into 8315458 >> - Requested changes >> - Remove Test >> - Move findMainMethod to non-public access >> - Handle javadoc on top level elements properly >> - Update Class.java >> - Update test >> - Update findMainMethod as requested >> - Add nested class test >> - Move Implicitly Declared Classes to single directory >> - ... and 30 more: https://git.openjdk.org/jdk/compare/2fae07f5...daacc0ba > > test/jdk/tools/launcher/modules/basic/LauncherErrors.java line 97: > >> 95: .shouldContain("Error: Unable to load main class " + >> MAIN_CLASS + " in module " + TEST_MODULE) >> 96: >> .shouldContain("java.security.AccessControlException") >> 97: .shouldNotHaveExitValue(0); > > The updates to the launcher tests look okay but I'm wondering if the "Caused > by:" is the output now? I get it that the "Unable to ..." message has changed > but has the format of the cause changed too? > > I assume you'll fix up the copyright date on Arrrrgs and LauncherErrors > before integrating. "Caused by:" has (always) been missing from the named module version of the error message. I'll add in and modify the test accordingly. Updated copyright. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16461#discussion_r1408356364