Add rule curly-bracket-spacing.

fixes #150

Author: armano2

docs/rules/curly-bracket-spacing.md

@@ -0,0 +1,67 @@
+# Enforce spacing on the style of curly brackets. (curly-bracket-spacing)
+
+- :wrench: The `--fix` option on the [command line](http://eslint.org/docs/user-guide/command-line-interface#fix) can automatically fix some of the problems reported by this rule.
+
+## :book: Rule Details
+
+This rule aims to enforce unified spacing of curly brackets.
+
+:-1: Examples of **incorrect** code for this rule:
+
+```html
+<template>
+  <div>{{   text   }}</div>
+</template>
+```
+
+:+1: Examples of **correct** code for this rule:
+
+```html
+<template>
+  <div>{{ text }}</div>
+</template>
+```
+
+## :wrench: Options
+
+Default spacing is set to `always`
+
+```
+'vue/curly-bracket-spacing': [2, 'always'|'never']
+```
+
+### `"always"` - Expect one space between expression and curly brackets.
+
+:+1: Examples of **correct** code`:
+
+```html
+<template>
+  <div>{{ text }}</div>
+</template>
+```
+
+:-1: Examples of **incorrect** code`:
+
+```html
+<template>
+  <div>{{text}}</div>
+</template>
+```
+
+### `"never"` - Expect no spaces between expression and curly brackets.
+
+:+1: Examples of **correct** code`:
+
+```html
+<template>
+  <div>{{text}}</div>
+</template>
+```
+
+:-1: Examples of **incorrect** code`:
+
+```html
+<template>
+  <div>{{ text }}</div>
+</template>
+```

lib/rules/curly-bracket-spacing.js

@@ -0,0 +1,86 @@
+/**
+ * @fileoverview Enforce spacing on the style of curly brackets.
+ * @author Armano
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const utils = require('../utils')
+
+// ------------------------------------------------------------------------------
+// Rule Definition
+// ------------------------------------------------------------------------------
+
+module.exports = {
+  meta: {
+    docs: {
+      description: 'Enforce spacing on the style of curly brackets.',
+      category: 'Stylistic Issues',
+      recommended: false
+    },
+    fixable: 'whitespace',
+    schema: [
+      {
+        enum: ['always', 'never']
+      }
+    ]
+  },
+
+  create (context) {
+    const options = context.options[0]
+    const optSpaces = options !== 'never' ? 1 : 0
+    const template = context.parserServices.getTemplateBodyTokenStore && context.parserServices.getTemplateBodyTokenStore()
+
+    // ----------------------------------------------------------------------
+    // Helpers
+    // ----------------------------------------------------------------------
+
+    function checkTokens (leftToken, rightToken) {
+      if (leftToken.loc.end.line === rightToken.loc.start.line) {
+        const spaces = rightToken.loc.start.column - leftToken.loc.end.column
+        if (optSpaces !== spaces) {
+          context.report({
+            node: rightToken,
+            loc: {
+              start: leftToken.loc.end,
+              end: rightToken.loc.start
+            },
+            message: 'Found {{spaces}} whitespaces, {{type}} expected.',
+            data: {
+              spaces: spaces === 0 ? 'none' : spaces,
+              type: optSpaces ? '1' : 'none'
+            },
+            fix: (fixer) => fixer.replaceTextRange([leftToken.range[1], rightToken.range[0]], optSpaces ? ' ' : '')
+          })
+        }
+      }
+    }
+
+    // ----------------------------------------------------------------------
+    // Public
+    // ----------------------------------------------------------------------
+
+    utils.registerTemplateBodyVisitor(context, {
+      VExpressionContainer (node) {
+        const tokens = template.getTokens(node, {
+          includeComments: true,
+          filter: token => token.type !== 'HTMLWhitespace' // When there is only whitespace between ignore it
+        })
+
+        const startToken = tokens.shift()
+        const endToken = tokens.pop()
+        if (tokens.length > 0) {
+          checkTokens(startToken, tokens[0])
+          checkTokens(tokens[tokens.length - 1], endToken)
+        } else {
+          checkTokens(startToken, endToken)
+        }
+      }
+    })
+
+    return { }
+  }
+}

package.json

@@ -45,7 +45,7 @@
   },
   "dependencies": {
     "requireindex": "^1.1.0",
-    "vue-eslint-parser": "2.0.0-beta.6"
+    "vue-eslint-parser": "2.0.0-beta.7"
   },
   "devDependencies": {
     "@types/node": "^4.2.16",

tests/lib/rules/curly-bracket-spacing.js

@@ -0,0 +1,172 @@
+/**
+ * @fileoverview Enforce spacing on the style of curly brackets.
+ * @author Armano
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const rule = require('../../../lib/rules/curly-bracket-spacing')
+const RuleTester = require('eslint').RuleTester
+
+// ------------------------------------------------------------------------------
+// Tests
+// ------------------------------------------------------------------------------
+
+const ruleTester = new RuleTester({
+  parser: 'vue-eslint-parser',
+  parserOptions: { ecmaVersion: 2015 }
+})
+
+ruleTester.run('curly-bracket-spacing', rule, {
+
+  valid: [
+    {
+      filename: 'test.vue',
+      code: '<template></template>'
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div></div></template>'
+    },
+    {
+      filename: 'test.vue',
+      code: '<template>             <div id="               "></div>         </template>'
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{ text }}</div></template>'
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{ }}</div></template>'
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{ }}</div></template>',
+      options: ['always']
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{}}</div></template>',
+      options: ['never']
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{text}}</div></template>',
+      options: ['never']
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{ text }}</div></template>',
+      options: ['always']
+    }
+  ],
+
+  invalid: [
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{         }}</div></template>',
+      output: '<template><div>{{ }}</div></template>',
+      options: ['always'],
+      errors: [{
+        message: 'Found 9 whitespaces, 1 expected.',
+        type: 'VExpressionEnd'
+      }]
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{ text}}</div></template>',
+      output: '<template><div>{{ text }}</div></template>',
+      options: ['always'],
+      errors: [{
+        message: 'Found none whitespaces, 1 expected.',
+        type: 'VExpressionEnd'
+      }]
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{text }}</div></template>',
+      output: '<template><div>{{ text }}</div></template>',
+      options: ['always'],
+      errors: [{
+        message: 'Found none whitespaces, 1 expected.',
+        type: 'Identifier'
+      }]
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{ text}}</div></template>',
+      output: '<template><div>{{text}}</div></template>',
+      options: ['never'],
+      errors: [{
+        message: 'Found 1 whitespaces, none expected.',
+        type: 'Identifier'
+      }]
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{text }}</div></template>',
+      output: '<template><div>{{text}}</div></template>',
+      options: ['never'],
+      errors: [{
+        message: 'Found 1 whitespaces, none expected.',
+        type: 'VExpressionEnd'
+      }]
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{text}}</div></template>',
+      output: '<template><div>{{ text }}</div></template>',
+      options: ['always'],
+      errors: [{
+        message: 'Found none whitespaces, 1 expected.',
+        type: 'Identifier'
+      }, {
+        message: 'Found none whitespaces, 1 expected.',
+        type: 'VExpressionEnd'
+      }]
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{ text }}</div></template>',
+      output: '<template><div>{{text}}</div></template>',
+      options: ['never'],
+      errors: [{
+        message: 'Found 1 whitespaces, none expected.',
+        type: 'Identifier'
+      }, {
+        message: 'Found 1 whitespaces, none expected.',
+        type: 'VExpressionEnd'
+      }]
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{   text   }}</div></template>',
+      output: '<template><div>{{ text }}</div></template>',
+      options: ['always'],
+      errors: [{
+        message: 'Found 3 whitespaces, 1 expected.',
+        type: 'Identifier'
+      }, {
+        message: 'Found 3 whitespaces, 1 expected.',
+        type: 'VExpressionEnd'
+      }]
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div>{{   text   }}</div></template>',
+      output: '<template><div>{{text}}</div></template>',
+      options: ['never'],
+      errors: [{
+        message: 'Found 3 whitespaces, none expected.',
+        type: 'Identifier'
+      }, {
+        message: 'Found 3 whitespaces, none expected.',
+        type: 'VExpressionEnd'
+      }]
+    }
+  ]
+})