RFR: 8325762: Use PassFailJFrame.Builder.splitUI() in PrintLatinCJKTest.java [v2]
    Tejesh R 
    tr at openjdk.org
       
    Tue Feb 20 06:58:56 UTC 2024
    
    
  
On Wed, 14 Feb 2024 14:06:15 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> The `test/jdk/java/awt/print/PrinterJob/PrintLatinCJKTest.java` displays a single *Print* button as its test UI. To simplify the UI, I used the new `splitUI` method from #17845.
>> 
>> In addition to it, I refactored the test:
>> 
>> - Extracted the rendered text into a constant to ensure the same text is printed and displayed in the instructions.
>> - Displayed an error message to the tester if `PrinterException` is caught and automatically failed the test.
>
> Alexey Ivanov has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits:
> 
>  - Merge 8294148-splitUI
>  - Revert the condition in print()
>  - Update copyright year
>  - Handle PrinterException
>  - Ensure the same text is printed and displayed in the instructions
>  - 8325762: Use split UI for PrintLatinCJKTest
test/jdk/java/awt/print/PrinterJob/PrintLatinCJKTest.java line 53:
> 51: 
> 52:     private static final String INFO = """
> 53:             You need a printer for this test. If you have none, let
Can we use `PrinterJob.lookupPrintServices().length` to check for printer availability?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17847#discussion_r1495322659
    
    
More information about the client-libs-dev
mailing list