> Please review changes to the IntlTest (test framework) class. 
> 
> These changes include
> - Logging the actual exception + stack trace. Previously, the test framework 
> would throw `InvocationTargetException` but hide the actual underlying 
> exception of the failing test unless the test class was ran with -nothrow.
> - Sorting the tests, ensuring for any given run, the tests are ran in the 
> same order. (In [JDK-8305853](https://bugs.openjdk.org/browse/JDK-8305853) it 
> was discovered that on Windows, the test framework would execute tests in a 
> random order each time).
> - Improving output.

Justin Lu has updated the pull request incrementally with one additional commit 
since the last revision:

  Replace toHexString() with HexFormat

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/13655/files
  - new: https://git.openjdk.org/jdk/pull/13655/files/a395ba67..773da120

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

  Stats: 70 lines in 2 files changed: 26 ins; 14 del; 30 mod
  Patch: https://git.openjdk.org/jdk/pull/13655.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/13655/head:pull/13655

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

Reply via email to