Merge pull request #57 from wgoto/did-update

Create modelDidUpdate method

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-axiom",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "React Axiom is a way to use models with React.",
   "repository": "https://github.com/wgoto/react-axiom",
   "main": "lib/index.js",

src/models/Model.js

@@ -67,6 +67,7 @@ class Model extends Publisher {
 
   setState(nextState) {
     const { prev, next, diff } = this._diffState(nextState);
+    const prevState = { ...this.state };
 
     if (diff) {
       // FOR LOGGING
@@ -76,10 +77,16 @@ class Model extends Publisher {
       // console.groupEnd();
 
       Object.assign(this.state, nextState);
+      this.modelDidUpdate(prevState);
       this.publish();
     }
   }
 
+  modelDidUpdate() {
+    // Template method for invoking behavior after
+    // a model has correctly updated state.
+  }
+
 
   //==================
   // INTERNAL METHODS

tests/models/Model.test.js

@@ -5,7 +5,8 @@ import Model from '../../src/models/Model';
 // TEST CONSTANTS
 //================
 
-const obj = { id: '1' };
+const id = '1';
+const obj = { id };
 const arr = [];
 
 
@@ -41,7 +42,7 @@ describe('Model', () => {
   let modelWithDefaultState;
 
   beforeEach(() => {
-    model = new Test({ id: '1', obj, arr });
+    model = new Test({ id, obj, arr });
     modelWithoutState = new TestWithoutState();
     modelWithDefaultState = new TestWithDefaultState();
   });
@@ -120,6 +121,7 @@ describe('Model', () => {
 
   describe('setState', () => {
     beforeEach(() => {
+      spyOn(model, 'modelDidUpdate');
       spyOn(model, 'publish');
     });
 
@@ -132,6 +134,10 @@ describe('Model', () => {
         expect(model.state.id).toBe('2');
       });
 
+      it('should call modelDidUpdate with the previous state', () => {
+        expect(model.modelDidUpdate).toBeCalledWith({ id, test: true, obj, arr });
+      });
+
       it('should call publish', () => {
         expect(model.publish).toBeCalled();
       });
@@ -147,6 +153,10 @@ describe('Model', () => {
         expect(model.state.obj.id).toBe('2');
       });
 
+      it('should not call modelDidUpdate', () => {
+        expect(model.modelDidUpdate).not.toBeCalled();
+      });
+
       it('should not call publish', () => {
         expect(model.publish).not.toBeCalled();
       });