@lumx/core 3.20.1-alpha.28 → 3.20.1-alpha.29

package/js/utils/collection/chunk.js

@@ -1,8 +1,13 @@
 /** Chunk array in slices of given size */
 function chunk(input, size) {
-  return input.reduce((arr, item, idx) => {
-    return idx % size === 0 ? [...arr, [item]] : [...arr.slice(0, -1), [...arr.slice(-1)[0], item]];
-  }, []);
+  if (size <= 0) {
+    throw new Error('Size must be greater than 0');
+  }
+  const result = [];
+  for (let i = 0; i < input.length; i += size) {
+    result.push(input.slice(i, i + size));
+  }
+  return result;
 }
 
 export { chunk };

package/js/utils/collection/chunk.test.js

@@ -10,4 +10,19 @@ describe(chunk, () => {
   it('should chunk array with size not perfectly dividing the array length', () => {
     expect(chunk([1, 2, 3, 4, 5], 2)).toEqual([[1, 2], [3, 4], [5]]);
   });
+  it('should chunk array with size perfectly dividing the array length', () => {
+    expect(chunk([1, 2, 3, 4], 2)).toEqual([[1, 2], [3, 4]]);
+  });
+  it('should work with size of 1', () => {
+    expect(chunk([1, 2, 3], 1)).toEqual([[1], [2], [3]]);
+  });
+  it('should work with size equal to array length', () => {
+    expect(chunk([1, 2, 3], 3)).toEqual([[1, 2, 3]]);
+  });
+  it('should throw error when size is 0', () => {
+    expect(() => chunk([1, 2, 3], 0)).toThrow('Size must be greater than 0');
+  });
+  it('should throw error when size is negative', () => {
+    expect(() => chunk([1, 2, 3], -1)).toThrow('Size must be greater than 0');
+  });
 });

package/js/utils/collection/chunk.test.ts

@@ -12,4 +12,27 @@ describe(chunk, () => {
     it('should chunk array with size not perfectly dividing the array length', () => {
         expect(chunk([1, 2, 3, 4, 5], 2)).toEqual([[1, 2], [3, 4], [5]]);
     });
+
+    it('should chunk array with size perfectly dividing the array length', () => {
+        expect(chunk([1, 2, 3, 4], 2)).toEqual([
+            [1, 2],
+            [3, 4],
+        ]);
+    });
+
+    it('should work with size of 1', () => {
+        expect(chunk([1, 2, 3], 1)).toEqual([[1], [2], [3]]);
+    });
+
+    it('should work with size equal to array length', () => {
+        expect(chunk([1, 2, 3], 3)).toEqual([[1, 2, 3]]);
+    });
+
+    it('should throw error when size is 0', () => {
+        expect(() => chunk([1, 2, 3], 0)).toThrow('Size must be greater than 0');
+    });
+
+    it('should throw error when size is negative', () => {
+        expect(() => chunk([1, 2, 3], -1)).toThrow('Size must be greater than 0');
+    });
 });

package/js/utils/collection/chunk.ts

@@ -1,6 +1,11 @@
 /** Chunk array in slices of given size */
 export function chunk<T>(input: Array<T>, size: number): T[][] {
-    return input.reduce((arr, item, idx) => {
-        return idx % size === 0 ? [...arr, [item]] : [...arr.slice(0, -1), [...arr.slice(-1)[0], item]];
-    }, [] as T[][]);
+    if (size <= 0) {
+        throw new Error('Size must be greater than 0');
+    }
+    const result: T[][] = [];
+    for (let i = 0; i < input.length; i += size) {
+        result.push(input.slice(i, i + size));
+    }
+    return result;
 }

package/js/utils/collection/isEmpty.test.js

