RFR: JDK-8299052 : ViewportOverlapping test fails intermittently on Win10 & Win11 [v2]
Alexey Ivanov
aivanov at openjdk.org
Wed Jun 7 11:22:55 UTC 2023
On Tue, 6 Jun 2023 22:55:06 GMT, Harshitha Onkar <honkar at openjdk.org> wrote:
>> test/jdk/java/awt/Mixing/AWT_Mixing/ViewportOverlapping.java line 164:
>>
>>> 162: OverlappingTestBase.doMain(args);
>>> 163: captureScreen("Img_5.png");
>>> 164: }
>>
>> It is just a test, so no need to over-engineer it but it 'feels wrong' to specify the file name
>> as foo.png here and separately inside captureScreen choose 'png' as the file type to use.
>> Perhaps captureScreen("Img_5.png", "png") ?
>
> Updated
It doesn't feel right this way either. A cleaner approach would be to pass the undecorated file name and add the extension and specify the format inside `captureScreen`. (Passing the same value as a parameter usually produces an IDE warning, which means the parameter is not necessary.)
captureScreen("Img_5");
ImageIO.write(
robot.createScreenCapture(new Rectangle(0, 0, screenSize.width, screenSize.height)),
"png",
new File(filename + ".png")
);
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14289#discussion_r1221410051
More information about the client-libs-dev
mailing list