> 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.
Alexey Semenyuk has updated the pull request incrementally with one additional commit since the last revision: Use Executor.trace() instead of TKit.trace() ------------- Changes: - all: https://git.openjdk.org/jdk/pull/23936/files - new: https://git.openjdk.org/jdk/pull/23936/files/54585147..3fdf653e Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=23936&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=23936&range=01-02 Stats: 3 lines in 1 file changed: 0 ins; 0 del; 3 mod Patch: https://git.openjdk.org/jdk/pull/23936.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/23936/head:pull/23936 PR: https://git.openjdk.org/jdk/pull/23936
