RFR: 8285687 : Remove jtreg tag manual=yesno for ava/awt/print/PrinterJob/PageRangesDlgTest.java [v3]

Sergey Bylokhov serb at openjdk.java.net
Fri Apr 29 00:43:57 UTC 2022


On Thu, 28 Apr 2022 22:29:51 GMT, lawrence.andrews <duke at openjdk.java.net> wrote:

>> 1) Fixed Parser error by removing  yesno from @run main/manual=yesno 
>> 2) Used PassFaileJFrame to show the test instruction to the user instead of printing the test instruction on the console or jtreg log file. 
>> 3) If printer is not configured then mark the test as passed.
>> 
>> @shurymury 
>> @aivanov-jdk
>
> lawrence.andrews has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - Merge branch 'openjdk:master' into JDK-8285687
>  - Added @key printer since this test needs printer
>  - 8285687 : Remove jtreg tag manual=yesno for ava/awt/print/PrinterJob/PageRangesDlgTest.java

Since the test now uses the printer keyword I doubt that it should pass silently if the printer is not installed.

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

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



More information about the client-libs-dev mailing list