On Tue, 30 Jan 2024 04:35:02 GMT, Prasanta Sadhukhan <[email protected]>
wrote:
>> vtstydev has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fix header and imports section in AlphaPrintingOffsets
>
> test/jdk/java/awt/print/PrinterJob/ImagePrinting/AlphaPrintingOffsets.java
> line 73:
>
>> 71: System.out.println("Printer not configured or available."
>> 72: + " Test cannot continue.");
>> 73: PassFailJFrame.forcePass();
>
> I guess it is now being preferred to throw jtreg.SkippedException instead of
> forcePass for printers not available scenarios?!
PrintGlyphVectorTest, ClippedImages use `PassFailJFrame.forcePass();`
I guided these samples.
If you want I to to use `throw` istead. ok.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17030#discussion_r1472373915
