RFR: 7903774: make all tests combinations printing nicer and/or configurable [v9]
Jiří Vaněk
jvanek at openjdk.org
Mon Feb 3 12:30:05 UTC 2025
On Mon, 30 Dec 2024 16:54:02 GMT, Jiří Vaněk <jvanek at openjdk.org> wrote:
>> This is still WIP, but already on row what [CODETOOLS-7903774](https://bugs.openjdk.org/browse/CODETOOLS-7903774) suggested.
>>
>> It will be finished once all desired output combinations are included, but already now is doing the usage of listing much more comfortable and code readable.
>>
>> From:
>>
>> `TESTS` - as introduced by https://github.com/openjdk/jcstress/pull/149
>> `COMBINATIONS` - as introduced in https://github.com/openjdk/jcstress/pull/149 verbose mode (thus surpassing the `-v` impact
>> `ALL_TESTS` - all tests as it was before https://github.com/openjdk/jcstress/pull/149
>> `COMBINATIONS_GROUPED ` - as suggested in https://github.com/openjdk/jcstress/pull/149#discussion_r1668678910
>> `COMBINATIONS_JSON` same as combinations, but a valid json
>> `COMBINATIONS_GROUPS` same as COMBINATIONS_GROUPED only instead of enumeration in each group, only count of members will be printed
>>
>>
>> * TESTS are already implemented as ALL_MATCHING
>> * COMBINATIONS as ALL_MATCHING_COMBINATIONS
>> * ALL_TESTS as ALL
>>
>> COMBINATIONS_GROUPED, and COMBINATIONS_GROUPS are under development.
>> The _JSON variants are tobe decided, but ability to JQL it sounds nice.
>>
>> Suggestion to better names welcomed.
>
> Jiří Vaněk has updated the pull request incrementally with two additional commits since the last revision:
>
> - Implemented proper json listings
>
> - id can be used both flatten (so equal to plaintext) or properly
> strcutured for JQ (default)
> - listing can be redirecd to stderr to use with `jq` directly
> - ALL now corectly return all tests
Just small recap, why this is done - I wanted to split full load of jcstresses, into several HWs. At least 50, so it can finish in reasonable time. This splitting is done by groups - eg https://github.com/adoptium/aqa-tests/blob/master/system/jcstress/playlist.xml#L1128 - to have those groups equally split, information provided by this PR have to be reasonably accessible.
-------------
PR Comment: https://git.openjdk.org/jcstress/pull/153#issuecomment-2630809733
More information about the jcstress-dev
mailing list