> Issue: > MyanmarTextTest.java produces a false positive result when some of the test > preconditions are not met. It checks whether certain fonts are present in the > system, for example, 'Padauk' on Linux. If the required font is missing, the > test simply returns early, and the test ends up passing, which is incorrect. > Ideally, it should throw a jtreg.SkippedException when the necessary > preconditions are not satisfied. > > Another scenario is that the test passes on headless machines even though it > creates GUI components. Ideally, when GUI components are created in code > running on a headless machine, a HeadlessException should be thrown. However, > since MyanmarTextTest.java exits before reaching the point where the GUI is > created (due to unmet preconditions), it incorrectly reports a pass. This > behavior may lead to a misinterpretation of the test as being headless, which > it is not. > > Fix: > Need to throw jtreg.SkippedException in cases where some pre-conditions for > running the test are not met. > > Testing: > Tested using mach5 in all available platforms and got full PASS.
Manukumar V S has updated the pull request incrementally with one additional commit since the last revision: Combined the fonts in to one list, formatting changes ------------- Changes: - all: https://git.openjdk.org/jdk/pull/25879/files - new: https://git.openjdk.org/jdk/pull/25879/files/bdc40569..b5a3f894 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=25879&range=04 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25879&range=03-04 Stats: 38 lines in 1 file changed: 2 ins; 23 del; 13 mod Patch: https://git.openjdk.org/jdk/pull/25879.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/25879/head:pull/25879 PR: https://git.openjdk.org/jdk/pull/25879
