test: add more types to test

Author: Hweinstock

packages/core/src/shared/utilities/collectionUtils.ts

@@ -305,7 +305,7 @@ export function assign<T extends Record<any, any>, U extends Partial<T>>(data: T
  * @param depth
  * @param omitKeys Omit properties matching these names (at any depth).
  * @param replacement Replacement for object whose fields extend beyond `depth`, and properties matching `omitKeys`.
- * @param maxStringLength truncates string values that exceed this threshold.
+ * @param maxStringLength truncates string values that exceed this threshold (includes values in nested arrays)
  */
 export function partialClone(
     obj: any,
@@ -318,8 +318,11 @@ export function partialClone(
 ): any {
     // Base case: If input is not an object or has no children, return it.
     if (typeof obj !== 'object' || obj === null || 0 === Object.getOwnPropertyNames(obj).length) {
-        const maxLength = options?.maxStringLength
-        return typeof obj === 'string' && maxLength !== undefined ? truncate(obj, maxLength, '...') : obj
+        if (typeof obj === 'string' && options?.maxStringLength) {
+            const maxStringLength = options?.maxStringLength
+            return truncate(obj, maxStringLength, '...')
+        }
+        return obj
     }
 
     // Create a new object of the same type as the input object.

packages/core/src/test/shared/utilities/collectionUtils.test.ts

@@ -770,26 +770,36 @@ describe('CollectionUtils', async function () {
 
         it('truncates properties by maxLength', function () {
             const testObj = {
-                a: '1',
-                b: '11',
-                c: '11111',
-                d: {
-                    e: {
-                        a: '11111',
-                        b: '11',
+                strValue: '1',
+                boolValue: true,
+                longString: '11111',
+                nestedObj: {
+                    nestedObjAgain: {
+                        longNestedStr: '11111',
+                        shortNestedStr: '11',
                     },
                 },
+                nestedObj2: {
+                    functionValue: (_: unknown) => {},
+                },
+                nestedObj3: {
+                    myArray: ['1', '11111', '1'],
+                },
+                objInArray: [{ shortString: '11', longString: '11111' }],
             }
             assert.deepStrictEqual(partialClone(testObj, 5, [], { maxStringLength: 2 }), {
-                a: '1',
-                b: '11',
-                c: '11...',
-                d: {
-                    e: {
-                        a: '11...',
-                        b: '11',
+                ...testObj,
+                longString: '11...',
+                nestedObj: {
+                    nestedObjAgain: {
+                        longNestedStr: '11...',
+                        shortNestedStr: '11',
                     },
                 },
+                nestedObj3: {
+                    myArray: ['1', '11...', '1'],
+                },
+                objInArray: [{ shortString: '11', longString: '11...' }],
             })
         })
     })