From 4cb319795acbd9aec197e37e3aa8233253c83215 Mon Sep 17 00:00:00 2001
From: yosuke ota <otameshiyo23@gmail.com>
Date: Tue, 19 May 2020 13:57:45 +0900
Subject: [PATCH] Add `vue/template-curly-spacing` rule

---
 docs/rules/README.md                      |  1 +
 docs/rules/template-curly-spacing.md      | 23 ++++++
 lib/configs/no-layout-rules.js            |  3 +-
 lib/index.js                              |  1 +
 lib/rules/template-curly-spacing.js       | 12 +++
 tests/lib/rules/template-curly-spacing.js | 90 +++++++++++++++++++++++
 6 files changed, 129 insertions(+), 1 deletion(-)
 create mode 100644 docs/rules/template-curly-spacing.md
 create mode 100644 lib/rules/template-curly-spacing.js
 create mode 100644 tests/lib/rules/template-curly-spacing.js

diff --git a/docs/rules/README.md b/docs/rules/README.md
index 83eda5692..fb5a5659a 100644
--- a/docs/rules/README.md
+++ b/docs/rules/README.md
@@ -300,6 +300,7 @@ For example:
 | [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: |
 | [vue/static-class-names-order](./static-class-names-order.md) | enforce static class names order | :wrench: |
+| [vue/template-curly-spacing](./template-curly-spacing.md) | require or disallow spacing around embedded expressions of template strings | :wrench: |
 | [vue/v-on-function-call](./v-on-function-call.md) | enforce or forbid parentheses after method calls without arguments in `v-on` directives | :wrench: |
 
 ## Deprecated
diff --git a/docs/rules/template-curly-spacing.md b/docs/rules/template-curly-spacing.md
new file mode 100644
index 000000000..92daa477e
--- /dev/null
+++ b/docs/rules/template-curly-spacing.md
@@ -0,0 +1,23 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/template-curly-spacing
+description: require or disallow spacing around embedded expressions of template strings
+---
+# vue/template-curly-spacing
+> require or disallow spacing around embedded expressions of template strings
+
+- :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 [template-curly-spacing] rule but it applies to the expressions in `<template>`.
+
+## :books: Further reading
+
+- [template-curly-spacing]
+
+[template-curly-spacing]: https://eslint.org/docs/rules/template-curly-spacing
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/template-curly-spacing.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/template-curly-spacing.js)
diff --git a/lib/configs/no-layout-rules.js b/lib/configs/no-layout-rules.js
index 24e7d0f88..c24233cc7 100644
--- a/lib/configs/no-layout-rules.js
+++ b/lib/configs/no-layout-rules.js
@@ -32,6 +32,7 @@ module.exports = {
     'vue/script-indent': 'off',
     'vue/singleline-html-element-content-newline': 'off',
     'vue/space-infix-ops': 'off',
-    'vue/space-unary-ops': 'off'
+    'vue/space-unary-ops': 'off',
+    'vue/template-curly-spacing': 'off'
   }
 }
diff --git a/lib/index.js b/lib/index.js
index 6adbbecd4..ccfe1aed6 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -109,6 +109,7 @@ module.exports = {
     'space-infix-ops': require('./rules/space-infix-ops'),
     'space-unary-ops': require('./rules/space-unary-ops'),
     'static-class-names-order': require('./rules/static-class-names-order'),
+    'template-curly-spacing': require('./rules/template-curly-spacing'),
     '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/template-curly-spacing.js b/lib/rules/template-curly-spacing.js
new file mode 100644
index 000000000..777c5d820
--- /dev/null
+++ b/lib/rules/template-curly-spacing.js
@@ -0,0 +1,12 @@
+/**
+ * @author Yosuke Ota
+ */
+'use strict'
+
+const { wrapCoreRule } = require('../utils')
+
+// eslint-disable-next-line no-invalid-meta, no-invalid-meta-docs-categories
+module.exports = wrapCoreRule(
+  require('eslint/lib/rules/template-curly-spacing'),
+  { skipDynamicArguments: true }
+)
diff --git a/tests/lib/rules/template-curly-spacing.js b/tests/lib/rules/template-curly-spacing.js
new file mode 100644
index 000000000..02aeb43e5
--- /dev/null
+++ b/tests/lib/rules/template-curly-spacing.js
@@ -0,0 +1,90 @@
+/**
+ * @author Yosuke Ota
+ */
+'use strict'
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../lib/rules/template-curly-spacing')
+
+const tester = new RuleTester({
+  parser: require.resolve('vue-eslint-parser'),
+  parserOptions: { ecmaVersion: 2020 }
+})
+
+tester.run('template-curly-spacing', rule, {
+  valid: [
+    `
+    <template>
+      <div :class="[\`foo-\${bar}\`]" />
+    </template>
+    `,
+    `
+    <template>
+      <div :[\`foo\${bar}\`]="value" />
+    </template>
+    `,
+    {
+      code: `
+      <template>
+        <div :class="[\`foo-\${ bar }\`]" />
+      </template>
+      `,
+      options: ['always']
+    },
+    {
+      code: `
+      <template>
+        <div :[\`foo\${bar}\`]="value" />
+      </template>
+      `,
+      options: ['always']
+    }
+  ],
+  invalid: [
+    {
+      code: `
+      <template>
+        <div :class="[\`foo-\${ bar }\`]" />
+      </template>
+      `,
+      output: `
+      <template>
+        <div :class="[\`foo-\${bar}\`]" />
+      </template>
+      `,
+      errors: [
+        {
+          message: "Unexpected space(s) after '${'.",
+          line: 3
+        },
+        {
+          message: "Unexpected space(s) before '}'.",
+          line: 3
+        }
+      ]
+    },
+    {
+      code: `
+      <template>
+        <div :class="[\`foo-\${bar}\`]" />
+      </template>
+      `,
+      options: ['always'],
+      output: `
+      <template>
+        <div :class="[\`foo-\${ bar }\`]" />
+      </template>
+      `,
+      errors: [
+        {
+          message: "Expected space(s) after '${'.",
+          line: 3
+        },
+        {
+          message: "Expected space(s) before '}'.",
+          line: 3
+        }
+      ]
+    }
+  ]
+})