On Thu, 31 Jul 2025 15:48:12 GMT, Alan Bateman <[email protected]> wrote:
>> Kevin Walls has updated the pull request incrementally with one additional >> commit since the last revision: >> >> One more case > > test/jdk/javax/management/MBeanServer/ExceptionTestNulls.java line 29: > >> 27: * @summary Test various null parameters and verify Exceptions thrown >> 28: * @modules java.management.rmi >> 29: * @run main ExceptionTestNulls > > The test looks okay but I wonder if you've considered making it a JUnit test? Thanks for the CSR update. I was just following the norm for the area with the test. _Should_ I make this a junit test, is that required, desirable, unnecessary... ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/26548#discussion_r2245897538