@@ -6,6 +6,7 @@ describe(isEmpty, () => {
     expect(isEmpty(null)).toBe(true);
     expect(isEmpty(0)).toBe(true);
     expect(isEmpty('')).toBe(true);
+    expect(isEmpty(false)).toBe(true);
   });
   it('should return true for empty object or array', () => {
     expect(isEmpty([])).toBe(true);

package/js/utils/collection/isEmpty.test.ts

@@ -6,6 +6,7 @@ describe(isEmpty, () => {
         expect(isEmpty(null)).toBe(true);
         expect(isEmpty(0)).toBe(true);
         expect(isEmpty('')).toBe(true);
+        expect(isEmpty(false)).toBe(true);
     });
 
     it('should return true for empty object or array', () => {

package/js/utils/collection/isEmpty.ts

@@ -1,4 +1,4 @@
 import type { Falsy } from '../../types';
 
 /** Check if object or array is empty (true on falsy values) */
-export const isEmpty = (obj: {} | Falsy) => !obj || Object.entries(obj).length === 0;
+export const isEmpty = (obj: NonNullable<unknown> | Falsy) => !obj || Object.entries(obj).length === 0;

package/js/utils/collection/last.test.js

@@ -0,0 +1,19 @@
+import { last } from '@lumx/core/js/utils/collection/last';
+
+describe(last, () => {
+  it('should return undefined for empty array', () => {
+    const input = [];
+    const output = last(input);
+    expect(output).toBeUndefined();
+  });
+  it('should return last element from array with single element', () => {
+    const input = [42];
+    const output = last(input);
+    expect(output).toBe(42);
+  });
+  it('should return last element from array with multiple elements', () => {
+    const input = [1, 2, 3, 4, 5];
+    const output = last(input);
+    expect(output).toBe(5);
+  });
+});

package/js/utils/collection/last.test.ts

@@ -0,0 +1,21 @@
+import { last } from '@lumx/core/js/utils/collection/last';
+
+describe(last, () => {
+    it('should return undefined for empty array', () => {
+        const input: number[] = [];
+        const output = last(input);
+        expect(output).toBeUndefined();
+    });
+
+    it('should return last element from array with single element', () => {
+        const input = [42];
+        const output = last(input);
+        expect(output).toBe(42);
+    });
+
+    it('should return last element from array with multiple elements', () => {
+        const input = [1, 2, 3, 4, 5];
+        const output = last(input);
+        expect(output).toBe(5);
+    });
+});

package/js/utils/collection/range.test.js

@@ -2,6 +2,7 @@ import { range } from './range.js';
 
 describe(range, () => {
   it('should generate a number range', () => {
+    expect(range(-2)).toEqual([]);
     expect(range(0)).toEqual([]);
     expect(range(1)).toEqual([0]);
     expect(range(5)).toEqual([0, 1, 2, 3, 4]);

package/js/utils/collection/range.test.ts

@@ -2,6 +2,7 @@ import { range } from './range';
 
 describe(range, () => {
     it('should generate a number range', () => {
+        expect(range(-2)).toEqual([]);
         expect(range(0)).toEqual([]);
         expect(range(1)).toEqual([0]);
         expect(range(5)).toEqual([0, 1, 2, 3, 4]);

package/package.json

@@ -35,7 +35,7 @@
         "update-version-changelog": "yarn version-changelog ../../CHANGELOG.md"
     },
     "sideEffects": false,
-    "version": "3.20.1-alpha.28",
+    "version": "3.20.1-alpha.29",
     "devDependencies": {
         "@babel/preset-typescript": "^7.26.0",
         "@rollup/plugin-babel": "^6.0.4",

package/js/utils/collection/pull.js

@@ -1,9 +0,0 @@
-/** Pull an element from an array (inverse of array.push) */
-const pull = (array, element) => {
-  const index = array.indexOf(element);
-  if (index > -1) {
-    array.splice(index, 1);
-  }
-};
-
-export { pull };

package/js/utils/collection/pull.test.js

@@ -1,16 +0,0 @@
-import { pull } from '@lumx/core/js/utils/collection//pull';
-
-describe(pull, () => {
-  it('should do nothing if element does not exist', () => {
-    const a = [1, 2];
-    pull(a, 0);
-    expect(a).toBe(a);
-    expect(a).toEqual([1, 2]);
-  });
-  it('should pull an element from the array', () => {
-    const a = [1, 2];
-    pull(a, 1);
-    expect(a).toBe(a);
-    expect(a).toEqual([2]);
-  });
-});

package/js/utils/collection/pull.test.ts

@@ -1,17 +0,0 @@
-import { pull } from '@lumx/core/js/utils/collection//pull';
-
-describe(pull, () => {
-    it('should do nothing if element does not exist', () => {
-        const a = [1, 2];
-        pull(a, 0);
-        expect(a).toBe(a);
-        expect(a).toEqual([1, 2]);
-    });
-
-    it('should pull an element from the array', () => {
-        const a = [1, 2];
-        pull(a, 1);
-        expect(a).toBe(a);
-        expect(a).toEqual([2]);
-    });
-});

package/js/utils/collection/pull.ts

@@ -1,7 +0,0 @@
-/** Pull an element from an array (inverse of array.push) */
-export const pull = <T>(array: Array<T>, element: T) => {
-    const index = array.indexOf(element);
-    if (index > -1) {
-        array.splice(index, 1);
-    }
-};