key-spacing fine-grained control issue

[DarkPark]

eslint v2.3.0

Rule:

"key-spacing": [1, {
    "singleLine": {
        "beforeColon": false, "afterColon": true, "mode": "strict"
    },
    "multiLine": {
        "beforeColon": false, "afterColon": true, "mode": "minimum"
    }
}],

Code:

module.exports = {
    foobar: 42,
    bat:    2 * 2
};

Expected result: no warnings.

Actual:

/home/dp/Projects/sdk/spa/plugin-css/index.js
  10:13  warning  Extra space before value for key 'bat'  key-spacing

✖ 1 problem (0 errors, 1 warning)

At the same time rule

"key-spacing": [1, {"beforeColon": false, "afterColon": true, "mode": "minimum"}],

gives no warnings
seems multiLine option is not working properly

[btmills]

Yeah, it looks like multiLine.mode isn't being applied correctly.

[btmills]

On line 373, isSingleLine is being passed a Property node, but it should be passed the ObjectExpression. @DarkPark want to submit a PR?

[DarkPark]

@btmills thanks, but I have not investigated the source code so it's better to be done by someone knowledgeable.

[platinumazure]

@btmills I'll take it.