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

Reply via email to