On Thu, 19 Jun 2025 11:11:32 GMT, Manukumar V S <m...@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

Reply via email to