improve unit test

mobxjs · Sep 24, 2018 · 5ed3786 · 5ed3786
1 parent 0a133bf
commit 5ed3786
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/test/base/action.js b/test/base/action.js
@@ -182,6 +182,10 @@ test("should be possible to change observed state in an action called from compu
     const testAction = mobx.action(() => {
         mobx._allowStateChangesInsideComputed(() => {
             a.set(3)
+            // a second level computed should throw
+            expect(() => c2.get()).toThrowError(
+                /Computed values are not allowed to cause side effects by changing observables that are already being observed/
+            )
         })
         expect(a.get()).toBe(3)
         expect(() => {
@@ -196,6 +200,11 @@ test("should be possible to change observed state in an action called from compu
         return a.get()
     })
 
+    const c2 = mobx.computed(() => {
+        a.set(6)
+        return a.get()
+    })
+
     c.get()
 
     mobx._resetGlobalState()