On Sat, 7 Mar 2026 13:49:47 GMT, Roger Riggs <[email protected]> wrote:
> Refactor ProcessBuilder and ProcessHandle tests to use Junit instead of
> TestNG.
> Remove old main testng driver stubs that allowed tests to be run from the
> command line.
test/jdk/java/lang/ProcessBuilder/ProcessReaperCCL.java line 60:
> 58: for (Thread th : Thread.getAllStackTraces().keySet()) {
> 59: if (th.getName().startsWith("process reaper")) {
> 60: Assertions.assertEquals(null, th.getContextClassLoader(),
> "CCL not null");
Nit - this could be replaced with `Assertions.assertNull`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30128#discussion_r2902818716
