test: raise coverage to 100%

Author: remy

lib/index.js

@@ -31,7 +31,7 @@ module.exports = (config, root) => {
   }).then(res => {
     require('./update')({ root: root });
     /* istanbul ignore if */
-    if (!settings.return && res !== undefined) {
+    if (!settings.return && res != undefined) { // jshint ignore:line
       return console.log(res);
     }
 

test/integration/help.test.js

@@ -90,4 +90,4 @@ test('user can trigger usage error', function (t) {
     t.equal(e.code, 'BAD_ARGS', 'bad arguments');
     t.match(e.message, /This did not work/i, 'user error shown');
   });
-});
+});

test/integration/silent-on-empty.test.js

@@ -0,0 +1,22 @@
+'use strict';
+
+var test = require('tap-only');
+var sampleArgs = ['node', 'script.js'];
+var fixtures = __dirname + '/../fixtures';
+var clite = require('../../');
+
+test('should not echo on empty resolve', function (t) {
+  process.argv = sampleArgs.slice();
+  var logged = false;
+  console.log = function (s) {
+    logged = true;
+    t.fail('logged: ' + s);
+  };
+
+  return clite({
+    commands: { _ : 'echo' },
+    return: false,
+  }, fixtures + '/basic-clite').then(function () {
+    t.notOk(logged, 'kept silent as expected');
+  });
+});

test/integration/stdin.test.js

@@ -1,16 +1,16 @@
 'use strict';
 
-var test = require('tap').test;
+var test = require('tap-only');
 var sampleArgs = ['node', 'script.js'];
 var fixtures = __dirname + '/../fixtures';
 var fs = require('fs');
 var body = fs.readFileSync(__filename, 'utf8');
-var stdin = require('mock-stdin').stdin();
 var clite = require('../../');
 
 test('reads from stdin', function (t) {
   process.argv = sampleArgs.concat('echo');
   delete process.env.TAP; // remove test check in lib/read-stdin
+  var stdin = require('mock-stdin').stdin();
 
   var p = clite({
     commands: {
@@ -22,6 +22,27 @@ test('reads from stdin', function (t) {
     process.env.TAP = '1'; // restore tap
   });
 
+  // send our mocked out stdin
+  stdin.send(body, 'ascii');
+  stdin.send(null);
+  return p;
+});
+
+test('no command but has body', function (t) {
+  process.argv = sampleArgs.slice();
+  delete process.env.TAP; // remove test check in lib/read-stdin
+  var stdin = require('mock-stdin').stdin();
+
+  var p = clite({
+    commands: {
+      echo: './echo'
+    },
+    return: true,
+  }, fixtures + '/basic-clite').catch(function (e) {
+    t.equals(e.code, 'NO_COMMAND', 'no command given');
+    process.env.TAP = '1'; // restore tap
+  });
+
   // send our mocked out stdin
   stdin.send(body, 'ascii');
   stdin.send(null);