Fix: Avoid setting displayName on series/parallel functions (ref gulpjs/gulp#2270) (#91)

Author: coreyfarrell

lib/parallel.js

@@ -13,14 +13,13 @@ function parallel() {
   var args = normalizeArgs(this._registry, arguments);
   var extensions = createExtensions(this);
   var fn = create(args, extensions);
-
-  fn.displayName = '<parallel>';
+  var name = '<parallel>';
 
   metadata.set(fn, {
-    name: fn.displayName,
+    name: name,
     branch: true,
     tree: {
-      label: fn.displayName,
+      label: name,
       type: 'function',
       branch: true,
       nodes: buildTree(args),

lib/series.js

@@ -13,14 +13,13 @@ function series() {
   var args = normalizeArgs(this._registry, arguments);
   var extensions = createExtensions(this);
   var fn = create(args, extensions);
-
-  fn.displayName = '<series>';
+  var name = '<series>';
 
   metadata.set(fn, {
-    name: fn.displayName,
+    name: name,
     branch: true,
     tree: {
-      label: fn.displayName,
+      label: name,
       type: 'function',
       branch: true,
       nodes: buildTree(args),

test/parallel.js

@@ -146,4 +146,10 @@ describe('parallel', function() {
       done();
     });
   });
+
+  it('should not register a displayName on the returned function by default', function(done) {
+    var task = taker.parallel(fn1);
+    expect(task.displayName).toEqual(undefined);
+    done();
+  });
 });

test/series.js

@@ -147,4 +147,9 @@ describe('series', function() {
     });
   });
 
+  it('should not register a displayName on the returned function by default', function(done) {
+    var task = taker.series(fn1);
+    expect(task.displayName).toEqual(undefined);
+    done();
+  });
 });