On Thu, 9 Mar 2023 20:24:11 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:
>> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Revise logging of ProcessBuilder.start to support DEBUG and TRACE logging. >> Revise the implNote to include a warning about logging security sensistive >> information. >> DEBUG logs only the command, directory, stack trace, and pid. >> TRACE additionally logs arguments. > > test/jdk/java/lang/ProcessBuilder/ProcessStartLoggingTest.java line 81: > >> 79: File nullDirectory = null; >> 80: File thisDirectory = new File("."); >> 81: > > I started to heavily use different `@test` sections with speaking names > instead of running a bunch of test in sequence; the advantage is better > parallelization of tests, that I can omit printing out the test name manually > (if the test name itself is speaking), and the ability to start individual > tests manually. It does come with lots of test sections though. Speakable names can be a plus but I'll stick to the parameterized version for now. Are you refering to JUnit sections or Jtreg sections? ------------- PR: https://git.openjdk.org/jdk/pull/12862