RFR: 8303911: [testbug] Some unit tests call System.exit [v2]

Lukasz Kostyra lkostyra at openjdk.org
Thu Apr 20 11:17:26 UTC 2023


> Changes added to this commit resolve cases where `System.exit()` was used to leave tests abruptly. These tests, if System.exit() is hit, could potentially create some unwanted behavior from JTest (ex. silently skipping these tests instead of failing them).
> 
> There were some files which I did not modify, mostly because they were a part of a test sub-app, or tests would require major rework to make it work.
> 
> Two FXML tests were performance tests already written as separate apps, so I moved them to appropriate directory in tests/performance.

Lukasz Kostyra has updated the pull request incrementally with one additional commit since the last revision:

  Address review issues, update copyright dates

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/1101/files
  - new: https://git.openjdk.org/jfx/pull/1101/files/23bfddd3..b2756a1d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=1101&range=01
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1101&range=00-01

  Stats: 5 lines in 4 files changed: 0 ins; 0 del; 5 mod
  Patch: https://git.openjdk.org/jfx/pull/1101.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/1101/head:pull/1101

PR: https://git.openjdk.org/jfx/pull/1101


More information about the openjfx-dev mailing list