Fix: Return empty list when range exceeds collection size (#752)

Author: meetgoti07

core/src/main/java/org/kohsuke/stapler/export/Range.java

@@ -2,6 +2,7 @@
 
 import java.util.Arrays;
 import java.util.Collection;
+import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
 
@@ -28,6 +29,9 @@ public <T> List<T> apply(List<T> s) {
             s = s.subList(0, max);
         }
         if (min > 0) {
+            if (min >= s.size()) {
+                return Collections.emptyList();
+            }
             s = s.subList(min, s.size());
         }
         return s;

core/src/test/java/org/kohsuke/stapler/export/RangeTest.java

@@ -2,6 +2,8 @@
 
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.contains;
+import static org.hamcrest.Matchers.empty;
+import static org.hamcrest.Matchers.emptyIterable;
 
 import java.util.Arrays;
 import java.util.LinkedHashSet;
@@ -38,4 +40,20 @@ void minOnlyRange() {
         assertThat(r.apply(list), contains("e", "f"));
         assertThat(r.apply(set), contains("e", "f"));
     }
+
+    @Test
+    void rangeBeyondListSizeReturnsEmpty() {
+        Range r = new Range(10, 20);
+        assertThat(r.apply(array), empty());
+        assertThat(r.apply(list), empty());
+        assertThat(r.apply(set), emptyIterable());
+    }
+
+    @Test
+    void rangeExactlyAtBoundaryReturnsEmpty() {
+        Range r = new Range(6, 10);
+        assertThat(r.apply(array), empty());
+        assertThat(r.apply(list), empty());
+        assertThat(r.apply(set), emptyIterable());
+    }
 }