Merge pull request #3 from markuplab/fix-sync-outputs

Fix sync action outputs to async actions

src/appstate.js

@@ -94,9 +94,9 @@ function runBranch (index, options) {
   }
 
   if (Array.isArray(currentBranch)) {
-    runAsyncBranch(index, currentBranch, options);
+    return runAsyncBranch(index, currentBranch, options);
   } else {
-    runSyncBranch(index, currentBranch, options);
+    return runSyncBranch(index, currentBranch, options);
   }
 }
 
@@ -206,7 +206,7 @@ function runSyncBranch (index, currentBranch, options) {
       });
 
       if (runResult && runResult.then) {
-        return result.then(() => {
+        return runResult.then(() => {
           return runBranch(index + 1, options);
         });
       }

test/suites/appstate.js

@@ -510,4 +510,38 @@ lab.experiment('#appstate', function () {
         done();
       });
   });
+
+  lab.test('should correct run tree with sync action that output to async', (done) => {
+    var counter = 0;
+
+    function async (args, state, output) {
+      setTimeout(() => {
+        counter += 1;
+        assert.equal(counter, 1);
+        output.success();
+      }, 0);
+    }
+
+    function sync (args, state, output) {
+      output.success();
+    }
+
+    var signal = appstate.create('test', [
+      sync, {
+        success: [
+          [
+            async, {
+              success: [noop]
+            }
+          ]
+        ]
+      }
+    ]);
+
+    signal(tree).then(function () {
+      counter += 1;
+      assert.equal(counter, 2);
+      done();
+    }).catch(done);
+  });
 });