On Mon, 4 Jan 2021 10:27:12 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> Test fails with timeout and it seems likely due to non-availability of 
>> printer in the test system. We should just update the test to check for 
>> default (or available) printers and return if there is no printer.
>
> Prasanta Sadhukhan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Fail the test if no printer is found

test/jdk/java/awt/print/PrinterJob/LandscapeStackOverflow.java line 50:

> 48:         PrintService defaultPrtSrv = 
> PrintServiceLookup.lookupDefaultPrintService();
> 49:         if (printjob.getPrintService() == null || defaultPrtSrv == null) {
> 50:             throw new RuntimeException("No printer found");

We need to make this more robust.
The checks should be before we create the pritner job and should be to find ANY 
printer and none
throws the excepton, but if there is only a non-default one, set that as the 
printer for the job after creating the job.

That may not matter on Windows but on other platforms I am sure you can have 
printers, but not a default one.

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

PR: https://git.openjdk.java.net/jdk/pull/1927

Reply via email to