Fixes

Author: smalluban

docs/migration.md

@@ -25,6 +25,8 @@ customName: {
 }
 ```
 
+Read more about the full object property descriptor in the [Structure](/component-model/structure.md#value) section.
+
 ### Attributes
 
 Writable properties are no longer automatically synchronized back to the attribute. You must set the `reflect` option to enable the synchronization:
@@ -45,6 +47,8 @@ Writable properties are no longer automatically synchronized back to the attribu
 }
 ```
 
+Read more about the attribute synchronization in the [Structure](/component-model/structure.md#reflect) section.
+
 ### Render and Content
 
 #### Names

test/spec/store.js

@@ -63,6 +63,17 @@ describe("store:", () => {
       expect(() => store.get({ value: null })).toThrow();
     });
 
+    it("throws an error when get method returns string", () => {
+      Model = {
+        value: "test",
+        [store.connect]: {
+          get: () => "test",
+        },
+      };
+
+      expect(() => store.get(Model)).toThrow();
+    });
+
     it("throws when nested object is used as a primary model", () => {
       store.get(Model, "1");
       expect(() => {
@@ -386,6 +397,20 @@ describe("store:", () => {
       ).toThrow();
     });
 
+    it("throws an error when set method returns undefined", () => {
+      Model = {
+        value: "test",
+        [store.connect]: {
+          get: () => ({}),
+          set: () => {
+            return "test";
+          },
+        },
+      };
+
+      expect(() => store.set(Model)).toThrow();
+    });
+
     it("rejects an error when array with external objects is set with wrong type", async () => {
       const model = await promise;
       expect(() =>
@@ -420,6 +445,22 @@ describe("store:", () => {
       });
     });
 
+    it("rejects an error when set method returns promise resolving to string", () => {
+      Model = {
+        value: "test",
+        [store.connect]: {
+          get: () => ({}),
+          set: async () => {
+            return "test";
+          },
+        },
+      };
+
+      return store.set(Model).catch((e) => {
+        expect(e).toBeInstanceOf(Error);
+      });
+    });
+
     it("returns a placeholder in error state for not found singleton model", () => {
       Model = {
         value: "test",