ifNesting of argumentSpacing should accept the same configuration as its parent. about jscodesniffer HOT 3 CLOSED

/*jshint -W068 */
/*jshint multistr: true */
var Sniffer = require( "../lib/Sniffer" );

require( "should" );

Array.prototype.hasErrorCode = function( errCode ) {
  return !!this.filter(function( msg ){
    return msg.errorCode === errCode;
  }).length;
};

var OPTIONS = {
      standard: "Jquery"
  };

describe( " Custom checks ", function () {
  var sniffer, logger = null;
  beforeEach(function(){
    sniffer = new Sniffer();
  });

  it(" must implement custom standard correctly", function () {
   var code = "define('library/uielements/textinput', [\
  'core'\
], function(require) {\
  var core = require('core'),\
      _textinput = {\
        create: function(input, type, options) {\
          switch(type) {\
            case _self.TYPE_NORMAL:\
              _self.normal(input, options);\
              break;\
          }\
       }\
     };\
});",

    modifiers = {
      "Indentation": false,
      "QuoteConventions": false,
      "ParametersSpacing": false,
      "LineLength": false,
      "ArrayLiteralSpacing": false,
      "ObjectLiteralSpacing": false,
      "CompoundStatementConventions": false,
      "ArgumentsSpacing": {
        "allowArgPrecedingWhitespaces": 1,
        "allowArgTrailingWhitespaces": 0,
        "exceptions": {
          "singleArg" : {
            "for": ["FunctionExpression", "ArrayExpression", "ObjectExpression", "Literal"],
            "allowArgPrecedingWhitespaces": 0,
            "allowArgTrailingWhitespaces": 0
          },
          "firstArg": {
            "for": [ "FunctionExpression" , "ArrayExpression", "ObjectExpression", "Literal"],
            "allowArgPrecedingWhitespaces": 0
          },
          "lastArg": {
            "for": [ "FunctionExpression", , "ArrayExpression", "ObjectExpression", "Literal" ],
            "allowArgTrailingWhitespaces": 0
          }
        },
        "ifNesting": false
      }
    };

    logger = sniffer.getTestResults( code, OPTIONS, modifiers );

    console.log(logger.getMessages());
    logger.getMessages().length.should.not.be.ok;
  });
});

it(" must accept ArgumentsSpacing ifNesting modifiers", function () {
        var code = "foo( 1, bar(1,1) )",
            modifiers = {
                "ArgumentsSpacing": {
                    "allowArgPrecedingWhitespaces": 1,
                    "allowArgTrailingWhitespaces": 1,
                    "ifNesting": {
                        "allowArgPrecedingWhitespaces": 0,
                        "allowArgTrailingWhitespaces": 0
                    }
                }
            };
    logger = sniffer.getTestResults( code, OPTIONS, modifiers );
        logger.getMessages().length.should.not.be.ok;
  });