fix: range creates array with only one element when end is 0

- because 0 is falsy the start was overwritten with 0 when end is 0 resulting in the array [0]
- explicitly check for end !== undefined to prevent this
- add test case where end is 0

Author: Richard-Krug

cdn/radash.esm.js

@@ -206,7 +206,7 @@ const unique = (array, toKey) => {
 };
 function* range(startOrLength, end, valueOrMapper = (i) => i, step = 1) {
   const mapper = isFunction(valueOrMapper) ? valueOrMapper : () => valueOrMapper;
-  const start = end ? startOrLength : 0;
+  const start = end !== void 0 ? startOrLength : 0;
   const final = end ?? startOrLength;
   for (let i = start; i <= final; i += step) {
     yield mapper(i);

cdn/radash.js

@@ -209,7 +209,7 @@ var radash = (function (exports) {
   };
   function* range(startOrLength, end, valueOrMapper = (i) => i, step = 1) {
     const mapper = isFunction(valueOrMapper) ? valueOrMapper : () => valueOrMapper;
-    const start = end ? startOrLength : 0;
+    const start = end !== void 0 ? startOrLength : 0;
     const final = end ?? startOrLength;
     for (let i = start; i <= final; i += step) {
       yield mapper(i);

cdn/radash.min.js

@@ -1,6 +1,6 @@
 {
   "name": "radash",
-  "version": "12.1.1",
+  "version": "12.1.2",
   "description": "Functional utility library - modern, simple, typed, powerful",
   "main": "dist/cjs/index.cjs",
   "module": "dist/esm/index.mjs",

package.json

@@ -325,7 +325,7 @@ export function* range<T = number>(
   step: number = 1
 ): Generator<T> {
   const mapper = isFunction(valueOrMapper) ? valueOrMapper : () => valueOrMapper
-  const start = end ? startOrLength : 0
+  const start = end !== undefined ? startOrLength : 0
   const final = end ?? startOrLength
   for (let i = start; i <= final; i += step) {
     yield mapper(i)

src/array.ts

@@ -404,6 +404,7 @@ describe('array module', () => {
       assert.deepEqual(toList(_.range(0, 4)), [0, 1, 2, 3, 4])
       assert.deepEqual(toList(_.range(3)), [0, 1, 2, 3])
       assert.deepEqual(toList(_.range(0, 3)), [0, 1, 2, 3])
+      assert.deepEqual(toList(_.range(-3, 0)), [-3, -2, -1, 0])
       assert.deepEqual(toList(_.range(0, 3, 'y')), ['y', 'y', 'y', 'y'])
       assert.deepEqual(toList(_.range(0, 3, () => 'y')), ['y', 'y', 'y', 'y'])
       assert.deepEqual(toList(_.range(0, 3, i => i)), [0, 1, 2, 3])