RFR: 7903519 : jtreg/jtharness is missing features for basic crash testing [v4]
Vladimir Sitnikov
vsitnikov at openjdk.org
Thu Dec 14 04:14:55 UTC 2023
On Wed, 13 Dec 2023 20:52:11 GMT, andrlos <duke at openjdk.org> wrote:
>> The implementation can access the original status with `result.getStatus()`.
>>
>> In other words, the framework should populate the result as usual with `result.setStatus(execStatus)`, and then it could call `resultModifier.modifyResult(result)`
>
> and if you are thinking about it @vlsi - The status is unmodifiable by design. I still stand by my implementation as a concept and believe it is pretty elegant.. I shall address judovana's remarks and improve naming a bit as I know that is not something that is particularly clear
if the status is unmodifiable, then `TestResultTransformer` with method `TestResult transform(TestResult)` would be better as it would cover more cases rather than changing status alone
-------------
PR Review Comment: https://git.openjdk.org/jtharness/pull/57#discussion_r1426152543
More information about the jtharness-dev
mailing list