Merge c771b7b into 36e0b2e

lib/helpers/normalizeArgs.js

@@ -16,7 +16,10 @@ function normalizeArgs(registry, args) {
     return fn;
   }
 
-  return map(flatten(args), getFunction);
+  var flattenArgs = flatten(args);
+  assert(flattenArgs.length, 'Arguments must be a non-empty array or string');
+
+  return map(flattenArgs, getFunction);
 }
 
 module.exports = normalizeArgs;

test/series.js

@@ -34,6 +34,42 @@ describe('series', function() {
     done();
   });
 
+  it('should throw on empty object on registered tasks array', function(done) {
+    function fail() {
+      taker.series(['test1', 'test2', 'test3', {}]);
+    }
+
+    expect(fail).toThrow(/Task never defined:/);
+    done();
+  });
+
+  it('should throw on empty object of registered task', function(done) {
+    function fail() {
+      taker.series({});
+    }
+
+    expect(fail).toThrow(/Task never defined:/);
+    done();
+  });
+
+  it('should throw on non registered tasks', function(done) {
+    function fail() {
+      taker.series();
+    }
+
+    expect(fail).toThrow(/Arguments must be a non-empty array or string/);
+    done();
+  });
+
+  it('should throw on empty array of registered tasks', function(done) {
+    function fail() {
+      taker.series([]);
+    }
+
+    expect(fail).toThrow(/Arguments must be a non-empty array or string/);
+    done();
+  });
+
   it('should take only one array of registered tasks', function(done) {
     taker.series(['test1', 'test2', 'test3'])(function(err, results) {
       expect(results).toEqual([1, 2, 3]);