> As I 
> [mentioned](https://github.com/openjdk/jdk/pull/11266#issuecomment-3577688690)
>  in #11266, I wrote an automatic test `PageRangesAuto.java` that verifies if 
> page range is printed correctly. I used this test to validate the fix for 
> [JDK-8297191](https://bugs.openjdk.org/browse/JDK-8297191) in addition to the 
> existing one, `java/awt/print/PrinterJob/PageRanges.java`.
> 
> With this PR, I'm contributing the test to OpenJDK.
> 
> Without the fix for JDK-8297191, the test fails with the following error 
> messages:
> 
> <details>
> <summary>test log</summary>
> 
> 
> java.lang.Error: Not all pages printed in PRINTABLE within the range [2, 3]: 
> {2}
> java.lang.Error: Not all pages printed in PRINTABLE within the range [3, 6]: 
> {4, 5}
> java.lang.Error: Not all pages printed in PRINTABLE within the range [4, 7]: 
> {6}
> java.lang.Error: Not all pages printed in PRINTABLE within the range [7, 7]: 
> {}
> java.lang.Error: Not all pages printed in PRINTABLE within the range [9, 10]: 
> {}
> java.lang.Error: Not all pages printed in PRINTABLE within the range [10, 
> 10]: {}
> java.lang.Error: Not all pages printed in PAGEABLE within the range [2, 3]: 
> {2}
> java.lang.Error: Not all pages printed in PAGEABLE within the range [3, 6]: 
> {4, 5}
> java.lang.Error: Not all pages printed in PAGEABLE within the range [4, 7]: 
> {6}
> java.lang.Error: Not all pages printed in PAGEABLE within the range [7, 7]: {}
> java.lang.Error: Not all pages printed in PAGEABLE within the range [9, 10]: 
> {}
> java.lang.Error: Not all pages printed in PAGEABLE within the range [10, 10]: 
> {}
> Exception in thread "main" java.lang.RuntimeException: Errors detected: 12. -
> java.lang.Error: Not all pages printed in PRINTABLE within the range [2, 3]: 
> {2}
>       at PageRangesAuto.main(PageRangesAuto.java:146)
> 
> </details>

Alexey Ivanov has updated the pull request incrementally with one additional 
commit since the last revision:

  Use the first page of the rage in the filename

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/29660/files
  - new: https://git.openjdk.org/jdk/pull/29660/files/aa53a5c2..e075bba0

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=29660&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=29660&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/29660.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/29660/head:pull/29660

PR: https://git.openjdk.org/jdk/pull/29660

Reply via email to