fix(range): fix range calculation when step is set (#4398)

Author: Yannick Croissant

src/lib/utils/__tests__/range-test.ts

@@ -13,20 +13,35 @@ describe('range', () => {
     expect(range({ end: 20, step: 5 })).toEqual([0, 5, 10, 15]);
   });
 
+  test('with start, end and step', () => {
+    expect(range({ start: 1300, end: 1350, step: 5 })).toEqual([
+      1300,
+      1305,
+      1310,
+      1315,
+      1320,
+      1325,
+      1330,
+      1335,
+      1340,
+      1345,
+    ]);
+  });
+
   test('rounds decimal array lengths', () => {
     // (5000 - 1) / 500 = 9.998 so we want the array length to be rounded to 10.
     expect(range({ start: 1, end: 5000, step: 500 })).toHaveLength(10);
     expect(range({ start: 1, end: 5000, step: 500 })).toEqual([
-      500,
-      1000,
-      1500,
-      2000,
-      2500,
-      3000,
-      3500,
-      4000,
-      4500,
-      5000,
+      1,
+      501,
+      1001,
+      1501,
+      2001,
+      2501,
+      3001,
+      3501,
+      4001,
+      4501,
     ]);
   });
 });

src/lib/utils/range.ts

@@ -16,7 +16,7 @@ function range({ start = 0, end, step = 1 }: RangeOptions): number[] {
   const arrayLength = Math.round((end - start) / limitStep);
 
   return [...Array(arrayLength)].map(
-    (_, current) => (start + current) * limitStep
+    (_, current) => start + current * limitStep
   );
 }