fix (objectify): Change return type from Record<Key, Value> to Partial<Record<Key, Value>> and update tests to handle possible undefined values.

Author: Yukiniro

src/array/objectify.ts

@@ -21,12 +21,9 @@ export function objectify<T, Key extends string | number | symbol, Value = T>(
   array: readonly T[],
   getKey: (item: T) => Key,
   getValue: (item: T) => Value = item => item as unknown as Value,
-): Record<Key, Value> {
-  return array.reduce(
-    (acc, item) => {
-      acc[getKey(item)] = getValue(item)
-      return acc
-    },
-    {} as Record<Key, Value>,
-  )
+): Partial<Record<Key, Value>> {
+  return array.reduce((acc, item) => {
+    acc[getKey(item)] = getValue(item)
+    return acc
+  }, {} as Partial<Record<Key, Value>>)
 }

tests/array/objectify.test.ts

@@ -14,8 +14,8 @@ describe('objectify', () => {
       x => x.id,
       x => x,
     )
-    expect(result.a.word).toBe('hello')
-    expect(result.b.word).toBe('bye')
+    expect(result.a?.word).toBe('hello')
+    expect(result.b?.word).toBe('bye')
   })
   test('does not fail on empty input list', () => {
     const result = _.objectify(