Fix bug with defer where it is not variadic

goatslacker · Feb 12, 2015 · eb4e3a0 · eb4e3a0
1 parent 48efd69
commit eb4e3a0
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 7 deletions.
diff --git a/dist/alt-with-runtime.js b/dist/alt-with-runtime.js
@@ -318,9 +318,13 @@ var Alt = (function () {
 
           // Set all the properties on action
           obj[action] = newAction[ACTION_HANDLER];
-          obj[action].defer = function (x) {
-            return setTimeout(function () {
-              return newAction[ACTION_HANDLER](x);
+          obj[action].defer = function () {
+            for (var _len = arguments.length, args = Array(_len), _key = 0; _key < _len; _key++) {
+              args[_key] = arguments[_key];
+            }
+
+            setTimeout(function () {
+              return newAction[ACTION_HANDLER].apply(null, args);
             });
           };
           obj[action][ACTION_KEY] = actionName;

diff --git a/dist/alt.js b/dist/alt.js
@@ -322,9 +322,13 @@ var Alt = (function () {
 
           // Set all the properties on action
           obj[action] = newAction[ACTION_HANDLER];
-          obj[action].defer = function (x) {
-            return setTimeout(function () {
-              return newAction[ACTION_HANDLER](x);
+          obj[action].defer = function () {
+            for (var _len = arguments.length, args = Array(_len), _key = 0; _key < _len; _key++) {
+              args[_key] = arguments[_key];
+            }
+
+            setTimeout(function () {
+              return newAction[ACTION_HANDLER].apply(null, args);
             });
           };
           obj[action][ACTION_KEY] = actionName;

diff --git a/src/alt.js b/src/alt.js
@@ -285,7 +285,9 @@ your own custom identifier for each store`
 
       // Set all the properties on action
       obj[action] = newAction[ACTION_HANDLER]
-      obj[action].defer = (x) => setTimeout(() => newAction[ACTION_HANDLER](x))
+      obj[action].defer = (...args) => {
+        setTimeout(() => newAction[ACTION_HANDLER].apply(null, args))
+      }
       obj[action][ACTION_KEY] = actionName
       obj[constant] = actionName
 

diff --git a/test/index.js b/test/index.js
@@ -386,6 +386,12 @@ module.exports = {
     assert.equal(Array.isArray(secondStore.getState().foo), true, 'shorthand for multiple elements pass through goes as array')
     assert.equal(secondStore.getState().foo[0], 1, 'shorthand for multiple elements pass through goes as array')
     assert.equal(secondStore.getState().foo[1], 0, 'shorthand for multiple elements pass through goes as array')
+
+    myActions.shortHandBinary.defer(2, 1)
+    setTimeout(() => {
+      assert.equal(secondStore.getState().foo[0], 2, 'shorthand for defer multiple elements pass through goes as array')
+      assert.equal(secondStore.getState().foo[1], 1, 'shorthand for defer multiple elements pass through goes as array')
+    })
   },
 
   'access of stores'() {