RFR: 7903519 : jtreg/jtharness is missing features for basic crash testing [v7]

Jiří Vaněk jvanek at openjdk.org
Thu Dec 14 14:14:04 UTC 2023


On Thu, 14 Dec 2023 13:58:13 GMT, andrlos <duke at openjdk.org> wrote:

>> this does not exactly add a crash testing option, however it allows users to define their own resultModifiers that they can force jtharness to use via advertising their resultModifier service in META-INF. I tested it with basic crashOnlyModificator and it works as intended - allows me to correctly skip and pass all the regular failures and only leaves me with crash tests as failures. Perfect for corner cases like JVM testing and such.
>> Also checked correct behavior in the logs, where the regular failure is still being reported and only the justification of the result modification is added at the bottom.
>
> andrlos has updated the pull request incrementally with one additional commit since the last revision:
> 
>   static final convention fix, renaming of search method to load method and comment edits

@vlsi  I'm partially surprised that you seems to ok to include this change to jtharness. When @andrlos come with the idea I was a bit sceptic.  Now I'm curious, do you think jtreg project could harbour some implementations of this interface, and eg produce them as separate jars?

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

PR Comment: https://git.openjdk.org/jtharness/pull/57#issuecomment-1855925591


More information about the jtharness-dev mailing list