From 9deb907c5a569ea2744ccd8fde3da1809a467cf8 Mon Sep 17 00:00:00 2001
From: Antoine SAVAJOLS <asavajols@digitaleo.com>
Date: Thu, 22 Dec 2022 09:30:32 +0100
Subject: [PATCH 1/3] Added array-element-newline rule from eslint

---
 docs/rules/array-element-newline.md      |  31 +++++++
 docs/rules/index.md                      |   1 +
 lib/configs/no-layout-rules.js           |   1 +
 lib/index.js                             |   1 +
 lib/rules/array-element-newline.js       |  11 +++
 tests/lib/rules/array-element-newline.js | 110 +++++++++++++++++++++++
 6 files changed, 155 insertions(+)
 create mode 100644 docs/rules/array-element-newline.md
 create mode 100644 lib/rules/array-element-newline.js
 create mode 100644 tests/lib/rules/array-element-newline.js

diff --git a/docs/rules/array-element-newline.md b/docs/rules/array-element-newline.md
new file mode 100644
index 000000000..024dcafe6
--- /dev/null
+++ b/docs/rules/array-element-newline.md
@@ -0,0 +1,31 @@
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/array-element-newline
+description: Enforce line breaks after each array element in `<template>`
+---
+# vue/array-element-newline
+
+> Enforce line breaks after each array element in `<template>`
+
+- :exclamation: <badge text="This rule has not been released yet." vertical="middle" type="error"> ***This rule has not been released yet.*** </badge>
+- :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-element-newline] 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
+
+- [array-bracket-newline]
+
+[array-bracket-newline]: https://eslint.org/docs/rules/array-bracket-newline
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/array-element-newline.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/array-element-newline.js)
+
+<sup>Taken with ❤️ [from ESLint core](https://eslint.org/docs/rules/array-element-newline)</sup>
diff --git a/docs/rules/index.md b/docs/rules/index.md
index 224b40313..2e21edc56 100644
--- a/docs/rules/index.md
+++ b/docs/rules/index.md
@@ -279,6 +279,7 @@ The following rules extend the rules provided by ESLint itself and apply them to
 |:--------|:------------|:--:|:--:|
 | [vue/array-bracket-newline](./array-bracket-newline.md) | Enforce linebreaks after opening and before closing array brackets in `<template>` | :wrench: | :lipstick: |
 | [vue/array-bracket-spacing](./array-bracket-spacing.md) | Enforce consistent spacing inside array brackets in `<template>` | :wrench: | :lipstick: |
+| [vue/array-element-newline](./array-element-newline.md) | Enforce line breaks after each array element in `<template>` | :wrench: | :lipstick: |
 | [vue/arrow-spacing](./arrow-spacing.md) | Enforce consistent spacing before and after the arrow in arrow functions in `<template>` | :wrench: | :lipstick: |
 | [vue/block-spacing](./block-spacing.md) | Disallow or enforce spaces inside of blocks after opening block and before closing block in `<template>` | :wrench: | :lipstick: |
 | [vue/brace-style](./brace-style.md) | Enforce consistent brace style for blocks in `<template>` | :wrench: | :lipstick: |
diff --git a/lib/configs/no-layout-rules.js b/lib/configs/no-layout-rules.js
index 58e62b601..b1486f157 100644
--- a/lib/configs/no-layout-rules.js
+++ b/lib/configs/no-layout-rules.js
@@ -7,6 +7,7 @@ module.exports = {
   rules: {
     'vue/array-bracket-newline': 'off',
     'vue/array-bracket-spacing': 'off',
+    'vue/array-element-newline': 'off',
     'vue/arrow-spacing': 'off',
     'vue/block-spacing': 'off',
     'vue/block-tag-newline': 'off',
diff --git a/lib/index.js b/lib/index.js
index 2d14cd404..f5da9a706 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -9,6 +9,7 @@ module.exports = {
   rules: {
     'array-bracket-newline': require('./rules/array-bracket-newline'),
     'array-bracket-spacing': require('./rules/array-bracket-spacing'),
+    'array-element-newline': require('./rules/array-element-newline'),
     'arrow-spacing': require('./rules/arrow-spacing'),
     'attribute-hyphenation': require('./rules/attribute-hyphenation'),
     'attributes-order': require('./rules/attributes-order'),
diff --git a/lib/rules/array-element-newline.js b/lib/rules/array-element-newline.js
new file mode 100644
index 000000000..251d72de9
--- /dev/null
+++ b/lib/rules/array-element-newline.js
@@ -0,0 +1,11 @@
+/**
+ * @author alshyra
+ */
+'use strict'
+
+const { wrapCoreRule } = require('../utils')
+
+// eslint-disable-next-line no-invalid-meta, no-invalid-meta-docs-categories
+module.exports = wrapCoreRule('array-element-newline', {
+  skipDynamicArguments: true
+})
diff --git a/tests/lib/rules/array-element-newline.js b/tests/lib/rules/array-element-newline.js
new file mode 100644
index 000000000..be9b3b082
--- /dev/null
+++ b/tests/lib/rules/array-element-newline.js
@@ -0,0 +1,110 @@
+/**
+ * @author alshyra
+ * See LICENSE file in root directory for full license.
+ */
+'use strict'
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../lib/rules/array-element-newline')
+
+const tester = new RuleTester({
+  parser: require.resolve('vue-eslint-parser'),
+  parserOptions: {
+    ecmaVersion: 2020,
+    sourceType: 'module'
+  }
+})
+
+tester.run('array-element-newline', rule, {
+  valid: [
+    '<template><div :attr="[]" /></template>',
+    '<template><div :attr="[a]" /></template>',
+    '<template><div :attr="[a,\nb,\nc]" /></template>',
+    '<template><div :attr="[a,\nb,\nc\n]" /></template>',
+    '<template><div :attr="[\na,\nb,\nc\n]" /></template>',
+    '<template><div :[attr]="a" /></template>',
+    '<template><div :[[attr]]="a" /></template>',
+    '<template><div :attr="[a\nb]" /></template>',
+    {
+      code: '<template><div :attr="[a,\nb,\nc]" /></template>',
+      options: ['always']
+    },
+    {
+      code: '<template><div :attr="[a]" /></template>',
+      options: ['never']
+    },
+    {
+      code: '<template><div :attr="[a,b,c]" /></template>',
+      options: ['never']
+    },
+    {
+      code: '<template><div :attr="[a, b, c]" /></template>',
+      options: [{ multiline: true }]
+    },
+    {
+      code: '<template><div :attr="[a, \n{\nb:c\n}]" /></template>',
+      options: [{ multiline: true }]
+    },
+    {
+      code: '<template><div :attr="[a, b, c]" /></template>',
+      options: ['consistent']
+    },
+    {
+      code: '<template><div :attr="[a,\n b,\n c\n]" /></template>',
+      options: ['consistent']
+    },
+    {
+      code: '<template><div :attr="[a,b]" /></template>',
+      options: [{ minItems: 3 }]
+    }
+  ],
+  invalid: [
+    {
+      code: '<template><div :attr="[a, b]" /></template>',
+      output: '<template><div :attr="[a,\nb]" /></template>',
+      errors: ['There should be a linebreak after this element.']
+    },
+    {
+      code: '<template><div :attr="[a,b,c]" /></template>',
+      output: '<template><div :attr="[a,\nb,\nc]" /></template>',
+      options: ['always'],
+      errors: [
+        'There should be a linebreak after this element.',
+        'There should be a linebreak after this element.'
+      ]
+    },
+    {
+      code: '<template><div :attr="[a,\nb,c]" /></template>',
+      output: '<template><div :attr="[a,\nb,\nc]" /></template>',
+      options: ['always'],
+      errors: ['There should be a linebreak after this element.']
+    },
+    {
+      code: '<template><div :attr="[a,\nb,c]" /></template>',
+      output: '<template><div :attr="[a,\nb,\nc]" /></template>',
+      options: ['consistent'],
+      errors: ['There should be a linebreak after this element.']
+    },
+    {
+      code: '<template><div :attr="[a,\nb, c]" /></template>',
+      output: '<template><div :attr="[a, b, c]" /></template>',
+      options: [{ multiline: true }],
+      errors: ['There should be no linebreak here.']
+    },
+    {
+      code: '<template><div :attr="[a,{\nb:c}]" /></template>',
+      output: '<template><div :attr="[a,\n{\nb:c}]" /></template>',
+      options: [{ multiline: true }],
+      errors: ['There should be a linebreak after this element.']
+    },
+    {
+      code: '<template><div :attr="[a,b,c]" /></template>',
+      output: '<template><div :attr="[a,\nb,\nc]" /></template>',
+      options: [{ minItems: 2 }],
+      errors: [
+        'There should be a linebreak after this element.',
+        'There should be a linebreak after this element.'
+      ]
+    }
+  ]
+})
\ No newline at end of file

From bdb62f76c93cb59ece7730599b71c2e15aefe7bb Mon Sep 17 00:00:00 2001
From: Antoine SAVAJOLS <asavajols@digitaleo.com>
Date: Thu, 22 Dec 2022 09:42:28 +0100
Subject: [PATCH 2/3] fixed lint: missing linebreak

---
 tests/lib/rules/array-element-newline.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/lib/rules/array-element-newline.js b/tests/lib/rules/array-element-newline.js
index be9b3b082..1514c2481 100644
--- a/tests/lib/rules/array-element-newline.js
+++ b/tests/lib/rules/array-element-newline.js
@@ -107,4 +107,4 @@ tester.run('array-element-newline', rule, {
       ]
     }
   ]
-})
\ No newline at end of file
+})

