Hi Philip, Thanks for the review comments. The testcase test/jdk/java/awt/print/PrinterJob/TestMediaTraySelection.java seems to be intended to test only the MediaTray functionality. Also, it doesn't seems to have anything that is Linux specific.
Rather than having a new testcase wouldn't it be fine to have os.family == "mac" change alone to the existing one.? Regarding the testing, the patch was already tested by our customer using a multi tray printer and was found to be working. Owing to WFH factor due to COVID19, multi tray printer is not accessible to us at this point of time. May I request if the community can help us do further testing with the multi tray printer. Thanks & Regards Vipin MV -----Philip Race <philip.r...@oracle.com> wrote: ----- To: Vipin Mv1 <vipin...@in.ibm.com> From: Philip Race <philip.r...@oracle.com> Date: 08/30/2020 10:02PM Cc: 2d-dev@openjdk.java.net Subject: [EXTERNAL] Re: [OpenJDK 2D-Dev] RFR: 8234393 [macos] printing ignores printer tray PS there is an existing manual regression test but it is currently geared towards Linux and in fact is set to run only on Linux test/jdk/java/awt/print/PrinterJob/TestMediaTraySelection.java So you still may find it easier to create a new test rather than modify this one to avoid breaking what it tests. -phil On 8/29/20, 11:10 AM, Philip Race wrote: > PS, there's a test case in the bug. Seems like it could be used as the > basis for a manual regression test. > > Make sure you use @requires printer as well as adding the manual and > headful keywords > and next you'll have to check for an installed printer with multiple > trays > > -phil > > On 8/27/20, 11:36 AM, Philip Race wrote: >> This looks reasonable but we need to test it first before approving it. >> >> -phil. >> >> On 8/27/20, 6:16 AM, Vipin Mv1 wrote: >>> Hi, >>> >>> Please find below a patch for the following issue. >>> >>> https://urldefense.proofpoint.com/v2/url?u=https-3A__bugs.openjdk.java.net_browse_JDK-2D8234393&d=DwICaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=YGcJOWSLdBGnjzzUPMOeoQB-1IAycTyy6T7cx_ZtPhM&m=SK-6ut1nk3emw0KXZM-vnvEkPaR30_91LDrjdVlgGXQ&s=79-ArdhpL1L3pemMcoI_Gcu8U-DYOGA-WHiRSN-nmeo&e= >>> >>> >>> >>> https://urldefense.proofpoint.com/v2/url?u=http-3A__cr.openjdk.java.net_-7Ealeonard_8234393_webrev.00&d=DwICaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=YGcJOWSLdBGnjzzUPMOeoQB-1IAycTyy6T7cx_ZtPhM&m=SK-6ut1nk3emw0KXZM-vnvEkPaR30_91LDrjdVlgGXQ&s=7JlYOmTp1P0BSU_rcAmw2HQA6Qhmd-VuEasiHDr1tjc&e= >>> >>> >>> Thanks& Regards >>> Vipin MV >>> >>>