RFR: CODETOOLS-7903748 - jcstress: Test list should honor concurrency settings [v2]

[Jiří Vaněk]

On Tue, 2 Jul 2024 17:32:12 GMT, PM <duke at openjdk.org> wrote:

>> Jiří Vaněk has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
>> 
>>   CODETOOLS-7903748 - jcstress: Test list should honor concurrency settings
>>   
>>   This is extracting  List<TestConfig> configs =prepareRunProgram(classes, tests);
>>   with all he HW/switches setup to separated method and reusing it in `-l` mode
>>   
>>   -l now honours also verbose mode, in which it prints not just matching
>>   tests but all really run tests, and thus enabling much more easy
>>   determining of all tests
>>   
>>   help adjusted.
>>   
>>   Maybe I'm missing plain quick initial all tests metod now, but with
>>   artificial -c MAX it seems doing exactly that
>
> jcstress-core/src/main/java/org/openjdk/jcstress/Main.java line 74:
> 
>> 72:             for (String test : testsToPrint) {
>> 73:                 System.out.println(test);
>> 74:             }
> 
> maybe move this code to a separate method, for better readability and maintainability

Sure! Done.

> jcstress-core/src/main/java/org/openjdk/jcstress/Options.java line 85:
> 
>> 83: 
>> 84:         OptionSpec<Boolean> list = parser.accepts("l", "List the available tests matching the requested settings, " +
>> 85:                         "after all filters (like CPU count) are applied. In verbose mode it prints all real combinations which will run.")
> 
> "List the available tests matching the requested settings, " +
>                         "after all filters (like CPU count) are applied. In verbose mode it prints all real combinations which will run."
> 
> this might have been done for better readability, but remove the string concatenation, define it as a part of single string, possibly at class level by making it static final

fixed

-------------

PR Review Comment: https://git.openjdk.org/jcstress/pull/149#discussion_r1663770684
PR Review Comment: https://git.openjdk.org/jcstress/pull/149#discussion_r1663766131