Fix getEntitiesValue returning an empty string if the value is false

src/actions/bind-entity-int-deferred.ts

@@ -29,7 +29,7 @@ export function bindEntityIntDeferred<
             setByPath(
               doc.entities[id],
               path,
-              parseInt(value || "0") as PathValue<T, typeof path>,
+              parseInt(value ?? "0") as PathValue<T, typeof path>,
             );
           });
           return doc;

src/actions/utilities.ts

@@ -13,7 +13,7 @@ export const getEntitiesValue = <
   const values = new Set(ids.map((id) => getByPath(doc.entities[id], path)));
 
   if (values.size === 1) {
-    return (getByPath(doc.entities[ids[0]], path) as string) || "";
+    return (getByPath(doc.entities[ids[0]], path) as string) ?? "";
   } else {
     return "";
   }

tests/actions/entity-deferred.test.ts

@@ -220,7 +220,7 @@ Object.entries({
       expect(input.value).toBe(toType);
 
       input.reset();
-      expect(input.value).toBe("");
+      expect(input.value).toBe("")
       expect(get(store).people.entities["1"][field]).toBe(
         document.people.entities["1"][field],
       );

tests/data.ts

@@ -19,6 +19,11 @@ export type DocumentType = {
 export type Person = {
   id: string;
   name: string;
+  surname: string;
+  age: number;
+  children: number;
+  married: boolean;
+  alive: boolean;
 };
 
 export const documentData: DocumentType = {
@@ -37,10 +42,20 @@ export const documentData: DocumentType = {
       "id-1": {
         id: "id-1",
         name: "Alex",
+        surname: "Smith",
+        age: 30,
+        children: 0,
+        married: true,
+        alive: true,
       },
       "id-2": {
         id: "id-2",
         name: "John",
+        surname: "Smith",
+        age: 40,
+        children: 0,
+        married: false,
+        alive: true,
       },
     },
   },

tests/utilities.test.ts

@@ -0,0 +1,49 @@
+import { EntityState } from "@onsetsoftware/entity-state";
+import { beforeEach, describe, expect, test } from "vitest";
+import { Person, documentData } from "./data";
+
+import { quickClone } from "../src/helpers/quick-clone";
+import { getEntitiesValue } from "../src/actions/utilities";
+
+describe("Get entities value", () => {
+  let people: EntityState<Person>;
+
+  beforeEach(() => {
+    people = quickClone(documentData.people);
+  });
+
+  const testCases = [
+    {
+      path: "name",
+      expected: "",
+    },
+    {
+      path: "surname",
+      expected: "Smith",
+    },
+    {
+      path: "age",
+      expected: "",
+    },
+    {
+      path: "children",
+      expected: 0,
+    },
+    {
+      path: "married",
+      expected: "",
+    },
+    {
+      path: "alive",
+      expected: true,
+    },
+  ] as const;
+
+  testCases.forEach((testCase) => {
+    test("it should return the correct " + testCase.path, () => {
+      expect(getEntitiesValue(people, ["id-1", "id-2"], testCase.path)).toBe(
+        testCase.expected,
+      );
+    });
+  });
+});