return '' if flag is string and empty

minimistjs · Feb 21, 2014 · fa63ed4 · fa63ed4
1 parent cd46bf9
commit fa63ed4
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -74,7 +74,7 @@ module.exports = function (args, opts) {
                 i++;
             }
             else {
-                setArg(key, true);
+                setArg(key, flags.strings[key] ? '' : true);
             }
         }
         else if (/^-[^-]+/.test(arg)) {
@@ -119,7 +119,7 @@ module.exports = function (args, opts) {
                     i++;
                 }
                 else {
-                    setArg(key, true);
+                    setArg(key, flags.strings[key] ? '' : true);
                 }
             }
         }

diff --git a/test/parse.js b/test/parse.js
@@ -162,6 +162,26 @@ test('stringArgs', function (t) {
     t.end();
 });
 
+test('empty strings', function(t) {
+    var s = parse([ '-s' ], { string: 's' }).s;
+    t.equal(s, '');
+    t.equal(typeof s, 'string');
+
+    var str = parse([ '--str' ], { string: 'str' }).str;
+    t.equal(str, '');
+    t.equal(typeof str, 'string');
+
+    var letters = parse([ '-at' ], {
+        string: 't'
+    });
+
+    t.equal(letters.a, true);
+    t.equal(letters.t, '');
+
+    t.end();
+});
+
+
 test('slashBreak', function (t) {
     t.same(
         parse([ '-I/foo/bar/baz' ]),