RFR: 8358697: TextLayout/MyanmarTextTest.java passes if no Myanmar font is found [v6]

SendaoYan syan at openjdk.org
Tue Jun 24 02:23:39 UTC 2025


On Thu, 19 Jun 2025 11:11:32 GMT, Manukumar V S <mvs at openjdk.org> wrote:

>> 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:
> 
>   Formatting changes, removed unwanted code

Test-fix only, throw SkippedException instead print the message.

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

PR Comment: https://git.openjdk.org/jdk/pull/25879#issuecomment-2998549922


More information about the client-libs-dev mailing list