fix: address pkgConf parsing bug outlined in #37 (#45)

yargs · Aug 9, 2016 · be76ee6 · be76ee6
1 parent 487fecf
commit be76ee6
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -351,7 +351,8 @@ function parse (args, opts) {
 
     // Set normalized value when key is in 'normalize' and in 'arrays'
     if (checkAllAliases(key, flags.normalize) && checkAllAliases(key, flags.arrays)) {
-      value = path.normalize(val)
+      if (Array.isArray(val)) value = val.map(path.normalize)
+      else value = path.normalize(val)
     }
 
     var splitKey = key.split('.')

diff --git a/test/yargs-parser.js b/test/yargs-parser.js
@@ -2025,4 +2025,14 @@ describe('yargs-parser', function () {
       parsed.error.message.should.equal('banana')
     })
   })
+
+  // see: https://github.com/yargs/yargs-parser/issues/37
+  it('normalizes all paths in array when provided via config object', function () {
+    var argv = parser([ '--foo', 'bar' ], {
+      array: ['a'],
+      normalize: ['a'],
+      configObjects: [{'a': ['bin/../a.txt', 'bin/../b.txt']}]
+    })
+    argv.a.should.deep.equal(['a.txt', 'b.txt'])
+  })
 })