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. This pull request has now been integrated. Changeset: 3aa6d62a Author: Alexey Semenyuk <asemen...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/3aa6d62afecbea5684618bc82425e59404233be5 Stats: 2112 lines in 37 files changed: 1377 ins; 459 del; 276 mod 8351372: Improve negative tests coverage of jpackage Reviewed-by: almatvee ------------- PR: https://git.openjdk.org/jdk/pull/23936
