> This PR is carefully arranged into 4 different commits: > > 1. The raw output of the JUnit conversion tool > (https://github.com/openjdk/jdk/commit/e6d81b314029f7c8b2942dd2ffe1d49f9d99434d) > 2. Changing `assertThrows` to use static imports like the other asserts > (https://github.com/openjdk/jdk/commit/1fafd582524ca45da4d39a3c85a12a1fad640e1f) > 3. Tweaks to make the tests actually pass > (https://github.com/openjdk/jdk/commit/d0632787aa9944c91408955fc9a1ab7ac8baab45) > 4. Reducing the scope of `assertThrows` to cover only the lines that are > actually expected to throw. I felt that his was needed to make some of the > changes more 'sane', instead of just having the `assertThrows` cover the > entire method body. > (https://github.com/openjdk/jdk/commit/80d4b52a8a9ad22b627cee976edca9d1a033ec4c) > > It's probably easiest to review the individual commits. (1) and (2) and (4) > are fairly mechanical changes, so the interesting thing to look at is (3). > > Depends on: https://github.com/openjdk/jdk/pull/32014 > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai).
Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision: Add back checks for expected exception messages ------------- Changes: - all: https://git.openjdk.org/jdk/pull/32042/files - new: https://git.openjdk.org/jdk/pull/32042/files/631ca4a5..3654ccfb Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=32042&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=32042&range=00-01 Stats: 37 lines in 8 files changed: 21 ins; 2 del; 14 mod Patch: https://git.openjdk.org/jdk/pull/32042.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/32042/head:pull/32042 PR: https://git.openjdk.org/jdk/pull/32042
