Merge pull request #346 from bcoe/fix-341

alias should be applied to camel-case form of argument

lib/parser.js

@@ -43,6 +43,7 @@ module.exports = function (args, opts, y18n) {
 
   extendAliases(opts.key)
   extendAliases(opts.alias)
+  extendAliases(opts.default)
 
   var defaults = opts['default'] || {}
   Object.keys(defaults).forEach(function (key) {
@@ -436,6 +437,11 @@ module.exports = function (args, opts, y18n) {
   // extend the aliases list with inferred aliases.
   function extendAliases (obj) {
     Object.keys(obj || {}).forEach(function (key) {
+      // short-circuit if we've already added a key
+      // to the aliases array, for example it might
+      // exist in both 'opts.default' and 'opts.key'.
+      if (aliases[key]) return
+
       aliases[key] = [].concat(opts.alias[key] || [])
       // For "--option-name", also set argv.optionName
       aliases[key].concat(key).forEach(function (x) {

test/parser.js

@@ -907,6 +907,15 @@ describe('parser tests', function () {
           argv2.parse([ ]).flag.should.be.false
         })
       })
+
+      // Fixes: https://github.com/bcoe/yargs/issues/341
+      it('should apply defaults to camel-case form of argument', function () {
+        var argv = yargs([])
+          .default('foo-bar', 99)
+          .argv
+
+        argv.fooBar.should.equal(99)
+      })
     })
 
     it('should define option as boolean and set default to true', function () {