Fixed the bug of reusing an old future.

Author: mitar

server.js

@@ -8,12 +8,12 @@ blocking = function (obj, fun) {
     fun = obj;
     obj = undefined;
   }
-  var future = new Future();
   var f = function () {
     if (_.isUndefined(obj)) {
       obj = this;
     }
     var args = _.toArray(arguments);
+    var future = new Future();
     fun.apply(obj, args.concat(future.resolver()));
     return future.wait();
   };

tests.js

@@ -9,10 +9,17 @@ Tinytest.add('blocking', function (test) {
   test.isTrue(isDefined, "blocking is not defined");
   test.isTrue(Package['peerlibrary:blocking'].blocking, "Package.peerlibrary:blocking.blocking is not defined");
 
-  test.equal(blocking(function (a, b, cb) { cb(null, a + b) })(1, 2), 3);
+  var blockingFunction = blocking(function (a, b, cb) { cb(null, a + b) });
 
+  // We run it twice to catch possible bugs of reusing old futures.
+  test.equal(blockingFunction(1, 2), 3);
+  test.equal(blockingFunction(1, 2), 3);
+
+  // We want to support the last argument of a blocking function being a function
+  // (and not switch to a non-blocking mode like Meteor.wrapAsync does).
   test.equal(blocking(function (a, b, f, cb) { cb(null, f(a, b)) })(1, 2, function (a, b) {return a + b}), 3);
 
+  // We want to work also with varargs functions .
   test.equal(blocking(function () {
     var a = arguments[0];
     var b = arguments[1];