Add description to failure message if it is available
Aleksey Shipilev
shade at redhat.com
Thu Jan 25 11:43:27 UTC 2018
On 12/26/2017 01:23 PM, Aleksey Shipilev wrote:
> On 12/26/2017 12:19 PM, Ivan Sopov wrote:
>> In the final print of results ids of test failures are included, but
>> descriptions are not (they are also called interpretations in the previous
>> output). It seems to me that it would be convenient to have those
>> descriptions right next to failure ids. I mean this:
>>
>> CustomConcurrentHashMapEntrySetRemoveIfTest []: Observed forbidden state:
>> false, true, true, true (Put T, Put F, Remove F)
>> CustomConcurrentHashMapEntrySetRemoveIfTest []: Observed forbidden state:
>> true, false, false, true (Put F, Put T, Cannot remove T)
>>
>> instead of this:
>>
>> CustomConcurrentHashMapEntrySetRemoveIfTest []: Observed forbidden state:
>> false, true, true, true
>> CustomConcurrentHashMapEntrySetRemoveIfTest []: Observed forbidden state:
>> true, false, false, true
>
> That makes sense.
>
>> Patch is attached.
>
> Please sign and submit the OCA, as per:
> http://openjdk.java.net/contribute/
I see you are on OCA signatories now, pushed:
https://bugs.openjdk.java.net/browse/CODETOOLS-7902109
-Aleksey
More information about the jcstress-dev
mailing list