Skip to content

Commit eaaa502

Browse files
committed
[java] Allow setting print options page ranges using list
1 parent c049595 commit eaaa502

File tree

2 files changed

+16
-0
lines changed

2 files changed

+16
-0
lines changed

java/src/org/openqa/selenium/print/PrintOptions.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,9 @@
1717

1818
package org.openqa.selenium.print;
1919

20+
import java.util.ArrayList;
2021
import java.util.HashMap;
22+
import java.util.List;
2123
import java.util.Map;
2224
import org.openqa.selenium.internal.Require;
2325

@@ -71,6 +73,11 @@ public void setPageRanges(String firstRange, String... ranges) {
7173
}
7274
}
7375

76+
public void setPageRanges(List<String> ranges) {
77+
this.pageRanges = new String[ranges.size()];
78+
this.pageRanges = ranges.toArray(this.pageRanges);
79+
}
80+
7481
public void setBackground(boolean background) {
7582
this.background = Require.nonNull("background", background);
7683
}

java/test/org/openqa/selenium/print/PrintOptionsTest.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919

2020
import static org.assertj.core.api.Assertions.assertThat;
2121

22+
import java.util.List;
2223
import java.util.Map;
2324
import org.junit.jupiter.api.Tag;
2425
import org.junit.jupiter.api.Test;
@@ -48,6 +49,14 @@ void setsValuesAsPassed() {
4849
assertThat(printOptions.getShrinkToFit()).isFalse();
4950
}
5051

52+
@Test
53+
void setsPageRanges() {
54+
PrintOptions printOptions = new PrintOptions();
55+
56+
printOptions.setPageRanges(List.of("1-2", "6-7"));
57+
assertThat(printOptions.getPageRanges().length).isEqualTo(2);
58+
}
59+
5160
@Test
5261
void toMapContainsProperKey() {
5362
PrintOptions printOptions = new PrintOptions();

0 commit comments

Comments
 (0)