fix: ensure empty string is added into argv._ (#140)

yargs · Nov 19, 2018 · 79cda98 · 79cda98
1 parent 1c5d556
commit 79cda98
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/lib/tokenize-arg-string.js b/lib/tokenize-arg-string.js
@@ -25,6 +25,7 @@ module.exports = function (argString) {
     // don't split the string if we're in matching
     // opening or closing single and double quotes.
     if (c === opening) {
+      if (!args[i]) args[i] = ''
       opening = null
       continue
     } else if ((c === "'" || c === '"') && !opening) {

diff --git a/test/tokenize-arg-string.js b/test/tokenize-arg-string.js
@@ -32,6 +32,20 @@ describe('TokenizeArgString', function () {
     args[2].should.equal('--bar=foo bar')
   })
 
+  it('handles single quoted empty string', function () {
+    var args = tokenizeArgString('--foo \'\' --bar=\'\'')
+    args[0].should.equal('--foo')
+    args[1].should.equal('')
+    args[2].should.equal('--bar=')
+  })
+
+  it('handles double quoted empty string', function () {
+    var args = tokenizeArgString('--foo "" --bar=""')
+    args[0].should.equal('--foo')
+    args[1].should.equal('')
+    args[2].should.equal('--bar=')
+  })
+
   it('handles quoted string with embeded quotes', function () {
     var args = tokenizeArgString('--foo "hello \'world\'" --bar=\'foo "bar"\'')
     args[0].should.equal('--foo')