On Thu, 6 Mar 2025 23:30:09 GMT, Alexey Semenyuk <asemen...@openjdk.org> wrote:
> Changes to tests: > - Added more test cases to ErrorTest. > - Added functionality to jpackage test lib to facilitate new ErrorTest test > cases. > - Make all other negative tests use the `CannedFormattedString` class to > validate error messages in jpackage output strictly. > - Removed redundant negative tests/test cases duplicating ErrorTest. > - Give unique descriptions to macos signing tests. > > Changes to jpackage: > - Main.java: Catch `java.nio.file.NoSuchFileException` as > `jdk.internal.opt.CommandLine.parse()` throws this exception instead of > `java.io.FileNotFoundException` to indicate missing command file. The issue > was uncovered with the new ErrorTest test case for `ERR_CannotParseOptions` > error ID. > - DeployParams.java: replace `ERR_MissingArgument` with > `error.no-input-parameter` as the error is not missing argument of `--input` > parameter, but missing `--input` parameter itself. The issue was uncovered > with the new ErrorTest test cases for `ERR_MissingArgument` and > ``error.no-input-parameter` error IDs. > - LauncherData.java: remove dead code as missing `--input` parameter is > handled earlier in DeployParams. > - DottedVersion.java: fix exception messages based on the new test cases in > ErrorTest. > - TokenRepalce.java/TokenRepalceTest.java: picked from > https://github.com/openjdk/jdk/pull/23923 PR. It is needed for ErrorTest as > well. Filed the CSR by accident. ------------- PR Comment: https://git.openjdk.org/jdk/pull/23936#issuecomment-2707820543
