Add --not alias for --invert-match

This is easier to remember and works well grammatically with the command
name.

Signed-off-by: Kevin Locke <[email protected]>

Author: kevinoid

bin/git-branch-is.js

@@ -65,6 +65,9 @@ function gitBranchIsCmd(args, callback) {
     .option('--git-path <path>', 'set the path to the git binary')
     .option('-i, --ignore-case', 'compare/match branch name case-insensitively')
     .option('-I, --invert-match', 'inverts/negates comparison')
+    // Note:  Commander.js only supports one long option per option call
+    // https://github.com/tj/commander.js/issues/430
+    .option('--not', 'inverts/negates comparison (same as --invert-match)')
     .option('-q, --quiet', 'suppress warning message if branch differs')
     .option('-r, --regex', 'match <branch name> as a regular expression')
     .option('-v, --verbose', 'print a message if the branch matches')
@@ -83,6 +86,9 @@ function gitBranchIsCmd(args, callback) {
   // pluralize --git-arg to cover multiple uses
   command.gitArgs = command.gitArg;
 
+  // treat --not as an alias for --invert-match
+  command.invertMatch = command.invertMatch || command.not;
+
   const expectedBranch = command.args[0];
 
   let expectedBranchRegExp;

test/git-branch-is-cmd.js

@@ -340,7 +340,7 @@ describe('git-branch-is', () => {
   });
 
   // Unlike an commands with expression arguments (e.g. find, test), follow
-  // the typical argument convention that repeated flag arguments are ignored.
+  // the convention that repeated flag arguments are ignored.
   it('does not double-invert', (done) => {
     const args = ARGS.concat('-I', '-I', 'invalid');
     gitBranchIsCmd(args, (err, result) => {
@@ -352,6 +352,29 @@ describe('git-branch-is', () => {
     });
   });
 
+  it('support --not as alias for -I', (done) => {
+    const args = ARGS.concat('--not', 'invalid');
+    gitBranchIsCmd(args, (err, result) => {
+      assert.ifError(err);
+      assert.strictEqual(result.code, 0);
+      assert(!result.stdout);
+      assert(!result.stderr);
+      done();
+    });
+  });
+
+  // Careful that alias isn't handled differently
+  it('does not double-invert with alias', (done) => {
+    const args = ARGS.concat('-I', '--not', 'invalid');
+    gitBranchIsCmd(args, (err, result) => {
+      assert.ifError(err);
+      assert.strictEqual(result.code, 0);
+      assert(!result.stdout);
+      assert(!result.stderr);
+      done();
+    });
+  });
+
   it('returns a Promise with the result', () => {
     const promise = gitBranchIsCmd(ARGS.concat(BRANCH_CURRENT));
     assert(promise instanceof global.Promise);