test history

Author: jcouyang

lib/__tests__/react-most-test.jsx

@@ -10,6 +10,8 @@ const CounterView = props=> (
     <span className="count">{props.count}</span>
     <span className="wrapperProps">{props.wrapperProps}</span>
     <span className="overwritedProps">{props.overwritedProps}</span>
+    <span className="backward" onClick={props.history.backward}>-</span>
+    <span className="forward" onClick={props.history.forward}>+</span>
   </div>
 )
 
@@ -50,11 +52,12 @@ describe('react-most', () => {
       let counter = TestUtils.findRenderedComponentWithType(counterWrapper, Counter)
 
       do$([()=>counter.actions.inc(),
-           ()=>counter.actions.inc()])
+           ()=>counter.actions.fromPromise(Promise.resolve({type:'inc'})),
+           ()=>counter.actions.fromEvent({type:'inc'})])
 
       return historyStreamOf(counter)
-        .take$(2)
-        .then(state=>expect(state.count).toEqual(2))
+        .take$(3)
+        .then(state=>expect(state.count).toEqual(3))
     })
 
     it('sink can also generate intent', ()=> {
@@ -134,4 +137,41 @@ describe('react-most', () => {
       })
     })
   });
+
+  describe('history', ()=>{
+    it('can undo', ()=> {
+      let counterWrapper = TestUtils.renderIntoDocument(
+        <Most >
+          <Counter history={true} />
+        </Most>
+      )
+      let counter = TestUtils.findRenderedComponentWithType(counterWrapper, Counter)
+      return do$([
+        ()=>counter.actions.inc(),
+        ()=>counter.actions.inc(),
+        ()=>counter.actions.inc()
+      ]).then(()=>{
+        let backward = TestUtils.findRenderedDOMComponentWithClass(counterWrapper, 'backward')
+        TestUtils.Simulate.click(backward)
+        TestUtils.Simulate.click(backward)
+        TestUtils.Simulate.click(backward)
+      }).then(()=>{
+        let count = TestUtils.findRenderedDOMComponentWithClass(counterWrapper, 'count')
+        expect(count.textContent).toBe('1')
+      }).then(()=>{
+        let forward = TestUtils.findRenderedDOMComponentWithClass(counterWrapper, 'forward')
+        TestUtils.Simulate.click(forward)
+      }).then(()=>{
+        let count = TestUtils.findRenderedDOMComponentWithClass(counterWrapper, 'count')
+        expect(count.textContent).toBe('2')
+      }).then(()=>{
+        let forward = TestUtils.findRenderedDOMComponentWithClass(counterWrapper, 'forward')
+        TestUtils.Simulate.click(forward)
+        TestUtils.Simulate.click(forward)
+      }).then(()=>{
+        let count = TestUtils.findRenderedDOMComponentWithClass(counterWrapper, 'count')
+        expect(count.textContent).toBe('3')
+      })
+    })
+  })
 })

lib/react-most.js

@@ -46,7 +46,7 @@ export function connect(main, initprops={}) {
           }
           else if(sinks[name] instanceof Function){
             _actions[name] = (...args)=>{
-              return this.context[intentStream].send(sinks[name].apply(null, args));
+              return this.context[intentStream].send(sinks[name].apply(this, args));
             }
           }
         }
@@ -60,7 +60,7 @@ export function connect(main, initprops={}) {
       }
       componentDidMount(){
         this.context[flatObserve](this.actionsSinks, (action)=>{
-          if(action instanceof Function)
+          if(action instanceof Function) {
             this.setState((prevState, props)=>{
               let newState = action.call(this, prevState,props);
               if(initprops.history && newState != prevState){
@@ -69,8 +69,10 @@ export function connect(main, initprops={}) {
               }
               return newState;
             });
-          else
+          } else {
+            /* istanbul ignore next */
             console.warn('action', action,'need to be a Function which map from current state to new state');
+          }
         });
       }
       render() {
@@ -88,6 +90,7 @@ let Most = React.createClass({
     let engineClass = this.props && this.props.engine || mostEngine
     let engine = engineClass();
     // TODO: add support for ReactiveX
+    /* istanbul ignore next */
     if(process.env.NODE_ENV==='debug') {
       if(engineClass == mostEngine) {
         engine.intentStream.timestamp()