fix(store): store factory function id should be always a string value

Author: smalluban

src/store.js

@@ -1519,7 +1519,10 @@ function store(Model, options = {}) {
   }
 
   const resolveId = options.id
-    ? options.id
+    ? (host) => {
+        const id = options.id(host);
+        return id !== undefined && id !== null ? String(id) : undefined;
+      }
     : (host, value) => {
         if (value !== null && value !== undefined) {
           return typeof value === "object" ? value.id : String(value);

test/spec/store.js

@@ -1215,6 +1215,9 @@ describe("store:", () => {
           byundefined: store(Model, { id: () => undefined }),
           byprop: store(Model, { id: "modelId" }),
           byfn: store(Model, { id: ({ modelId }) => modelId }),
+          byfnwithnumber: store(Model, {
+            id: ({ modelId }) => Number(modelId),
+          }),
           withoutid: store(Model),
           draft: store(Model, { draft: true, id: "modelId" }),
           draftwithoutid: store(Model, { draft: true }),
@@ -1256,6 +1259,11 @@ describe("store:", () => {
           });
         });
 
+        fit("gets a model instance when id is a number zero", () => {
+          el.modelId = "0";
+          expect(el.byfnwithnumber).toBe(store.get(Model, "0"));
+        });
+
         describe("in draft mode", () => {
           it("throws when try to set value by assertion", () => {
             expect(() => {