Fix should assertion

Author: dmitry-zaets

src/assertions/toDispatchActionsWithState.js

@@ -53,6 +53,7 @@ function toDispatchActionsWithState(initialState, actionUnderTest, expectedActio
         if (!find(dispatchedActions, expectedUnrolledActions[index])) {
           throw new Error(
             `Expected action ${JSON.stringify(expectedUnrolledActions[index])} was not dispatched.`
+            + `\nActual dispatched actions: ${JSON.stringify(dispatchedActions)}`
           );
         }
       }

src/chai/index.js

@@ -18,10 +18,9 @@ function registerAssertions() {
 
       const state = utils.flag(this, 'state');
       if (state) {
-        assertions.toDispatchActionsWithState(state, this._obj, expectedActions, done);
-      } else {
-        assertions.toDispatchActions(this._obj, expectedActions, done);
+        return assertions.toDispatchActionsWithState(state, this._obj, expectedActions, done);
       }
+      return assertions.toDispatchActions(this._obj, expectedActions, done);
     }
 
     function isDispatchingWithState(actualAction, expectedActions, state, done) {

src/should/index.js

@@ -15,8 +15,9 @@ function dispatchActions(expectedActions, done) {
     const action = this.obj.action;
     const state = this.obj.state;
     assertions.toDispatchActionsWithState(state, action, expectedActions, done);
+  } else {
+    assertions.toDispatchActions(this.obj, expectedActions, done);
   }
-  assertions.toDispatchActions(this.obj, expectedActions, done);
 }
 
 function registerAssertions() {

test/should/index.js

@@ -29,7 +29,7 @@ describe('should', () => {
 
     it('should have alias .state', (done) => {
       should(actions.actionCreatorWithGetState())
-        .with.state(() => { return { property: 'value1' };})
+        .with.state(() => { return { property: 'value' };})
         .dispatchActions(actions.actionWithGetState({ property: 'value' }), done);
     });
   });