fredrick - npm Package Compare versions

Comparing version 1.1.0 to 1.1.1

index.js

@@ -43,3 +43,6 @@ 'use strict';
 
-    if (!plugin) return;
+    if (!plugin) {
+      this.error('Invalid command');
+      return this.exit(1);
+    }
 
@@ -46,0 +49,0 @@ var options  = minimist(args.slice(1));

package.json

 {
   "name": "fredrick",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "simple module for writing your spiffy command line tools",
@@ -5,0 +5,0 @@ "main": "index.js",

test/fredrick_test.js

@@ -325,2 +325,22 @@ var test     = require('tape');
 
+test('Fredrick handles invalid commands', function(t) {
+
+  var fakeStderr = { write: sinon.spy() };
+  var fakeExit   = sinon.spy();
+
+  var fredrick = new Fredrick('fredrick', {
+    stderr: fakeStderr, exit: fakeExit
+  });
+
+  var args = ['testing'];
+
+  fredrick.respond(args);
+
+  t.plan(2);
+  t.ok(fakeStderr.write.calledWith('Invalid command\n'), 'writes error');
+
+  t.ok(fakeExit.calledWith(1), 'exits cleanly');
+
+});
+
 test('Fredrick responds to usage command', function(t) {
@@ -327,0 +347,0 @@

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

16294

increased by3.06%

Lines of code

531

increased by2.91%

No dependency changes