Fix getClusterExpansionZoom for maxZoom (#106)

* add failing test * fix failing test
mapbox · Nov 5, 2018 · d26e515 · d26e515
1 parent 0851f7a
commit d26e515
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -155,7 +155,7 @@ class SuperCluster {
 
     getClusterExpansionZoom(clusterId) {
         let clusterZoom = (clusterId % 32) - 1;
-        while (clusterZoom < this.options.maxZoom) {
+        while (clusterZoom <= this.options.maxZoom) {
             const children = this.getChildren(clusterId);
             clusterZoom++;
             if (children.length !== 1) break;

diff --git a/test/test.js b/test/test.js
@@ -62,6 +62,17 @@ test('returns cluster expansion zoom', (t) => {
     t.end();
 });
 
+test('returns cluster expansion zoom for maxZoom', (t) => {
+    const index = supercluster({
+        radius: 60,
+        extent: 256,
+        maxZoom: 4,
+    }).load(places.features);
+
+    t.same(index.getClusterExpansionZoom(2436), 5);
+    t.end();
+});
+
 test('aggregates cluster properties with reduce', (t) => {
     const index = supercluster({
         initial: () => ({sum: 0}),