Add preventative debug assert

See #941 (comment)
georust · Dec 6, 2022 · c61abfa · c61abfa
1 parent 4886fc7
commit c61abfa
Showing 1 changed file with 13 additions and 0 deletions.
diff --git a/geo/src/algorithm/simplify.rs b/geo/src/algorithm/simplify.rs
@@ -83,6 +83,7 @@ where
                 }
             },
         );
+    debug_assert_ne!(farthest_index, 0);
 
     if farthest_distance > *epsilon {
         // The farthest index was larger than epsilon, so we will recursively simplify subsegments
@@ -248,6 +249,18 @@ mod test {
     use crate::geo_types::coord;
     use crate::{line_string, polygon};
 
+    #[test]
+    fn recursion_test() {
+        let vec = [
+            coord! { x: 8.0, y: 100.0 },
+            coord! { x: 9.0, y: 100.0 },
+            coord! { x: 12.0, y: 100.0 },
+        ];
+        let compare = vec![coord! {x: 8.0, y: 100.0}, coord! {x: 12.0, y: 100.0}];
+        let simplified = rdp(vec.into_iter(), &1.0);
+        assert_eq!(simplified, compare);
+    }
+
     #[test]
     fn rdp_test() {
         let vec = vec![