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

andrlos duke at openjdk.org
Thu Dec 14 13:14:14 UTC 2023


> 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 two additional commits since the last revision:

 - missing part of a comment
 - addressing more of PR remarks, mostly renaming and StatusTransformer is now initialized with Script instance initialization

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

Changes:
  - all: https://git.openjdk.org/jtharness/pull/57/files
  - new: https://git.openjdk.org/jtharness/pull/57/files/6e1a8ea6..da209eb1

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jtharness&pr=57&range=04
 - incr: https://webrevs.openjdk.org/?repo=jtharness&pr=57&range=03-04

  Stats: 35 lines in 2 files changed: 3 ins; 14 del; 18 mod
  Patch: https://git.openjdk.org/jtharness/pull/57.diff
  Fetch: git fetch https://git.openjdk.org/jtharness.git pull/57/head:pull/57

PR: https://git.openjdk.org/jtharness/pull/57


More information about the jtharness-dev mailing list