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