fix(cli): error handling for ENOENT is fixed

It should warn the correct unfound file names. Also it should continue if one file cannot be found. Tests are added for these

cli.js

@@ -38,9 +38,13 @@ forEach(cli.input, function(arg) {
 var length = filePaths.length;
 
 function processFile(fileIndex) {
-  fs.createReadStream(filePaths[fileIndex])
+  var filePath = filePaths[fileIndex];
+  fs.createReadStream(filePath)
     .on('error', function(err) {
-      console.log('Failed to read file ' + filePaths[0] + '\n' + err);
+      console.warn('Failed to read file ' + filePath + '\n' + err);
+      if(++fileIndex < length) {
+        processFile(fileIndex);
+      }
     })
     .pipe(split(separator))
     .pipe(conventionalCommitsParser(cli.flags))

test/cli.spec.js

@@ -5,16 +5,17 @@ var spawn = require('child_process').spawn;
 var concat = require('concat-stream');
 
 var cliPath = './cli.js';
+var output1 = readFileSync('test/expected/output1.txt', 'utf-8');
+var output2 = readFileSync('test/expected/output2.txt', 'utf-8');
 
 describe('cli', function() {
   it('should parse commits in a file', function(done) {
     var cp = spawn(cliPath, ['test/fixtures/log.txt']);
 
     cp.stdout
       .pipe(concat(function(chunk) {
-        var expected = readFileSync('test/expected/output1.txt', 'utf-8');
 
-        expect(chunk.toString()).to.equal(expected);
+        expect(chunk.toString()).to.equal(output1);
         done();
       }));
   });
@@ -24,9 +25,8 @@ describe('cli', function() {
 
     cp.stdout
       .pipe(concat(function(chunk) {
-        var expected = readFileSync('test/expected/output2.txt', 'utf-8');
 
-        expect(chunk.toString()).to.equal(expected);
+        expect(chunk.toString()).to.equal(output2);
         done();
       }));
   });
@@ -36,10 +36,19 @@ describe('cli', function() {
 
     cp.stdout
       .pipe(concat(function(chunk) {
-        var expected = readFileSync('test/expected/output_both.txt', 'utf-8');
+        var expected = output1 + output2;
 
         expect(chunk.toString()).to.equal(expected);
         done();
       }));
   });
+
+  it('should error if files cannot be found', function(done) {
+    var cp = spawn(cliPath, ['test/fixtures/log.txt', 'test/fixtures/log3.txt', 'test/fixtures/log2.txt', 'test/fixtures/log4.txt', '===']);
+    cp.stderr
+      .pipe(concat(function(chunk) {
+        expect(chunk.toString()).to.equal('Failed to read file test/fixtures/log3.txt\nError: ENOENT, open \'test/fixtures/log3.txt\'\nFailed to read file test/fixtures/log4.txt\nError: ENOENT, open \'test/fixtures/log4.txt\'\n');
+        done();
+      }));
+  });
 });