RFR: 8288495: [test] Make OutputAnalyzer exception more informative [v3]

Roger Riggs rriggs at openjdk.org
Thu Jun 23 19:01:47 UTC 2022


> The RuntimeException from OutputAnalyzer for expected and unexpected exit values should include both the actual and expected values.
> The method `shouldHaveExitvalue` and `shouldNotHaveExitValue` include the value expected and the value not expected instead of the actual value returned. The values provided in the exception are already known quantities in the code, what is not known is the actual value.  
> The actual value is printed to stderr but not the expected value and in the logs, the exception report is frequently to stdout and therefor separated in the log from the output of stderr.  Including both expected and actual values in the exception makes it easier to understand.
> 
> Exception messages should not include newlines.

Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:

  Copyright update

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/9247/files
  - new: https://git.openjdk.org/jdk/pull/9247/files/e73bf2bd..a46b4065

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=9247&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9247&range=01-02

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/9247.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9247/head:pull/9247

PR: https://git.openjdk.org/jdk/pull/9247


More information about the core-libs-dev mailing list