From 888e933736a8069859c97fd4835720d43e5d1da5 Mon Sep 17 00:00:00 2001
From: Antoine SAVAJOLS <asavajols@digitaleo.com>
Date: Thu, 22 Dec 2022 10:40:20 +0100
Subject: [PATCH 3/3] Changed tests to error objects and use template string
 for readability

---
 tests/lib/rules/array-element-newline.js | 220 +++++++++++++++++++----
 1 file changed, 190 insertions(+), 30 deletions(-)

diff --git a/tests/lib/rules/array-element-newline.js b/tests/lib/rules/array-element-newline.js
index 1514c2481..54e60f83d 100644
--- a/tests/lib/rules/array-element-newline.js
+++ b/tests/lib/rules/array-element-newline.js
@@ -19,14 +19,39 @@ tester.run('array-element-newline', rule, {
   valid: [
     '<template><div :attr="[]" /></template>',
     '<template><div :attr="[a]" /></template>',
-    '<template><div :attr="[a,\nb,\nc]" /></template>',
-    '<template><div :attr="[a,\nb,\nc\n]" /></template>',
-    '<template><div :attr="[\na,\nb,\nc\n]" /></template>',
+    `
+      <template>
+        <div :attr="[a,
+                     b,
+                     c]" />
+      </template>`,
+    `<template>
+      <div :attr="[a,
+                   b,
+                   c
+                  ]" />
+    </template>`,
     '<template><div :[attr]="a" /></template>',
     '<template><div :[[attr]]="a" /></template>',
-    '<template><div :attr="[a\nb]" /></template>',
+    `<template>
+      <div :attr="[
+                   a,
+                   b,
+                   c
+                  ]" />
+    </template>`,
+    `
+      <template>
+        <div :attr="[a
+                     b]" />
+      </template>`,
     {
-      code: '<template><div :attr="[a,\nb,\nc]" /></template>',
+      code: `
+        <template>
+          <div :attr="[a,
+                       b,
+                       c]" />
+        </template>`,
       options: ['always']
     },
     {
@@ -42,7 +67,13 @@ tester.run('array-element-newline', rule, {
       options: [{ multiline: true }]
     },
     {
-      code: '<template><div :attr="[a, \n{\nb:c\n}]" /></template>',
+      code: `
+        <template>
+          <div :attr="[a, 
+                      {
+                        b:c
+                      }]" />
+        </template>`,
       options: [{ multiline: true }]
     },
     {
@@ -50,7 +81,13 @@ tester.run('array-element-newline', rule, {
       options: ['consistent']
     },
     {
-      code: '<template><div :attr="[a,\n b,\n c\n]" /></template>',
+      code: `
+        <template>
+          <div :attr="[a,
+                       b,
+                       c
+                      ]" />
+        </template>`,
       options: ['consistent']
     },
     {
@@ -60,50 +97,173 @@ tester.run('array-element-newline', rule, {
   ],
   invalid: [
     {
-      code: '<template><div :attr="[a, b]" /></template>',
-      output: '<template><div :attr="[a,\nb]" /></template>',
-      errors: ['There should be a linebreak after this element.']
+      code: `
+        <template>
+          <div :attr="[a, b]" />
+        </template>`,
+      output: `
+        <template>
+          <div :attr="[a,
+b]" />
+        </template>`,
+      errors: [
+        {
+          message: 'There should be a linebreak after this element.',
+          line: 3,
+          column: 26,
+          endLine: 3,
+          endColumn: 27
+        }
+      ]
     },
     {
-      code: '<template><div :attr="[a,b,c]" /></template>',
-      output: '<template><div :attr="[a,\nb,\nc]" /></template>',
+      code: `
+        <template>
+          <div :attr="[a,b,c]" />
+        </template>`,
+      output: `
+        <template>
+          <div :attr="[a,
+b,
+c]" />
+        </template>`,
       options: ['always'],
       errors: [
-        'There should be a linebreak after this element.',
-        'There should be a linebreak after this element.'
+        {
+          message: 'There should be a linebreak after this element.',
+          line: 3,
+          column: 26,
+          endLine: 3,
+          endColumn: 26
+        },
+        {
+          message: 'There should be a linebreak after this element.',
+          line: 3,
+          column: 28,
+          endLine: 3,
+          endColumn: 28
+        }
       ]
     },
     {
-      code: '<template><div :attr="[a,\nb,c]" /></template>',
-      output: '<template><div :attr="[a,\nb,\nc]" /></template>',
+      code: `
+        <template>
+          <div :attr="[a,
+                      b,c]" />
+        </template>`,
+      output: `
+        <template>
+          <div :attr="[a,
+                      b,
+c]" />
+        </template>`,
       options: ['always'],
-      errors: ['There should be a linebreak after this element.']
+      errors: [
+        {
+          message: 'There should be a linebreak after this element.',
+          line: 4,
+          column: 25,
+          endLine: 4,
+          endColumn: 25
+        }
+      ]
     },
     {
-      code: '<template><div :attr="[a,\nb,c]" /></template>',
-      output: '<template><div :attr="[a,\nb,\nc]" /></template>',
+      code: `
+        <template>
+          <div :attr="[a,
+                       b,c]" />
+        </template>`,
+      output: `
+        <template>
+          <div :attr="[a,
+                       b,
+c]" />
+        </template>`,
       options: ['consistent'],
-      errors: ['There should be a linebreak after this element.']
+      errors: [
+        {
+          message: 'There should be a linebreak after this element.',
+          line: 4,
+          column: 26,
+          endLine: 4,
+          endColumn: 26
+        }
+      ]
     },
     {
-      code: '<template><div :attr="[a,\nb, c]" /></template>',
-      output: '<template><div :attr="[a, b, c]" /></template>',
+      code: `
+        <template>
+          <div :attr="[a,
+                       b, c]" />
+        </template>`,
+      output: `
+        <template>
+          <div :attr="[a, b, c]" />
+        </template>`,
       options: [{ multiline: true }],
-      errors: ['There should be no linebreak here.']
+      errors: [
+        {
+          message: 'There should be no linebreak here.',
+          line: 3,
+          column: 26,
+          endLine: 4,
+          endColumn: 24
+        }
+      ]
     },
     {
-      code: '<template><div :attr="[a,{\nb:c}]" /></template>',
-      output: '<template><div :attr="[a,\n{\nb:c}]" /></template>',
+      code: `
+        <template>
+          <div :attr="[a, {
+                            b:c
+                          }]" />
+        </template>`,
+      output: `
+        <template>
+          <div :attr="[a,
+{
+                            b:c
+                          }]" />
+        </template>`,
       options: [{ multiline: true }],
-      errors: ['There should be a linebreak after this element.']
+      errors: [
+        {
+          message: 'There should be a linebreak after this element.',
+          line: 3,
+          column: 26,
+          endLine: 3,
+          endColumn: 27
+        }
+      ]
     },
     {
-      code: '<template><div :attr="[a,b,c]" /></template>',
-      output: '<template><div :attr="[a,\nb,\nc]" /></template>',
+      code: `
+        <template>
+          <div :attr="[a,b,c]" />
+        </template>`,
+      output: `
+        <template>
+          <div :attr="[a,
+b,
+c]" />
+        </template>`,
       options: [{ minItems: 2 }],
       errors: [
-        'There should be a linebreak after this element.',
-        'There should be a linebreak after this element.'
+        {
+          message: 'There should be a linebreak after this element.',
+          line: 3,
+          column: 26,
+          endLine: 3,
+          endColumn: 26
+        },
+        {
+          message: 'There should be a linebreak after this element.',
+          line: 3,
+          column: 28,
+          endLine: 3,
+          endColumn: 28
+        }
       ]
     }
   ]