Merge pull request #1007 from ddprrt/4.0

checking for watch parameters, fixes #1002

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;

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();
+      }
+    });
+
   });
 });