RFR: 8320443: [macos] Test java/awt/print/PrinterJob/PrinterDevice.java fails on macOS [v2]

Alexander Zvegintsev azvegint at openjdk.org
Tue Dec 5 20:09:35 UTC 2023


On Tue, 5 Dec 2023 19:41:54 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Phil Race has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8320443
>
> test/jdk/java/awt/print/PrinterJob/PrinterDevice.java line 52:
> 
>> 50: public class PrinterDevice implements Printable {
>> 51: 
>> 52:     static volatile boolean failed = false;
> 
> Is it really needed? In all the cases where you assign `true` to the `failed` field, you also explicitly throw an exception, which is enough to fail the test.

It is to test the "the macOS printing implementation was swallowing exceptions it should not."

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16773#discussion_r1416221561


More information about the client-libs-dev mailing list