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

Jiří Vaněk jvanek at openjdk.org
Mon Jul 8 14:48:45 UTC 2024


On Mon, 8 Jul 2024 14:28:58 GMT, Jiří Vaněk <jvanek at openjdk.org> wrote:

>> This is extracting  List<TestConfig> configs =prepareRunProgram(classes, tests);
>> with all he HW/switches setup to separated method and reusing it in `-l` mode
>> 
>> I'm aware of triplicated removal of -agentlib, and will clean it up as
>> CODETOOLS-7903756 will progress.
>> 
>> -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
>
> Jiří Vaněk has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Renamed configsWithScheduler to config as suggester in review
>  - Moved listTests to JCStress class, made ConfigsWithScheduler private

Sorry, I understood the `Wait, let's make sure it passes the GHA tests, enable the workflows first, and then update the PR.` that it will get enabled somehow on its own or by you.

I'm failing to see whre to enabel them as aprt of my PRs

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

PR Comment: https://git.openjdk.org/jcstress/pull/149#issuecomment-2214294269


More information about the jcstress-dev mailing list