From 28e14f0815c0649aaeddd4dabff7b701b442b30a Mon Sep 17 00:00:00 2001
From: yosuke ota <otameshiyo23@gmail.com>
Date: Tue, 19 May 2020 13:44:00 +0900
Subject: [PATCH] Add `vue/prefer-template` rule

---
 docs/rules/README.md               |  1 +
 docs/rules/prefer-template.md      | 23 +++++++++++
 lib/index.js                       |  1 +
 lib/rules/prefer-template.js       | 11 ++++++
 tests/lib/rules/prefer-template.js | 63 ++++++++++++++++++++++++++++++
 5 files changed, 99 insertions(+)
 create mode 100644 docs/rules/prefer-template.md
 create mode 100644 lib/rules/prefer-template.js
 create mode 100644 tests/lib/rules/prefer-template.js

diff --git a/docs/rules/README.md b/docs/rules/README.md
index 83eda5692..dee76cd27 100644
--- a/docs/rules/README.md
+++ b/docs/rules/README.md
@@ -292,6 +292,7 @@ For example:
 | [vue/no-unsupported-features](./no-unsupported-features.md) | disallow unsupported Vue.js syntax on the specified version | :wrench: |
 | [vue/object-curly-spacing](./object-curly-spacing.md) | enforce consistent spacing inside braces | :wrench: |
 | [vue/padding-line-between-blocks](./padding-line-between-blocks.md) | require or disallow padding lines between blocks | :wrench: |
+| [vue/prefer-template](./prefer-template.md) | require template literals instead of string concatenation | :wrench: |
 | [vue/require-direct-export](./require-direct-export.md) | require the component to be directly exported |  |
 | [vue/require-explicit-emits](./require-explicit-emits.md) | require `emits` option with name triggered by `$emit()` |  |
 | [vue/require-name-property](./require-name-property.md) | require a name property in Vue components |  |
diff --git a/docs/rules/prefer-template.md b/docs/rules/prefer-template.md
new file mode 100644
index 000000000..cbc6d8125
--- /dev/null
+++ b/docs/rules/prefer-template.md
@@ -0,0 +1,23 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/prefer-template
+description: require template literals instead of string concatenation
+---
+# vue/prefer-template
+> require template literals instead of string concatenation
+
+- :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 [prefer-template] rule but it applies to the expressions in `<template>`.
+
+## :books: Further reading
+
+- [prefer-template]
+
+[prefer-template]: https://eslint.org/docs/rules/prefer-template
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/prefer-template.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/prefer-template.js)
diff --git a/lib/index.js b/lib/index.js
index 6adbbecd4..5b6196cb2 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -89,6 +89,7 @@ module.exports = {
     'object-curly-spacing': require('./rules/object-curly-spacing'),
     'order-in-components': require('./rules/order-in-components'),
     'padding-line-between-blocks': require('./rules/padding-line-between-blocks'),
+    'prefer-template': require('./rules/prefer-template'),
     'prop-name-casing': require('./rules/prop-name-casing'),
     'require-component-is': require('./rules/require-component-is'),
     'require-default-prop': require('./rules/require-default-prop'),
diff --git a/lib/rules/prefer-template.js b/lib/rules/prefer-template.js
new file mode 100644
index 000000000..70e041e57
--- /dev/null
+++ b/lib/rules/prefer-template.js
@@ -0,0 +1,11 @@
+/**
+ * @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/prefer-template')
+)
diff --git a/tests/lib/rules/prefer-template.js b/tests/lib/rules/prefer-template.js
new file mode 100644
index 000000000..73ad5e5b6
--- /dev/null
+++ b/tests/lib/rules/prefer-template.js
@@ -0,0 +1,63 @@
+/**
+ * @author Yosuke Ota
+ */
+'use strict'
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../lib/rules/prefer-template')
+
+const tester = new RuleTester({
+  parser: require.resolve('vue-eslint-parser'),
+  parserOptions: { ecmaVersion: 2020 }
+})
+
+tester.run('prefer-template', rule, {
+  valid: [
+    `
+    <template>
+      <div :class="[\`foo-\${bar}\`]" />
+    </template>
+    `,
+    `
+    <template>
+      <div :[\`foo\${bar}\`]="value" />
+    </template>
+    `
+  ],
+  invalid: [
+    {
+      code: `
+      <template>
+        <div :class="['foo-' + bar]" />
+      </template>
+      `,
+      output: `
+      <template>
+        <div :class="[\`foo-\${  bar}\`]" />
+      </template>
+      `,
+      errors: [
+        {
+          message: 'Unexpected string concatenation.',
+          line: 3
+        }
+      ]
+    },
+    {
+      code: `
+      <template>
+        <div :['foo'+bar]="value" />
+      </template>`,
+      output: `
+      <template>
+        <div :[\`foo\${bar}\`]="value" />
+      </template>`,
+      errors: [
+        {
+          message: 'Unexpected string concatenation.',
+          line: 3
+        }
+      ]
+    }
+  ]
+})