Merge pull request #1514 from quanganhtran/observe-set

Update object api set to write through parent mobx administrator

src/api/object-api.ts

@@ -65,7 +65,7 @@ export function set(obj: any, key: any, value?: any): void {
         const adm = ((obj as any) as IIsObservableObject).$mobx
         const existingObservable = adm.values[key]
         if (existingObservable) {
-            existingObservable.set(value)
+            adm.write(obj, key, value)
         } else {
             defineObservableProperty(obj, key, value, adm.defaultEnhancer)
         }

test/base/object-api.js

@@ -177,6 +177,25 @@ test("observe & intercept", () => {
     })
 })
 
+test("observe & intercept set called multiple times", () => {
+    const a = mobx.observable({})
+    const interceptLogs = []
+    const observeLogs = []
+
+    mobx.intercept(a, change => {
+        interceptLogs.push(`${change.name}: ${change.newValue}`)
+        return change
+    })
+    mobx.observe(a, change => observeLogs.push(`${change.name}: ${change.newValue}`))
+
+    mobx.set(a, "x", 0)
+    a.x = 1
+    mobx.set(a, "x", 2)
+
+    expect(interceptLogs).toEqual(["x: 0", "x: 1", "x: 2"])
+    expect(observeLogs).toEqual(["x: 0", "x: 1", "x: 2"])
+})
+
 test("dynamically adding properties should preserve the original modifiers of an object", () => {
     const todos = observable(
         {