Fix empty files leading to no realSrc and error in output.js

arrowrowe · Oct 17, 2015 · 970d7de · 970d7de
1 parent 6a500e8
commit 970d7de
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/lib/file/output.js b/lib/file/output.js
@@ -4,6 +4,10 @@ var path = require('path');
 // It does not return. The argument `commands` will change.
 module.exports = function (assets, pkg, commands) {
 
+  if (commands.length === 0) {
+    return;
+  }
+
   var srcLength = pkg.realSrc.length;
   var pkgDist = pkg.dist || pkg.name;
   var dist = path.resolve(assets.dist, pkgDist);

diff --git a/test/lib/file/output-spec.js b/test/lib/file/output-spec.js
@@ -1,5 +1,6 @@
 var utilOutput = require('../../../lib/file/output');
 var expect = require('chai').expect;
+var sinon = require('sinon');
 
 describe('lib/file/output', function () {
 
@@ -77,4 +78,18 @@ describe('lib/file/output', function () {
     }).to.throw('Unrecognized behavior [whatever]!');
   });
 
+  it('does nothing if no commands provided', function () {
+    var spy = sinon.spy();
+    utilOutput({
+      dist: '/fake'
+    }, {
+      name: 'fake',
+      realSrc: ''
+    }, {
+      length: 0,
+      forEach: spy
+    });
+    expect(spy.callCount).to.equal(0);
+  });
+
 });