From ebd07e6bd9be54cf9370f08296541017bad41386 Mon Sep 17 00:00:00 2001
From: Toru Nagashima <star.ctor@gmail.com>
Date: Sun, 6 Jan 2019 00:29:33 +0900
Subject: [PATCH] New: some core rules for <template> (#680)

* add core rule wrapper
* add `array-bracket-spacing` rule
* add `key-spacing` rule
* add `eqeqeq` rule
* add `object-curly-spacing` rule
* add `space-infix-ops` rule
* add `space-unary-ops` rule
* switch error messages for minimal version tests
---
 docs/rules/README.md                          |  8 +-
 docs/rules/array-bracket-spacing.md           | 23 ++++++
 docs/rules/eqeqeq.md                          | 23 ++++++
 docs/rules/key-spacing.md                     | 23 ++++++
 docs/rules/object-curly-spacing.md            | 23 ++++++
 docs/rules/space-infix-ops.md                 | 23 ++++++
 docs/rules/space-unary-ops.md                 | 23 ++++++
 .../consistent-docs-description.js            | 14 ++--
 eslint-internal-rules/no-invalid-meta.js      | 11 +--
 lib/configs/no-layout-rules.js                |  7 +-
 lib/index.js                                  |  6 ++
 lib/rules/array-bracket-spacing.js            |  9 +++
 lib/rules/eqeqeq.js                           |  9 +++
 lib/rules/key-spacing.js                      |  9 +++
 lib/rules/object-curly-spacing.js             |  9 +++
 lib/rules/space-infix-ops.js                  |  9 +++
 lib/rules/space-unary-ops.js                  |  9 +++
 lib/utils/index.js                            | 64 +++++++++++++++
 package.json                                  |  1 +
 tests/lib/rules/array-bracket-spacing.js      | 79 +++++++++++++++++++
 tests/lib/rules/eqeqeq.js                     | 24 ++++++
 tests/lib/rules/key-spacing.js                | 27 +++++++
 tests/lib/rules/object-curly-spacing.js       | 79 +++++++++++++++++++
 tests/lib/rules/space-infix-ops.js            | 38 +++++++++
 tests/lib/rules/space-unary-ops.js            | 29 +++++++
 25 files changed, 566 insertions(+), 13 deletions(-)
 create mode 100644 docs/rules/array-bracket-spacing.md
 create mode 100644 docs/rules/eqeqeq.md
 create mode 100644 docs/rules/key-spacing.md
 create mode 100644 docs/rules/object-curly-spacing.md
 create mode 100644 docs/rules/space-infix-ops.md
 create mode 100644 docs/rules/space-unary-ops.md
 create mode 100644 lib/rules/array-bracket-spacing.js
 create mode 100644 lib/rules/eqeqeq.js
 create mode 100644 lib/rules/key-spacing.js
 create mode 100644 lib/rules/object-curly-spacing.js
 create mode 100644 lib/rules/space-infix-ops.js
 create mode 100644 lib/rules/space-unary-ops.js
 create mode 100644 tests/lib/rules/array-bracket-spacing.js
 create mode 100644 tests/lib/rules/eqeqeq.js
 create mode 100644 tests/lib/rules/key-spacing.js
 create mode 100644 tests/lib/rules/object-curly-spacing.js
 create mode 100644 tests/lib/rules/space-infix-ops.js
 create mode 100644 tests/lib/rules/space-unary-ops.js

diff --git a/docs/rules/README.md b/docs/rules/README.md
index c548e5163..513827d01 100644
--- a/docs/rules/README.md
+++ b/docs/rules/README.md
@@ -132,16 +132,22 @@ For example:
 ```json
 {
   "rules": {
-    "vue/component-name-in-template-casing": "error"
+    "vue/array-bracket-spacing": "error"
   }
 }
 ```
 
 | Rule ID | Description |    |
 |:--------|:------------|:---|
+| [vue/array-bracket-spacing](./array-bracket-spacing.md) | enforce consistent spacing inside array brackets | :wrench: |
 | [vue/component-name-in-template-casing](./component-name-in-template-casing.md) | enforce specific casing for the component naming style in template | :wrench: |
+| [vue/eqeqeq](./eqeqeq.md) | require the use of `===` and `!==` | :wrench: |
+| [vue/key-spacing](./key-spacing.md) | enforce consistent spacing between keys and values in object literal properties | :wrench: |
 | [vue/match-component-file-name](./match-component-file-name.md) | require component name property to match its file name |  |
+| [vue/object-curly-spacing](./object-curly-spacing.md) | enforce consistent spacing inside braces | :wrench: |
 | [vue/script-indent](./script-indent.md) | enforce consistent indentation in `<script>` | :wrench: |
+| [vue/space-infix-ops](./space-infix-ops.md) | require spacing around infix operators | :wrench: |
+| [vue/space-unary-ops](./space-unary-ops.md) | enforce consistent spacing before or after unary operators | :wrench: |
 
 ## Deprecated
 
diff --git a/docs/rules/array-bracket-spacing.md b/docs/rules/array-bracket-spacing.md
new file mode 100644
index 000000000..7d1040f1f
--- /dev/null
+++ b/docs/rules/array-bracket-spacing.md
@@ -0,0 +1,23 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/array-bracket-spacing
+description: enforce consistent spacing inside array brackets
+---
+# vue/array-bracket-spacing
+> enforce consistent spacing inside array brackets
+
+- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
+
+This rule is the same rule as core [array-bracket-spacing] rule but it applies to the expressions in `<template>`.
+
+## :books: Further reading
+
+- [array-bracket-spacing]
+
+[array-bracket-spacing]: https://eslint.org/docs/rules/array-bracket-spacing
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/array-bracket-spacing.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/array-bracket-spacing.js)
diff --git a/docs/rules/eqeqeq.md b/docs/rules/eqeqeq.md
new file mode 100644
index 000000000..06a0a0b67
--- /dev/null
+++ b/docs/rules/eqeqeq.md
@@ -0,0 +1,23 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/eqeqeq
+description: require the use of `===` and `!==`
+---
+# vue/eqeqeq
+> require the use of `===` and `!==`
+
+- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
+
+This rule is the same rule as core [eqeqeq] rule but it applies to the expressions in `<template>`.
+
+## :books: Further reading
+
+- [eqeqeq]
+
+[eqeqeq]: https://eslint.org/docs/rules/eqeqeq
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/eqeqeq.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/eqeqeq.js)
diff --git a/docs/rules/key-spacing.md b/docs/rules/key-spacing.md
new file mode 100644
index 000000000..afed8557e
--- /dev/null
+++ b/docs/rules/key-spacing.md
@@ -0,0 +1,23 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/key-spacing
+description: enforce consistent spacing between keys and values in object literal properties
+---
+# vue/key-spacing
+> enforce consistent spacing between keys and values in object literal properties
+
+- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
+
+This rule is the same rule as core [key-spacing] rule but it applies to the expressions in `<template>`.
+
+## :books: Further reading
+
+- [key-spacing]
+
+[key-spacing]: https://eslint.org/docs/rules/key-spacing
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/key-spacing.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/key-spacing.js)
diff --git a/docs/rules/object-curly-spacing.md b/docs/rules/object-curly-spacing.md
new file mode 100644
index 000000000..c9ee00a89
--- /dev/null
+++ b/docs/rules/object-curly-spacing.md
@@ -0,0 +1,23 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/object-curly-spacing
+description: enforce consistent spacing inside braces
+---
+# vue/object-curly-spacing
+> enforce consistent spacing inside braces
+
+- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
+
+This rule is the same rule as core [object-curly-spacing] rule but it applies to the expressions in `<template>`.
+
+## :books: Further reading
+
+- [object-curly-spacing]
+
+[object-curly-spacing]: https://eslint.org/docs/rules/object-curly-spacing
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/object-curly-spacing.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/object-curly-spacing.js)
diff --git a/docs/rules/space-infix-ops.md b/docs/rules/space-infix-ops.md
new file mode 100644
index 000000000..14301e4ad
--- /dev/null
+++ b/docs/rules/space-infix-ops.md
@@ -0,0 +1,23 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/space-infix-ops
+description: require spacing around infix operators
+---
+# vue/space-infix-ops
+> require spacing around infix operators
+
+- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
+
+This rule is the same rule as core [space-infix-ops] rule but it applies to the expressions in `<template>`.
+
+## :books: Further reading
+
+- [space-infix-ops]
+
+[space-infix-ops]: https://eslint.org/docs/rules/space-infix-ops
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/space-infix-ops.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/space-infix-ops.js)
diff --git a/docs/rules/space-unary-ops.md b/docs/rules/space-unary-ops.md
new file mode 100644
index 000000000..63976d3a0
--- /dev/null
+++ b/docs/rules/space-unary-ops.md
@@ -0,0 +1,23 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/space-unary-ops
+description: enforce consistent spacing before or after unary operators
+---
+# vue/space-unary-ops
+> enforce consistent spacing before or after unary operators
+
+- :wrench: The `--fix` option on the [command line](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
+
+This rule is the same rule as core [space-unary-ops] rule but it applies to the expressions in `<template>`.
+
+## :books: Further reading
+
+- [space-unary-ops]
+
+[space-unary-ops]: https://eslint.org/docs/rules/space-unary-ops
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/space-unary-ops.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/space-unary-ops.js)
diff --git a/eslint-internal-rules/consistent-docs-description.js b/eslint-internal-rules/consistent-docs-description.js
index 42e5970a5..f93149e78 100644
--- a/eslint-internal-rules/consistent-docs-description.js
+++ b/eslint-internal-rules/consistent-docs-description.js
@@ -23,14 +23,15 @@ const ALLOWED_FIRST_WORDS = [
  * @returns {ASTNode} The Property node or null if not found.
  */
 function getPropertyFromObject (property, node) {
-  const properties = node.properties
+  if (node && node.type === 'ObjectExpression') {
+    const properties = node.properties
 
-  for (let i = 0; i < properties.length; i++) {
-    if (properties[i].key.name === property) {
-      return properties[i]
+    for (let i = 0; i < properties.length; i++) {
+      if (properties[i].key.name === property) {
+        return properties[i]
+      }
     }
   }
-
   return null
 }
 
@@ -128,7 +129,8 @@ module.exports = {
             node.right &&
             node.left.type === 'MemberExpression' &&
             node.left.object.name === 'module' &&
-            node.left.property.name === 'exports') {
+            node.left.property.name === 'exports' &&
+            node.right.type === 'ObjectExpression') {
           checkMetaDocsDescription(context, node.right)
         }
       }
diff --git a/eslint-internal-rules/no-invalid-meta.js b/eslint-internal-rules/no-invalid-meta.js
index af4afd8e3..bd96f4f74 100644
--- a/eslint-internal-rules/no-invalid-meta.js
+++ b/eslint-internal-rules/no-invalid-meta.js
@@ -17,14 +17,15 @@
  * @returns {ASTNode} The Property node or null if not found.
  */
 function getPropertyFromObject (property, node) {
-  const properties = node.properties
+  if (node && node.type === 'ObjectExpression') {
+    const properties = node.properties
 
-  for (let i = 0; i < properties.length; i++) {
-    if (properties[i].key.name === property) {
-      return properties[i]
+    for (let i = 0; i < properties.length; i++) {
+      if (properties[i].key.name === property) {
+        return properties[i]
+      }
     }
   }
-
   return null
 }
 
diff --git a/lib/configs/no-layout-rules.js b/lib/configs/no-layout-rules.js
index 052301e5f..560bde3cf 100644
--- a/lib/configs/no-layout-rules.js
+++ b/lib/configs/no-layout-rules.js
@@ -5,17 +5,22 @@
  */
 module.exports = {
   rules: {
+    'vue/array-bracket-spacing': 'off',
     'vue/html-closing-bracket-newline': 'off',
     'vue/html-closing-bracket-spacing': 'off',
     'vue/html-indent': 'off',
     'vue/html-quotes': 'off',
     'vue/html-self-closing': 'off',
+    'vue/key-spacing': 'off',
     'vue/max-attributes-per-line': 'off',
     'vue/multiline-html-element-content-newline': 'off',
     'vue/mustache-interpolation-spacing': 'off',
     'vue/no-multi-spaces': 'off',
     'vue/no-spaces-around-equal-signs-in-attribute': 'off',
+    'vue/object-curly-spacing': 'off',
     'vue/script-indent': 'off',
-    'vue/singleline-html-element-content-newline': 'off'
+    'vue/singleline-html-element-content-newline': 'off',
+    'vue/space-infix-ops': 'off',
+    'vue/space-unary-ops': 'off'
   }
 }
diff --git a/lib/index.js b/lib/index.js
index b74b3dc8f..2cc622b09 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -7,10 +7,12 @@
 
 module.exports = {
   rules: {
+    'array-bracket-spacing': require('./rules/array-bracket-spacing'),
     'attribute-hyphenation': require('./rules/attribute-hyphenation'),
     'attributes-order': require('./rules/attributes-order'),
     'comment-directive': require('./rules/comment-directive'),
     'component-name-in-template-casing': require('./rules/component-name-in-template-casing'),
+    'eqeqeq': require('./rules/eqeqeq'),
     'html-closing-bracket-newline': require('./rules/html-closing-bracket-newline'),
     'html-closing-bracket-spacing': require('./rules/html-closing-bracket-spacing'),
     'html-end-tags': require('./rules/html-end-tags'),
@@ -18,6 +20,7 @@ module.exports = {
     'html-quotes': require('./rules/html-quotes'),
     'html-self-closing': require('./rules/html-self-closing'),
     'jsx-uses-vars': require('./rules/jsx-uses-vars'),
+    'key-spacing': require('./rules/key-spacing'),
     'match-component-file-name': require('./rules/match-component-file-name'),
     'max-attributes-per-line': require('./rules/max-attributes-per-line'),
     'multiline-html-element-content-newline': require('./rules/multiline-html-element-content-newline'),
@@ -40,6 +43,7 @@ module.exports = {
     'no-unused-vars': require('./rules/no-unused-vars'),
     'no-use-v-if-with-v-for': require('./rules/no-use-v-if-with-v-for'),
     'no-v-html': require('./rules/no-v-html'),
+    'object-curly-spacing': require('./rules/object-curly-spacing'),
     'order-in-components': require('./rules/order-in-components'),
     'prop-name-casing': require('./rules/prop-name-casing'),
     'require-component-is': require('./rules/require-component-is'),
@@ -52,6 +56,8 @@ module.exports = {
     'return-in-computed-property': require('./rules/return-in-computed-property'),
     'script-indent': require('./rules/script-indent'),
     'singleline-html-element-content-newline': require('./rules/singleline-html-element-content-newline'),
+    'space-infix-ops': require('./rules/space-infix-ops'),
+    'space-unary-ops': require('./rules/space-unary-ops'),
     'this-in-template': require('./rules/this-in-template'),
     'use-v-on-exact': require('./rules/use-v-on-exact'),
     'v-bind-style': require('./rules/v-bind-style'),
diff --git a/lib/rules/array-bracket-spacing.js b/lib/rules/array-bracket-spacing.js
new file mode 100644
index 000000000..e03abeb08
--- /dev/null
+++ b/lib/rules/array-bracket-spacing.js
@@ -0,0 +1,9 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const { wrapCoreRule } = require('../utils')
+
+// eslint-disable-next-line
+module.exports = wrapCoreRule(require('eslint/lib/rules/array-bracket-spacing'))
diff --git a/lib/rules/eqeqeq.js b/lib/rules/eqeqeq.js
new file mode 100644
index 000000000..5e9d4d8b9
--- /dev/null
+++ b/lib/rules/eqeqeq.js
@@ -0,0 +1,9 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const { wrapCoreRule } = require('../utils')
+
+// eslint-disable-next-line
+module.exports = wrapCoreRule(require('eslint/lib/rules/eqeqeq'))
diff --git a/lib/rules/key-spacing.js b/lib/rules/key-spacing.js
new file mode 100644
index 000000000..c7c46fcf6
--- /dev/null
+++ b/lib/rules/key-spacing.js
@@ -0,0 +1,9 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const { wrapCoreRule } = require('../utils')
+
+// eslint-disable-next-line
+module.exports = wrapCoreRule(require('eslint/lib/rules/key-spacing'))
diff --git a/lib/rules/object-curly-spacing.js b/lib/rules/object-curly-spacing.js
new file mode 100644
index 000000000..fc41b0d36
--- /dev/null
+++ b/lib/rules/object-curly-spacing.js
@@ -0,0 +1,9 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const { wrapCoreRule } = require('../utils')
+
+// eslint-disable-next-line
+module.exports = wrapCoreRule(require('eslint/lib/rules/object-curly-spacing'))
diff --git a/lib/rules/space-infix-ops.js b/lib/rules/space-infix-ops.js
new file mode 100644
index 000000000..fb94b74b2
--- /dev/null
+++ b/lib/rules/space-infix-ops.js
@@ -0,0 +1,9 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const { wrapCoreRule } = require('../utils')
+
+// eslint-disable-next-line
+module.exports = wrapCoreRule(require('eslint/lib/rules/space-infix-ops'))
diff --git a/lib/rules/space-unary-ops.js b/lib/rules/space-unary-ops.js
new file mode 100644
index 000000000..419607929
--- /dev/null
+++ b/lib/rules/space-unary-ops.js
@@ -0,0 +1,9 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const { wrapCoreRule } = require('../utils')
+
+// eslint-disable-next-line
+module.exports = wrapCoreRule(require('eslint/lib/rules/space-unary-ops'))
diff --git a/lib/utils/index.js b/lib/utils/index.js
index 009bbae8e..c36d0cdac 100644
--- a/lib/utils/index.js
+++ b/lib/utils/index.js
@@ -13,8 +13,29 @@ const HTML_ELEMENT_NAMES = new Set(require('./html-elements.json'))
 const SVG_ELEMENT_NAMES = new Set(require('./svg-elements.json'))
 const VOID_ELEMENT_NAMES = new Set(require('./void-elements.json'))
 const assert = require('assert')
+const path = require('path')
 const vueEslintParser = require('vue-eslint-parser')
 
+/**
+ * Wrap the rule context object to override methods which access to tokens (such as getTokenAfter).
+ * @param {RuleContext} context The rule context object.
+ * @param {TokenStore} tokenStore The token store object for template.
+ */
+function wrapContextToOverrideTokenMethods (context, tokenStore) {
+  const sourceCode = new Proxy(context.getSourceCode(), {
+    get (object, key) {
+      return key in tokenStore ? tokenStore[key] : object[key]
+    }
+  })
+
+  return {
+    __proto__: context,
+    getSourceCode () {
+      return sourceCode
+    }
+  }
+}
+
 // ------------------------------------------------------------------------------
 // Exports
 // ------------------------------------------------------------------------------
@@ -41,6 +62,49 @@ module.exports = {
     return context.parserServices.defineTemplateBodyVisitor(templateBodyVisitor, scriptVisitor)
   },
 
+  /**
+   * Wrap a given core rule to apply it to Vue.js template.
+   * @param {Rule} coreRule The core rule implementation to wrap.
+   * @param {string|undefined} category The category of this rule.
+   * @returns {Rule} The wrapped rule implementation.
+   */
+  wrapCoreRule (coreRule, category) {
+    return {
+      create (context) {
+        const tokenStore =
+          context.parserServices.getTemplateBodyTokenStore &&
+          context.parserServices.getTemplateBodyTokenStore()
+
+        // The `context.getSourceCode()` cannot access the tokens of templates.
+        // So override the methods which access to tokens by the `tokenStore`.
+        if (tokenStore) {
+          context = wrapContextToOverrideTokenMethods(context, tokenStore)
+        }
+
+        // Move `Program` handlers to `VElement[parent.type!='VElement']`
+        const handlers = coreRule.create(context)
+        if (handlers.Program) {
+          handlers["VElement[parent.type!='VElement']"] = handlers.Program
+          delete handlers.Program
+        }
+        if (handlers['Program:exit']) {
+          handlers["VElement[parent.type!='VElement']:exit"] = handlers['Program:exit']
+          delete handlers['Program:exit']
+        }
+
+        // Apply the handlers to templates.
+        return module.exports.defineTemplateBodyVisitor(context, handlers)
+      },
+
+      meta: Object.assign({}, coreRule.meta, {
+        docs: Object.assign({}, coreRule.meta.docs, {
+          category,
+          url: `https://vuejs.github.io/eslint-plugin-vue/rules/${path.basename(coreRule.meta.docs.url || '')}.html`
+        })
+      })
+    }
+  },
+
   /**
    * Check whether the given node is the root element or not.
    * @param {ASTNode} node The element node to check.
diff --git a/package.json b/package.json
index 6389b7db1..e8d7149a5 100644
--- a/package.json
+++ b/package.json
@@ -59,6 +59,7 @@
     "lodash": "^4.17.4",
     "mocha": "^5.2.0",
     "nyc": "^12.0.2",
+    "semver": "^5.6.0",
     "typescript": "^3.1.3",
     "typescript-eslint-parser": "^20.0.0",
     "vue-eslint-editor": "^0.1.4",
diff --git a/tests/lib/rules/array-bracket-spacing.js b/tests/lib/rules/array-bracket-spacing.js
new file mode 100644
index 000000000..509ebb1d1
--- /dev/null
+++ b/tests/lib/rules/array-bracket-spacing.js
@@ -0,0 +1,79 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../lib/rules/array-bracket-spacing')
+
+const tester = new RuleTester({
+  parser: 'vue-eslint-parser',
+  parserOptions: { ecmaVersion: 2015 }
+})
+
+tester.run('array-bracket-spacing', rule, {
+  valid: [
+    '<template><div :attr="[a]" /></template>',
+    {
+      code: '<template><div :attr="[a]" /></template>',
+      options: ['never']
+    },
+    {
+      code: '<template><div :attr="[ a ]" /></template>',
+      options: ['always']
+    }
+  ],
+  invalid: [
+    {
+      code: '<template><div :attr="[ a]" /></template>',
+      errors: ["There should be no space after '['."]
+    },
+    {
+      code: '<template><div :attr="[a ]" /></template>',
+      errors: ["There should be no space before ']'."]
+    },
+    {
+      code: '<template><div :attr="[ a ]" /></template>',
+      errors: [
+        "There should be no space after '['.",
+        "There should be no space before ']'."
+      ]
+    },
+    {
+      code: '<template><div :attr="[ a]" /></template>',
+      options: ['never'],
+      errors: ["There should be no space after '['."]
+    },
+    {
+      code: '<template><div :attr="[a ]" /></template>',
+      options: ['never'],
+      errors: ["There should be no space before ']'."]
+    },
+    {
+      code: '<template><div :attr="[ a ]" /></template>',
+      options: ['never'],
+      errors: [
+        "There should be no space after '['.",
+        "There should be no space before ']'."
+      ]
+    },
+    {
+      code: '<template><div :attr="[ a]" /></template>',
+      options: ['always'],
+      errors: ["A space is required before ']'."]
+    },
+    {
+      code: '<template><div :attr="[a ]" /></template>',
+      options: ['always'],
+      errors: ["A space is required after '['."]
+    },
+    {
+      code: '<template><div :attr="[a]" /></template>',
+      options: ['always'],
+      errors: [
+        "A space is required after '['.",
+        "A space is required before ']'."
+      ]
+    }
+  ]
+})
diff --git a/tests/lib/rules/eqeqeq.js b/tests/lib/rules/eqeqeq.js
new file mode 100644
index 000000000..f637df57c
--- /dev/null
+++ b/tests/lib/rules/eqeqeq.js
@@ -0,0 +1,24 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../lib/rules/eqeqeq')
+
+const tester = new RuleTester({
+  parser: 'vue-eslint-parser',
+  parserOptions: { ecmaVersion: 2015 }
+})
+
+tester.run('eqeqeq', rule, {
+  valid: [
+    '<template><div :attr="a === 1" /></template>'
+  ],
+  invalid: [
+    {
+      code: '<template><div :attr="a == 1" /></template>',
+      errors: ["Expected '===' and instead saw '=='."]
+    }
+  ]
+})
diff --git a/tests/lib/rules/key-spacing.js b/tests/lib/rules/key-spacing.js
new file mode 100644
index 000000000..52adaaf9d
--- /dev/null
+++ b/tests/lib/rules/key-spacing.js
@@ -0,0 +1,27 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../lib/rules/key-spacing')
+
+const tester = new RuleTester({
+  parser: 'vue-eslint-parser',
+  parserOptions: { ecmaVersion: 2015 }
+})
+
+tester.run('key-spacing', rule, {
+  valid: [
+    '<template><div :attr="{a: 1}" /></template>'
+  ],
+  invalid: [
+    {
+      code: '<template><div :attr="{a :1}" /></template>',
+      errors: [
+        "Extra space after key 'a'.",
+        "Missing space before value for key 'a'."
+      ]
+    }
+  ]
+})
diff --git a/tests/lib/rules/object-curly-spacing.js b/tests/lib/rules/object-curly-spacing.js
new file mode 100644
index 000000000..bf6cbc8a8
--- /dev/null
+++ b/tests/lib/rules/object-curly-spacing.js
@@ -0,0 +1,79 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../lib/rules/object-curly-spacing')
+
+const tester = new RuleTester({
+  parser: 'vue-eslint-parser',
+  parserOptions: { ecmaVersion: 2015 }
+})
+
+tester.run('object-curly-spacing', rule, {
+  valid: [
+    '<template><div :attr="{a: 1}" /></template>',
+    {
+      code: '<template><div :attr="{a: 1}" /></template>',
+      options: ['never']
+    },
+    {
+      code: '<template><div :attr="{ a: 1 }" /></template>',
+      options: ['always']
+    }
+  ],
+  invalid: [
+    {
+      code: '<template><div :attr="{ a: 1}" /></template>',
+      errors: ["There should be no space after '{'."]
+    },
+    {
+      code: '<template><div :attr="{a: 1 }" /></template>',
+      errors: ["There should be no space before '}'."]
+    },
+    {
+      code: '<template><div :attr="{ a: 1 }" /></template>',
+      errors: [
+        "There should be no space after '{'.",
+        "There should be no space before '}'."
+      ]
+    },
+    {
+      code: '<template><div :attr="{ a: 1}" /></template>',
+      options: ['never'],
+      errors: ["There should be no space after '{'."]
+    },
+    {
+      code: '<template><div :attr="{a: 1 }" /></template>',
+      options: ['never'],
+      errors: ["There should be no space before '}'."]
+    },
+    {
+      code: '<template><div :attr="{ a: 1 }" /></template>',
+      options: ['never'],
+      errors: [
+        "There should be no space after '{'.",
+        "There should be no space before '}'."
+      ]
+    },
+    {
+      code: '<template><div :attr="{ a: 1}" /></template>',
+      options: ['always'],
+      errors: ["A space is required before '}'."]
+    },
+    {
+      code: '<template><div :attr="{a: 1 }" /></template>',
+      options: ['always'],
+      errors: ["A space is required after '{'."]
+    },
+    {
+      code: '<template><div :attr="{a: 1}" /></template>',
+      options: ['always'],
+      errors: [
+        "A space is required after '{'.",
+        "A space is required before '}'."
+      ]
+    }
+  ]
+})
diff --git a/tests/lib/rules/space-infix-ops.js b/tests/lib/rules/space-infix-ops.js
new file mode 100644
index 000000000..ba67575ff
--- /dev/null
+++ b/tests/lib/rules/space-infix-ops.js
@@ -0,0 +1,38 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const { RuleTester, CLIEngine } = require('eslint')
+const semver = require('semver')
+const rule = require('../../../lib/rules/space-infix-ops')
+
+const tester = new RuleTester({
+  parser: 'vue-eslint-parser',
+  parserOptions: { ecmaVersion: 2015 }
+})
+
+const message = semver.lt(CLIEngine.version, '5.10.0')
+  ? () => 'Infix operators must be spaced.'
+  : operator => `Operator '${operator}' must be spaced.`
+
+tester.run('space-infix-ops', rule, {
+  valid: [
+    '<template><div :attr="a + 1" /></template>',
+    '<template><div :attr="a ? 1 : 2" /></template>'
+  ],
+  invalid: [
+    {
+      code: '<template><div :attr="a+1" /></template>',
+      errors: [message('+')]
+    },
+    {
+      code: '<template><div :attr="a?1 : 2" /></template>',
+      errors: [message('?')]
+    },
+    {
+      code: '<template><div :attr="a ? 1:2" /></template>',
+      errors: [message(':')]
+    }
+  ]
+})
diff --git a/tests/lib/rules/space-unary-ops.js b/tests/lib/rules/space-unary-ops.js
new file mode 100644
index 000000000..2f36302bb
--- /dev/null
+++ b/tests/lib/rules/space-unary-ops.js
@@ -0,0 +1,29 @@
+/**
+ * @author Toru Nagashima
+ */
+'use strict'
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../lib/rules/space-unary-ops')
+
+const tester = new RuleTester({
+  parser: 'vue-eslint-parser',
+  parserOptions: { ecmaVersion: 2015 }
+})
+
+tester.run('space-unary-ops', rule, {
+  valid: [
+    '<template><div :attr="-a" /></template>',
+    '<template><div :attr="typeof a" /></template>'
+  ],
+  invalid: [
+    {
+      code: '<template><div :attr="- a" /></template>',
+      errors: ['Unexpected space after unary operator \'-\'.']
+    },
+    {
+      code: '<template><div :attr="typeof(a)" /></template>',
+      errors: ['Unary word operator \'typeof\' must be followed by whitespace.']
+    }
+  ]
+})