test: 100% test coverage 🎉

Make CLI, which is mainly for testing, use the no-argument form of
Logger() when possible in order to achieve 100% branch coverage in
Logger() function.

Remove the un-used support for boolean flags that default to true (would
emit a --no-OPT entry in the usage message), bringing the cli itself up
to 100% coverage.

This package now has 100% line and branch coverage.

Signed-off-by: Ryan Graham <[email protected]>

Author: rmg

lib/cli.js

@@ -21,6 +21,8 @@ function cli(args) {
     version($0, p);
   } else if (opts.h || opts.help) {
     usage($0, p);
+  } else if (args.length < 3) {
+    process.stdin.pipe(Logger()).pipe(process.stdout);
   } else {
     process.stdin.pipe(Logger(opts)).pipe(process.stdout);
   }
@@ -48,9 +50,8 @@ function usage($0, p) {
   p('');
 
   function boolOpt(name, def) {
-    name = def ? 'no-' + name : name;
     name = name + PADDING.slice(0, 20-name.length);
-    p('   --%s default: %s', name, def ? 'on' : 'off');
+    p('   --%s default: %s', name, def);
   }
 
   function stdOpt(name, def) {

package.json

@@ -24,7 +24,7 @@
   },
   "main": "index.js",
   "scripts": {
-    "test": "tap --coverage --coverage-report=cobertura test/test-*"
+    "test": "tap --100 test/test-*"
   },
   "dependencies": {
     "byline": "^5.0.0",