Add sanity check for stepSize (#9679)

diff --git a/src/scales/scale.linearbase.js b/src/scales/scale.linearbase.js
index dc00265ee194b7cc95ab243d25bb1f5c7c546a04..0930d38f5549add0419adfe0b20d2d568a5fdf80 100644 (file)
--- a/src/scales/scale.linearbase.js
+++ b/src/scales/scale.linearbase.js
@@ -222,6 +222,10 @@ export default class LinearScaleBase extends Scale {
 
     if (stepSize) {
       maxTicks = Math.ceil(this.max / stepSize) - Math.floor(this.min / stepSize) + 1;
+      if (maxTicks > 1000) {
+        console.warn(`scales.${this.id}.ticks.stepSize: ${stepSize} would result generating up to ${maxTicks} ticks. Limiting to 1000.`);
+        maxTicks = 1000;
+      }
     } else {
       maxTicks = this.computeTickLimit();
       maxTicksLimit = maxTicksLimit || 11;

diff --git a/test/specs/scale.linear.tests.js b/test/specs/scale.linear.tests.js
index e2ad37e41a20d5948325ca8d38c89faad210b315..3307c39621af2e9e9fb0a4d171d3913c1407c45a 100644 (file)
--- a/test/specs/scale.linear.tests.js
+++ b/test/specs/scale.linear.tests.js
@@ -639,6 +639,29 @@ describe('Linear Scale', function() {
     expect(getLabels(chart.scales.y)).toEqual(['1', '3', '5', '7', '9', '11']);
   });
 
+  it('Should not generate insane amounts of ticks with small stepSize and large range', function() {
+    var chart = window.acquireChart({
+      type: 'bar',
+      options: {
+        scales: {
+          y: {
+            type: 'linear',
+            min: 1,
+            max: 1E10,
+            ticks: {
+              stepSize: 2,
+              autoSkip: false
+            }
+          }
+        }
+      }
+    });
+
+    expect(chart.scales.y.min).toBe(1);
+    expect(chart.scales.y.max).toBe(1E10);
+    expect(chart.scales.y.ticks.length).toBeLessThanOrEqual(1000);
+  });
+
   it('Should create decimal steps if stepSize is a decimal number', function() {
     var chart = window.acquireChart({
       type: 'bar',