RFR: 8320443: [macos] Test java/awt/print/PrinterJob/PrinterDevice.java fails on macOS [v2]
Phil Race
prr at openjdk.org
Tue Dec 5 20:13:56 UTC 2023
On Tue, 5 Dec 2023 20:07:15 GMT, Alexander Zvegintsev <azvegint at openjdk.org> wrote:
>> 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."
Not sufficient if something catches the exception as used to happen
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16773#discussion_r1416222101
More information about the client-libs-dev
mailing list