On Fri, 24 Jul 2026 14:56:44 GMT, Jorn Vernee <[email protected]> wrote:
> 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). This pull request has now been integrated. Changeset: 284e0f8d Author: Jorn Vernee <[email protected]> URL: https://git.openjdk.org/jdk/commit/284e0f8daf8187d24af04cb2eb35adc3c9fc706d Stats: 2646 lines in 100 files changed: 878 ins; 268 del; 1500 mod 8388793: Convert test/jdk/java/foreign tests to use JUnit Reviewed-by: pminborg ------------- PR: https://git.openjdk.org/jdk/pull/32042
