Ensure _chooseSplitIndex always return a number. Fix #69 (#94)

mourner · Jul 31, 2019 · 15c31bd · 15c31bd
1 parent 4b11244
commit 15c31bd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -346,7 +346,7 @@ export default class RBush {
             }
         }
 
-        return index;
+        return index || M - m;
     }
 
     // sorts node children by the best axis for split

diff --git a/test/test.js b/test/test.js
@@ -174,6 +174,8 @@ t('#insert handles the insertion of maxEntries + 2 empty bboxes', (t) => {
 
     t.equal(tree.toJSON().height, 2);
     sortedEqual(t, tree.all(), emptyData);
+    t.equal(tree.data.children[0].children.length, 4);
+    t.equal(tree.data.children[1].children.length, 2);
 
     t.end();
 });