On Thu, 7 Nov 2024 17:49:22 GMT, Daniel Gredler <d...@openjdk.org> wrote:

>> There are multiple issue with this test case
>> 1) Parser error due to yesno in @run main/manual=yesno
>> 2) User can only compare the UI rendering and compare with the print out. 
>> User can't mark the test as pass or fail due to pass or fail buttons are 
>> missing.
>> 3) When the test is executed using jtreg after user click on the print 
>> button on the print dialog the whole test UIs ( frames) gets dispose and 
>> user cannot compare the printout with the UI. But this works as expected in 
>> test is running individually using java PrintTextTest
>
> Daniel Gredler has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Whitespace fixes

Marked as reviewed by honkar (Reviewer).

test/jdk/ProblemList.txt line 809:

> 807: java/awt/Frame/FrameStateTest/FrameStateTest.java 8203920 
> macosx-all,linux-all
> 808: java/awt/print/PrinterJob/ScaledText/ScaledText.java 8231226 macosx-all
> 809: java/awt/print/PrinterJob/PrintTextTest.java 8148334 generic-all

Looks like the test has issues (test window vs print to pdf/printer- glyphs 
look different) on all platforms and problem listing on all platforms 
(generic-all) is necessary.

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

PR Review: https://git.openjdk.org/jdk/pull/21716#pullrequestreview-2421798244
PR Review Comment: https://git.openjdk.org/jdk/pull/21716#discussion_r1833153441

Reply via email to