checking for watch parameters, fixes #1002

gulpjs · Jun 1, 2015 · 9ca708b · 9ca708b
1 parent 2259369
commit 9ca708b
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 0 deletions.
diff --git a/index.js b/index.js
@@ -13,6 +13,12 @@ Gulp.prototype.src = vfs.src;
 Gulp.prototype.dest = vfs.dest;
 Gulp.prototype.symlink = vfs.symlink;
 Gulp.prototype.watch = function(glob, opt, task) {
+  if (typeof opt === 'string' || typeof task === 'string' ||
+    Array.isArray(opt) || Array.isArray(task)) {
+    throw new Error('watching ' + glob + ': watch task has to be ' +
+      'gulp.series, gulp.parallel or a function');
+  }
+
   if (typeof opt === 'function') {
     task = opt;
     opt = null;

diff --git a/test/watch.js b/test/watch.js
@@ -126,5 +126,29 @@ describe('gulp', function() {
       updateTempFile(tempFile);
     });
 
+    it('should throw an error: passed parameter (string) is not a function', function(done) {
+      var tempFile = path.join(outpath, 'empty.txt');
+
+      createTempFile(tempFile);
+      try {
+        gulp.watch(tempFile, 'task1');
+      } catch (err) {
+        err.message.should.equal('watching ' + tempFile +  ': watch task has to be gulp.series, gulp.parallel or a function');
+        done();
+      }
+    });
+
+    it('should throw an error: passed parameter (array) is not a function', function(done) {
+      var tempFile = path.join(outpath, 'empty.txt');
+
+      createTempFile(tempFile);
+      try {
+        gulp.watch(tempFile, ['task1']);
+      } catch (err) {
+        err.message.should.equal('watching ' + tempFile +  ': watch task has to be gulp.series, gulp.parallel or a function');
+        done();
+      }
+    });
+
   });
 });