+ content
+
+
+
+
+
+
+
+ content
+
+ content
+
+ content
+
+ one
two
`.
-- The root element has `v-for` directives. E.g. `{{x}}
`.
-- The root element is `` or `hello
- hello
-`
-```
-
-```html
+hello
hello
```
+abc
'
-```
-
-```html
-
- abc
-
-```
-
-```js
-template: 'abc
'
-```
-
-```html
+abc
-
-```
-
-```js
-template: `
- abc
- def
-`
-```
-
-```html
-
- abc
- def
+
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
```
+
-
-
-```
+::: warning Note
+This rule does not check syntax errors in directives because it's checked by [no-parsing-error] rule.
+:::
## :wrench: Options
@@ -51,5 +60,9 @@ Nothing.
- [no-parsing-error]
-
[no-parsing-error]: no-parsing-error.md
+
+## :mag: Implementation
+
+- [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/valid-v-model.js)
+- [Test source](https://github.com/vuejs/eslint-plugin-vue/blob/master/tests/lib/rules/valid-v-model.js)
diff --git a/docs/rules/valid-v-on.md b/docs/rules/valid-v-on.md
index 18580ccc0..e9eaa5a86 100644
--- a/docs/rules/valid-v-on.md
+++ b/docs/rules/valid-v-on.md
@@ -1,4 +1,11 @@
-# enforce valid `v-on` directives (vue/valid-v-on)
+---
+pageClass: rule-details
+sidebarDepth: 0
+title: vue/valid-v-on
+description: enforce valid `v-on` directives
+---
+# vue/valid-v-on
+> enforce valid `v-on` directives
- :gear: This rule is included in all of `"plugin:vue/essential"`, `"plugin:vue/strongly-recommended"` and `"plugin:vue/recommended"`.
@@ -12,35 +19,62 @@ This rule reports `v-on` directives in the following cases:
- The directive has invalid modifiers. E.g. ``
- The directive does not have that attribute value and any verb modifiers. E.g. ``
+
+
+
+```
+
+If you want to disallow `eslint-disable` functionality in ``, disable [vue/comment-directive](../rules/comment-directive.md) rule.
+
+## :computer: Editor integrations
+
+#### Visual Studio Code
+
+Use [dbaeumer.vscode-eslint](https://marketplace.visualstudio.com/items?itemName=dbaeumer.vscode-eslint) extension that Microsoft provides officially.
+
+You have to configure the `eslint.validate` option of the extension to check `.vue` files because the extension targets only `*.js` or `*.jsx` files by default.
+
+Example **.vscode/settings.json**:
+
+```json
+{
+ "eslint.validate": [
+ "javascript",
+ "javascriptreact",
+ { "language": "vue", "autoFix": true }
+ ]
+}
+```
+
+If you use `Vetur` plugin, set `"vetur.validation.template": false` to avoid default Vetur template validation. Check out [vetur documentation](https://vuejs.github.io/vetur/linting-error.html) for more info.
+
+#### Sublime Text
+
+Use [SublimeLinter-eslint](https://github.com/SublimeLinter/SublimeLinter-eslint) extension that SublimeLinter provides for eslint.
+
+You have to open command-palette via `cmd/ctrl+shift+p` and type `Preferences: SublimeLinter Settings`, paste to the config on the right side:
+
+```json
+{
+ "linters": {
+ "eslint": {
+ "selector": "text.html.vue, source.js - meta.attribute-with-value"
+ }
+ }
+}
+```
+
+#### Atom editor
+
+You need to go into `Settings -> Packages -> linter-eslint`, under the option "List of scopes to run eslint on", add `text.html.vue`. You may need to restart Atom.
+
+#### IntelliJ IDEA / JetBrains WebStorm
+
+In the **Settings/Preferences** dialog (`Ctrl+Alt+S`), choose JavaScript under **Languages and Frameworks** and then choose **ESLint** under **Code Quality Tools**.
+On the **ESLint page** that opens, select the *Enable* checkbox.
+
+If your ESLint configuration is updated (manually or from your version control), open it in the editor and choose **Apply ESLint Code Style Rules** on the context menu.
+
+read more: [JetBrains - ESLint](https://www.jetbrains.com/help/idea/eslint.html)
+
+## :question: FAQ
+
+### What is the "Use the latest vue-eslint-parser" error?
+
+The most rules of `eslint-plugin-vue` require `vue-eslint-parser` to check `` ASTs.
+
+Make sure you have one of the following settings in your **.eslintrc**:
+
+- `"extends": ["plugin:vue/recommended"]`
+- `"extends": ["plugin:vue/base"]`
+
+If you already use other parser (e.g. `"parser": "babel-eslint"`), please move it into `parserOptions`, so it doesn't collide with the `vue-eslint-parser` used by this plugin's configuration:
+
+```diff
+- "parser": "babel-eslint",
+ "parserOptions": {
++ "parser": "babel-eslint",
+ "ecmaVersion": 2017,
+ "sourceType": "module"
+ }
+```
+
+See also: "[Use together with custom parsers](#use-together-with-custom-parsers)" section.
+
+### Why doesn't it work on .vue file?
+
+1. Make sure you don't have `eslint-plugin-html` in your config. The `eslint-plugin-html` extracts the content from `
\ No newline at end of file
diff --git a/tests/fixtures/script-indent/array-expression-04.vue b/tests/fixtures/script-indent/array-expression-04.vue
new file mode 100644
index 000000000..1530c24b5
--- /dev/null
+++ b/tests/fixtures/script-indent/array-expression-04.vue
@@ -0,0 +1,6 @@
+
+
\ No newline at end of file
diff --git a/tests/fixtures/script-indent/if-statement-04.js b/tests/fixtures/script-indent/if-statement-04.js
deleted file mode 100644
index 41831505e..000000000
--- a/tests/fixtures/script-indent/if-statement-04.js
+++ /dev/null
@@ -1,4 +0,0 @@
-/*{}*/
-if (a)
- b;
-c
diff --git a/tests/fixtures/script-indent/if-statement-04.vue b/tests/fixtures/script-indent/if-statement-04.vue
new file mode 100644
index 000000000..e6ba11eb4
--- /dev/null
+++ b/tests/fixtures/script-indent/if-statement-04.vue
@@ -0,0 +1,7 @@
+
+
+
diff --git a/tests/fixtures/script-indent/if-statement-05.js b/tests/fixtures/script-indent/if-statement-05.js
deleted file mode 100644
index 4e9af67f4..000000000
--- a/tests/fixtures/script-indent/if-statement-05.js
+++ /dev/null
@@ -1,6 +0,0 @@
-/*{}*/
-function wrap() {
- if (x)
- this.doSomething();
- foo
-}
diff --git a/tests/fixtures/script-indent/indent-valid-fixture-01.vue b/tests/fixtures/script-indent/indent-valid-fixture-01.vue
new file mode 100644
index 000000000..c1423bf9b
--- /dev/null
+++ b/tests/fixtures/script-indent/indent-valid-fixture-01.vue
@@ -0,0 +1,541 @@
+
+
+
\ No newline at end of file
diff --git a/tests/fixtures/script-indent/issue441.vue b/tests/fixtures/script-indent/issue441.vue
new file mode 100644
index 000000000..a6738f2a3
--- /dev/null
+++ b/tests/fixtures/script-indent/issue441.vue
@@ -0,0 +1,10 @@
+
+
\ No newline at end of file
diff --git a/tests/fixtures/script-indent/issue443.vue b/tests/fixtures/script-indent/issue443.vue
new file mode 100644
index 000000000..4a8c356e7
--- /dev/null
+++ b/tests/fixtures/script-indent/issue443.vue
@@ -0,0 +1,27 @@
+
+
\ No newline at end of file
diff --git a/tests/fixtures/script-indent/issue625.vue b/tests/fixtures/script-indent/issue625.vue
new file mode 100644
index 000000000..6c205ee83
--- /dev/null
+++ b/tests/fixtures/script-indent/issue625.vue
@@ -0,0 +1,12 @@
+
+
diff --git a/tests/fixtures/script-indent/opts-baseindent1.js b/tests/fixtures/script-indent/opts-baseindent1.js
deleted file mode 100644
index e5456c834..000000000
--- a/tests/fixtures/script-indent/opts-baseindent1.js
+++ /dev/null
@@ -1,16 +0,0 @@
-/*{"options":[2,{"baseIndent":1}]}*/
-var aaa = {
- test: 1
-}
-var bbb = {
- test: 1
- },
- ccc = {
- test: 1
- }
-const ddd = {
- test: 1
- },
- eee = (a) => {
- foo(a)
- }
diff --git a/tests/fixtures/script-indent/sequence-expression-04.vue b/tests/fixtures/script-indent/sequence-expression-04.vue
new file mode 100644
index 000000000..f263bf523
--- /dev/null
+++ b/tests/fixtures/script-indent/sequence-expression-04.vue
@@ -0,0 +1,8 @@
+
+
diff --git a/tests/lib/autofix.js b/tests/lib/autofix.js
new file mode 100644
index 000000000..5a38dd0e4
--- /dev/null
+++ b/tests/lib/autofix.js
@@ -0,0 +1,160 @@
+/**
+ * @author Yosuke Ota
@@ -185,7 +189,8 @@ tester.run('attributes-order', rule, {
'RENDER_MODIFIERS',
'GLOBAL',
'UNIQUE',
- 'BINDING',
+ 'TWO_WAY_BINDING',
+ 'OTHER_DIRECTIVES',
'OTHER_ATTR',
'EVENTS',
'CONTENT',
@@ -202,8 +207,9 @@ tester.run('attributes-order', rule, {
'RENDER_MODIFIERS',
'GLOBAL',
'UNIQUE',
- 'BINDING',
+ 'TWO_WAY_BINDING',
'DEFINITION',
+ 'OTHER_DIRECTIVES',
'OTHER_ATTR',
'EVENTS',
'CONTENT']
@@ -220,9 +226,9 @@ tester.run('attributes-order', rule, {
is="header"
v-on:click="functionCall"
ref="header"
- :prop="headerData"
v-text="textContent"
id="uniqueID"
+ :prop="headerData"
myProp="prop"
>
@@ -236,10 +242,38 @@ tester.run('attributes-order', rule, {
'DEFINITION',
'EVENTS',
'UNIQUE',
- 'BINDING',
+ 'TWO_WAY_BINDING',
'CONTENT',
'GLOBAL',
- 'OTHER_ATTR'
+ 'OTHER_ATTR',
+ 'OTHER_DIRECTIVES'
+ ]
+ }]
+ },
+ {
+ filename: 'test.vue',
+ code:
+ `
+
+
+ `,
+ options: [
+ { order:
+ [
+ 'CONDITIONALS',
+ 'LIST_RENDERING',
+ 'RENDER_MODIFIERS',
+ 'DEFINITION',
+ 'EVENTS',
+ 'UNIQUE',
+ ['BINDING', 'OTHER_ATTR'],
+ 'CONTENT',
+ 'GLOBAL'
]
}]
}
@@ -272,7 +306,7 @@ tester.run('attributes-order', rule, {
model="baz"
v-model="toggle"
propOne="bar"
- :bindingProp="foo">
+ :id="foo">
`,
output:
@@ -280,7 +314,7 @@ tester.run('attributes-order', rule, {
+
+ `,
+ options: [
+ { order:
+ [
+ 'CONDITIONALS',
+ 'LIST_RENDERING',
+ 'RENDER_MODIFIERS',
+ 'DEFINITION',
+ 'EVENTS',
+ 'UNIQUE',
+ ['BINDING', 'OTHER_ATTR'],
+ 'CONTENT',
+ 'GLOBAL'
+ ]
+ }],
+ output:
+ `
+
+
+ `,
+ errors: [
+ {
+ message: 'Attribute "v-if" should go before "class".',
+ nodeType: 'VIdentifier'
+ }
+ ]
}
]
})
diff --git a/tests/lib/rules/component-name-in-template-casing.js b/tests/lib/rules/component-name-in-template-casing.js
new file mode 100644
index 000000000..8dcd3937f
--- /dev/null
+++ b/tests/lib/rules/component-name-in-template-casing.js
@@ -0,0 +1,359 @@
+/**
+ * @author Yosuke Ota
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const rule = require('../../../lib/rules/component-name-in-template-casing')
+const RuleTester = require('eslint').RuleTester
+
+// ------------------------------------------------------------------------------
+// Tests
+// ------------------------------------------------------------------------------
+
+const tester = new RuleTester({
+ parser: 'vue-eslint-parser'
+})
+
+tester.run('component-name-in-template-casing', rule, {
+ valid: [
+ // default
+ '',
+ 'Title
', + 'Title
', + '', + '
+ id=""
+ >
`,
@@ -144,20 +145,20 @@ tester.run('html-closing-bracket-newline', rule, {
code: `
+ id="">
`,
output: `
+ id=""
+>
`,
errors: [
- 'Expected no line breaks before closing bracket, but 1 line break found.'
+ 'Expected 1 line break before closing bracket, but no line breaks found.'
]
},
{
diff --git a/tests/lib/rules/html-indent.js b/tests/lib/rules/html-indent.js
index b2f7ebe0b..724ee961f 100644
--- a/tests/lib/rules/html-indent.js
+++ b/tests/lib/rules/html-indent.js
@@ -108,114 +108,151 @@ tester.run('html-indent', rule, loadPatterns(
// Valid
[
// TemplateLiteral
- unIndent`
-
-
-
- `,
+ test
+ \`
+ "
+ >
+
-
- `,
- unIndent`
-
-
-
-
- `,
- unIndent`
-
-
-
- `,
- unIndent`
-
-
-
-
- `,
+ * comment
+ */
+ }}
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: unIndent`
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: unIndent`
+
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: unIndent`
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: unIndent`
+
+
+
+
+ `
+ },
// Ignores
{
+ filename: 'test.vue',
code: unIndent`
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: unIndent`
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: unIndent`
+
+
@@ -243,6 +281,106 @@ tester.run('html-indent', rule, loadPatterns(
`
+ },
+ {
+ filename: 'test.vue',
+ code: unIndent`
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: unIndent`
+
+ + aaa + bbb + ccc ++ + ` + }, + { + filename: 'test.vue', + code: unIndent` + +
aaa + bbb ccc + ddd + fff+ + ` + }, + { + filename: 'test.vue', + code: unIndent` + +
aaa + bbb ccc + ddd + fff+ + ` + }, + { + filename: 'test.vue', + code: unIndent` + +
aaa + bbb ccc + ddd + fff
+ + ++ + ` + }, + { + filename: 'test.vue', + code: unIndent` + +
+ text + text + text ++
+
+
+ + text+
+ text+
+ text+ + ` } ], @@ -250,6 +388,7 @@ tester.run('html-indent', rule, loadPatterns( [ // TemplateLiteral { + filename: 'test.vue', code: unIndent`
@@ -303,6 +443,7 @@ tester.run('html-indent', rule, loadPatterns(
]
},
{
+ filename: 'test.vue',
code: unIndent`
\t
@@ -325,6 +466,7 @@ tester.run('html-indent', rule, loadPatterns(
// Comments
{
+ filename: 'test.vue',
code: unIndent`
@@ -355,6 +497,7 @@ tester.run('html-indent', rule, loadPatterns(
]
},
{
+ filename: 'test.vue',
code: unIndent`
{{
@@ -381,6 +524,7 @@ tester.run('html-indent', rule, loadPatterns(
]
},
{
+ filename: 'test.vue',
code: unIndent`
{{
@@ -410,6 +554,7 @@ tester.run('html-indent', rule, loadPatterns(
]
},
{
+ filename: 'test.vue',
code: unIndent`
{{
@@ -440,6 +585,7 @@ tester.run('html-indent', rule, loadPatterns(
// Ignores
{
+ filename: 'test.vue',
code: unIndent`
{{
@@ -495,8 +642,9 @@ tester.run('html-indent', rule, loadPatterns(
]
},
- // Pre
+ // Pre, Textarea
{
+ filename: 'test.vue',
code: unIndent`
`,
- output: `
@@ -116,7 +152,7 @@ job="Vet"
{
code: `
@@ -164,7 +200,7 @@ name="John Doe"
@@ -183,7 +219,7 @@ age="30"
`,
options: [{ singleline: 3, multiline: 1 }],
output: `
@@ -203,7 +239,7 @@ age="30"
options: [{ singleline: 3, multiline: { max: 2, allowFirstLine: false }}],
output: `
`,
@@ -222,7 +258,7 @@ petname="Snoopy">
options: [{ singleline: 3, multiline: { max: 2, allowFirstLine: false }}],
output: `
`,
diff --git a/tests/lib/rules/multiline-html-element-content-newline.js b/tests/lib/rules/multiline-html-element-content-newline.js
new file mode 100644
index 000000000..b1eaa64c4
--- /dev/null
+++ b/tests/lib/rules/multiline-html-element-content-newline.js
@@ -0,0 +1,545 @@
+/**
+ * @author Yosuke Ota
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const rule = require('../../../lib/rules/multiline-html-element-content-newline')
+const RuleTester = require('eslint').RuleTester
+
+// ------------------------------------------------------------------------------
+// Tests
+// ------------------------------------------------------------------------------
+
+const tester = new RuleTester({
+ parser: 'vue-eslint-parser',
+ parserOptions: {
+ ecmaVersion: 2015
+ }
+})
+
+tester.run('multiline-html-element-content-newline', rule, {
+ valid: [
+ `content
+ content
+ content
+ content
+ `,
+ options: [{
+ ignores: ['ignore-tag']
+ }]
+ },
+ {
+ code: `
+
+ content
+ content
+ content
+ content
+ `,
+ options: [{
+ ignores: ['IgnoreTag']
+ }]
+ },
+ {
+ code: `
+
+ content
+ content
+ content
+ content
+ `,
+ options: [{
+ ignores: ['IgnoreTag']
+ }]
+ },
+ // Ignore if no closing brackets
+ `
+
+
++ + `, + errors: [ + { message: 'Expected indentation of 2 spaces but found 0 spaces.', line: 2 }, + { message: 'Expected indentation of 4 spaces but found 0 spaces.', line: 3 }, + { message: 'Expected indentation of 2 spaces but found 0 spaces.', line: 4 } + ] + }, + { + filename: 'test.vue', + code: unIndent` + +
+ aaa + bbb + ccc ++ + `, + output: unIndent` + +
+ aaa + bbb + ccc ++ + `, + errors: [ + { message: 'Expected indentation of 2 spaces but found 0 spaces.', line: 2 }, + { message: 'Expected indentation of 4 spaces but found 0 spaces.', line: 3 }, + { message: 'Expected indentation of 6 spaces but found 0 spaces.', line: 4 }, + { message: 'Expected indentation of 6 spaces but found 0 spaces.', line: 5 }, + { message: 'Expected indentation of 6 spaces but found 0 spaces.', line: 6 }, + { message: 'Expected indentation of 4 spaces but found 0 spaces.', line: 7 }, + { message: 'Expected indentation of 2 spaces but found 0 spaces.', line: 8 } + ] + }, + { + filename: 'test.vue', + code: unIndent` + + + + `, + output: unIndent` + + `, errors: [ { message: 'Expected indentation of 2 spaces but found 0 spaces.', line: 2 }, { message: 'Expected indentation of 4 spaces but found 0 spaces.', line: 3 }, - { message: 'Expected indentation of 2 spaces but found 0 spaces.', line: 4 }, + { message: 'Expected indentation of 6 spaces but found 0 spaces.', line: 4 }, + { message: 'Expected indentation of 6 spaces but found 0 spaces.', line: 5 }, + { message: 'Expected indentation of 6 spaces but found 0 spaces.', line: 6 }, + { message: 'Expected indentation of 4 spaces but found 0 spaces.', line: 7 }, { message: 'Expected indentation of 2 spaces but found 0 spaces.', line: 8 } ] } diff --git a/tests/lib/rules/max-attributes-per-line.js b/tests/lib/rules/max-attributes-per-line.js index 0e8ab915f..ce4d45ffe 100644 --- a/tests/lib/rules/max-attributes-per-line.js +++ b/tests/lib/rules/max-attributes-per-line.js @@ -91,17 +91,53 @@ ruleTester.run('max-attributes-per-line', rule, { invalid: [ { code: `
content
`,
+ ``,
+ ``,
+ `
+
+
+ content
+
+ `,
+ `
+
+
+ content
+
+ `,
+ `
+
+
+
+ `,
+ `
+ content
+ content
+
+ multiline end tag
`,
+ // empty
+ ``,
+ `
+
+
+
+ `,
+ // self closing
+ `
+
+ content+
content+
+content
content + content+ + + + `, + { + code: ` + +
content
content
content
+
content
+
+ `,
+ output: `
+
+
+content
+
+
+ `,
+ errors: [
+ {
+ message: 'Expected 1 line break after opening tag (``), but no line breaks found.',
+ line: 5,
+ column: 12,
+ nodeType: 'HTMLTagClose',
+ endLine: 5,
+ endColumn: 12
+ },
+ {
+ message: 'Expected 1 line break before closing tag (`
`), but no line breaks found.',
+ line: 5,
+ column: 19,
+ nodeType: 'HTMLEndTagOpen',
+ endLine: 5,
+ endColumn: 19
+ }
+ ]
+ },
+ // spaces
+ {
+ code: `
+
+ content
+
+ `,
+ output: `
+
+
+content
+
+
+ `,
+ errors: [
+ {
+ message: 'Expected 1 line break after opening tag (``), but no line breaks found.',
+ line: 5,
+ column: 12,
+ endLine: 5,
+ endColumn: 15
+ },
+ {
+ message: 'Expected 1 line break before closing tag (`
`), but no line breaks found.',
+ line: 5,
+ column: 22,
+ endLine: 5,
+ endColumn: 22
+ }
+ ]
+ },
+ // elements
+ {
+ code: `
+
+
+
+
+ `,
+ output: `
+
+
+
+
+
+
+ `,
+ errors: [
+ {
+ message: 'Expected 1 line break after opening tag (``), but no line breaks found.',
+ line: 3,
+ column: 16,
+ endLine: 3,
+ endColumn: 16
+ },
+ {
+ message: 'Expected 1 line break before closing tag (`
`), but no line breaks found.',
+ line: 4,
+ column: 22,
+ endLine: 4,
+ endColumn: 22
+ }
+ ]
+ },
+ // contents
+ {
+ code: `
+
+ multiline
+ content
+ `,
+ output: `
+
+
+multiline
+ content
+
+ `,
+ errors: [
+ 'Expected 1 line break after opening tag (``), but no line breaks found.',
+ 'Expected 1 line break before closing tag (`
`), but no line breaks found.'
+ ]
+ },
+ {
+ code: `
+
+ multiline content
+
+ `,
+ output: `
+
+
+multiline content
+
+ `,
+ errors: [
+ 'Expected 1 line break after opening tag (``), but no line breaks found.'
+ ]
+ },
+ {
+ code: `
+
+
`), but no line breaks found.'
+ ]
+ },
+ // comments
+ {
+ code: `
+
+
+ multiline content
+ `,
+ output: `
+
+
+ multiline content
+
+ `,
+ errors: [
+ 'Expected 1 line break before closing tag (`
+
+
+ `,
+ output: `
+
+
+
+
+
+
+ `,
+ errors: [
+ 'Expected 1 line break after opening tag (``), but no line breaks found.',
+ 'Expected 1 line break before closing tag (`
`), but no line breaks found.'
+ ]
+ },
+ {
+ code: `
+
+
+
+ `,
+ output: `
+
+
+
+
+
+ `,
+ errors: [
+ 'Expected 1 line break after opening tag (``), but no line breaks found.',
+ 'Expected 1 line break before closing tag (`
`), but no line breaks found.'
+ ]
+ },
+ // one error
+ {
+ code: `
+
+ content
+ content
+
+
+ `,
+ output: `
+
+
+content
+ content
+
+
+ `,
+ errors: [
+ 'Expected 1 line break after opening tag (``), but no line breaks found.'
+ ]
+ },
+ {
+ code: `
+
+
`), but no line breaks found.'
+ ]
+ },
+ // multi
+ {
+ code: `
+
+ content
+ content
+
+ `,
+ output: `
+
+
+ content
+ content
+
+
+ `,
+ errors: [
+ 'Expected 1 line break before closing tag (`content
+ `,
+ output: `
+
+content
+ content
content
+content
+
+ `,
+ errors: [
+ 'Expected 1 line break after opening tag (``), but no line breaks found.',
+ 'Expected 1 line break after opening tag (`
+content
+ content
+
content
+`), but no line breaks found.',
+ 'Expected 1 line break after opening tag (`
`), but no line breaks found.',
+ 'Expected 1 line break before closing tag (``), but no line breaks found.'
+ ]
+ },
+ // multi line breaks
+ {
+ code: `
+
+ `), but no line breaks found.',
+ 'Expected 1 line break before closing tag (`
`), but no line breaks found.',
+ 'Expected 1 line break before closing tag (`
+
+ content
+ content
+
+
+
+ `,
+ output: `
+
+
+content
+ content
+
+
+ `,
+ errors: [
+ 'Expected 1 line break after opening tag (``), but 2 line breaks found.',
+ 'Expected 1 line break before closing tag (`
`), but 2 line breaks found.'
+ ]
+ },
+ // mustache
+ {
+ code: `
+
+ {{content}}
+ {{content2}}
+
+ `,
+ output: `
+
+
+{{content}}
+ {{content2}}
+
+
+ `,
+ errors: [
+ 'Expected 1 line break after opening tag (``), but no line breaks found.',
+ 'Expected 1 line break before closing tag (`
`), but no line breaks found.'
+ ]
+ },
+ // mix
+ {
+ code: `
+
+ content
+
+
+ `,
+ output: `
+
+
+content
+
+
+ `,
+ errors: [
+ 'Expected 1 line break after opening tag (``), but no line breaks found.',
+ 'Expected 1 line break before closing tag (`
`), but no line breaks found.'
+ ]
+ },
+ // start tag
+ {
+ code: `
+
+ content
+
+ `,
+ output: `
+
+
+content
+
+
+ `,
+ errors: [
+ 'Expected 1 line break after opening tag (``), but no line breaks found.',
+ 'Expected 1 line break before closing tag (`
`), but no line breaks found.'
+ ]
+ },
+ {
+ code: `
+
+ content
+
+ `,
+ output: `
+
+
+content
+
+
+ `,
+ errors: [
+ 'Expected 1 line break after opening tag (``), but no line breaks found.',
+ 'Expected 1 line break before closing tag (`
`), but no line breaks found.'
+ ]
+ },
+ {
+ code: `
+
+
+
+ `,
+ output: `
+
+
+
+
+ `,
+ options: [{ ignoreWhenEmpty: false }],
+ errors: ['Expected 1 line break after opening tag (``), but no line breaks found.']
+ }
+ ]
+})
diff --git a/tests/lib/rules/name-property-casing.js b/tests/lib/rules/name-property-casing.js
index d5d833352..1bdeb8769 100644
--- a/tests/lib/rules/name-property-casing.js
+++ b/tests/lib/rules/name-property-casing.js
@@ -16,9 +16,8 @@ const RuleTester = require('eslint').RuleTester
// ------------------------------------------------------------------------------
const parserOptions = {
- ecmaVersion: 6,
- sourceType: 'module',
- ecmaFeatures: { experimentalObjectRestSpread: true }
+ ecmaVersion: 2018,
+ sourceType: 'module'
}
const ruleTester = new RuleTester()
diff --git a/tests/lib/rules/no-async-in-computed-properties.js b/tests/lib/rules/no-async-in-computed-properties.js
index 2e7c27882..23f59965b 100644
--- a/tests/lib/rules/no-async-in-computed-properties.js
+++ b/tests/lib/rules/no-async-in-computed-properties.js
@@ -12,15 +12,8 @@ const rule = require('../../../lib/rules/no-async-in-computed-properties')
const RuleTester = require('eslint').RuleTester
const parserOptions = {
- ecmaVersion: 6,
- sourceType: 'module',
- ecmaFeatures: { experimentalObjectRestSpread: true }
-}
-
-const parserOptions8 = {
- ecmaVersion: 8,
- sourceType: 'module',
- ecmaFeatures: { experimentalObjectRestSpread: true }
+ ecmaVersion: 2018,
+ sourceType: 'module'
}
// ------------------------------------------------------------------------------
@@ -77,7 +70,7 @@ ruleTester.run('no-async-in-computed-properties', rule, {
}));
}
`,
- parserOptions: parserOptions8
+ parserOptions
}
],
@@ -93,7 +86,7 @@ ruleTester.run('no-async-in-computed-properties', rule, {
}
}
`,
- parserOptions: parserOptions8,
+ parserOptions,
errors: [{
message: 'Unexpected async function declaration in "foo" computed property.',
line: 4
@@ -113,7 +106,7 @@ ruleTester.run('no-async-in-computed-properties', rule, {
}
}
`,
- parserOptions: parserOptions8,
+ parserOptions,
errors: [{
message: 'Unexpected async function declaration in "foo" computed property.',
line: 4
diff --git a/tests/lib/rules/no-dupe-keys.js b/tests/lib/rules/no-dupe-keys.js
index f9031d1e6..1abb67642 100644
--- a/tests/lib/rules/no-dupe-keys.js
+++ b/tests/lib/rules/no-dupe-keys.js
@@ -79,7 +79,7 @@ ruleTester.run('no-dupe-keys', rule, {
},
}
`,
- parserOptions: { ecmaVersion: 8, sourceType: 'module', ecmaFeatures: { experimentalObjectRestSpread: true }}
+ parserOptions: { ecmaVersion: 2018, sourceType: 'module' }
},
{
diff --git a/tests/lib/rules/no-multi-spaces.js b/tests/lib/rules/no-multi-spaces.js
index 5238db724..68433af44 100644
--- a/tests/lib/rules/no-multi-spaces.js
+++ b/tests/lib/rules/no-multi-spaces.js
@@ -44,6 +44,36 @@ ruleTester.run('no-multi-spaces', rule, {
{
filename: 'test.js',
code: 'export default { }'
+ },
+ {
+ code: `
+
+
+
+ `,
+ options: [{
+ ignoreProperties: true
+ }]
+ },
+ {
+ code: `
+
+
+
+ `,
+ options: [{
+ ignoreProperties: true
+ }]
}
],
invalid: [
@@ -176,6 +206,62 @@ ruleTester.run('no-multi-spaces', rule, {
type: 'Punctuator'
}
]
+ },
+ {
+ code: `
+
+
+
+ `,
+ output: `
+
+
+
+ `,
+ errors: [
+ {
+ message: "Multiple spaces found before ':'.",
+ type: 'Punctuator'
+ }
+ ]
+ },
+ {
+ code: `
+
+
+
+ `,
+ output: `
+
+
+
+ `,
+ errors: [
+ {
+ message: "Multiple spaces found before 'isExpanded'.",
+ type: 'Identifier'
+ }
+ ]
}
]
})
diff --git a/tests/lib/rules/no-parsing-error.js b/tests/lib/rules/no-parsing-error.js
index fcb4c351a..365bbb09e 100644
--- a/tests/lib/rules/no-parsing-error.js
+++ b/tests/lib/rules/no-parsing-error.js
@@ -207,7 +207,8 @@ tester.run('no-parsing-error', rule, {
options: [{ 'x-invalid-namespace': false }]
},
'',
- ').extend({
+ props: {
+ a: {
+ type: String,
+ required: true
+ } as PropOptions
+ }
+ });
`,
- parserOptions
+ parser: 'typescript-eslint-parser'
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default Vue.extend({
+ props: {
+ a: {
+ type: String,
+ required: true
+ } as PropOptions
+ }
+ });
+ `,
+ parser: 'typescript-eslint-parser'
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default {
+ props: {
+ bar,
+ baz: prop,
+ bar1: foo()
+ }
+ }
+ `
}
],
@@ -118,11 +170,14 @@ ruleTester.run('require-default-prop', rule, {
d: {
type: Number,
required: false
+ },
+ e: [Boolean, String],
+ f: {
+ type: [Boolean, String],
}
}
}
`,
- parserOptions,
errors: [{
message: `Prop 'a' requires default value to be set.`,
line: 4
@@ -135,6 +190,46 @@ ruleTester.run('require-default-prop', rule, {
}, {
message: `Prop 'd' requires default value to be set.`,
line: 9
+ }, {
+ message: `Prop 'e' requires default value to be set.`,
+ line: 13
+ }, {
+ message: `Prop 'f' requires default value to be set.`,
+ line: 14
+ }]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default (Vue as VueConstructor).extend({
+ props: {
+ a: {
+ type: String
+ } as PropOptions
+ }
+ });
+ `,
+ parser: 'typescript-eslint-parser',
+ errors: [{
+ message: `Prop 'a' requires default value to be set.`,
+ line: 4
+ }]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default Vue.extend({
+ props: {
+ a: {
+ type: String
+ } as PropOptions
+ }
+ });
+ `,
+ parser: 'typescript-eslint-parser',
+ errors: [{
+ message: `Prop 'a' requires default value to be set.`,
+ line: 4
}]
}
]
diff --git a/tests/lib/rules/require-prop-type-constructor.js b/tests/lib/rules/require-prop-type-constructor.js
new file mode 100644
index 000000000..fd86930d7
--- /dev/null
+++ b/tests/lib/rules/require-prop-type-constructor.js
@@ -0,0 +1,170 @@
+/**
+ * @fileoverview require prop type to be a constructor
+ * @author Michał Sajnóg
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const rule = require('../../../lib/rules/require-prop-type-constructor')
+const RuleTester = require('eslint').RuleTester
+
+// ------------------------------------------------------------------------------
+// Tests
+// ------------------------------------------------------------------------------
+
+const ruleTester = new RuleTester({
+ parserOptions: {
+ ecmaVersion: 2018,
+ sourceType: 'module'
+ }
+})
+ruleTester.run('require-prop-type-constructor', rule, {
+
+ valid: [
+ {
+ filename: 'SomeComponent.vue',
+ code: `
+ export default {
+ props: {
+ ...props,
+ myProp: Number,
+ anotherType: [Number, String],
+ extraProp: {
+ type: Number,
+ default: 10
+ },
+ lastProp: {
+ type: [Number, Boolean]
+ },
+ nullProp: null,
+ nullTypeProp: { type: null }
+ }
+ }
+ `
+ }
+ ],
+
+ invalid: [
+ {
+ filename: 'SomeComponent.vue',
+ code: `
+ export default {
+ props: {
+ myProp: 'Number',
+ anotherType: ['Number', 'String'],
+ extraProp: {
+ type: 'Number',
+ default: 10
+ },
+ lastProp: {
+ type: ['Boolean']
+ },
+ nullProp: 'null'
+ }
+ }
+ `,
+ output: `
+ export default {
+ props: {
+ myProp: Number,
+ anotherType: [Number, String],
+ extraProp: {
+ type: Number,
+ default: 10
+ },
+ lastProp: {
+ type: [Boolean]
+ },
+ nullProp: null
+ }
+ }
+ `,
+ errors: [{
+ message: 'The "myProp" property should be a constructor.',
+ line: 4
+ }, {
+ message: 'The "anotherType" property should be a constructor.',
+ line: 5
+ }, {
+ message: 'The "anotherType" property should be a constructor.',
+ line: 5
+ }, {
+ message: 'The "extraProp" property should be a constructor.',
+ line: 7
+ }, {
+ message: 'The "lastProp" property should be a constructor.',
+ line: 11
+ }, {
+ message: 'The "nullProp" property should be a constructor.',
+ line: 13
+ }]
+ },
+ {
+ filename: 'SomeComponent.vue',
+ code: `
+ export default {
+ props: {
+ a: \`String\`,
+ b: Foo + '',
+ c: 1,
+ d: true,
+ }
+ }
+ `,
+ output: `
+ export default {
+ props: {
+ a: String,
+ b: Foo + '',
+ c: 1,
+ d: true,
+ }
+ }
+ `,
+ errors: [{
+ message: 'The "a" property should be a constructor.',
+ line: 4
+ }, {
+ message: 'The "b" property should be a constructor.',
+ line: 5
+ }, {
+ message: 'The "c" property should be a constructor.',
+ line: 6
+ }, {
+ message: 'The "d" property should be a constructor.',
+ line: 7
+ }]
+ },
+ {
+ filename: 'SomeComponent.vue',
+ code: `
+ export default {
+ props: {
+ a: {
+ type: 'String',
+ default: 10
+ } as PropOptions,
+ }
+ }
+ `,
+ output: `
+ export default {
+ props: {
+ a: {
+ type: String,
+ default: 10
+ } as PropOptions,
+ }
+ }
+ `,
+ errors: [{
+ message: 'The "a" property should be a constructor.',
+ line: 5
+ }],
+ parser: 'typescript-eslint-parser'
+ }
+ ]
+})
diff --git a/tests/lib/rules/require-prop-types.js b/tests/lib/rules/require-prop-types.js
index 683e8a5bb..aac856342 100644
--- a/tests/lib/rules/require-prop-types.js
+++ b/tests/lib/rules/require-prop-types.js
@@ -31,7 +31,7 @@ ruleTester.run('require-prop-types', rule, {
}
}
`,
- parserOptions: { ecmaVersion: 6, sourceType: 'module', ecmaFeatures: { experimentalObjectRestSpread: true }}
+ parserOptions: { ecmaVersion: 2018, sourceType: 'module' }
},
{
filename: 'test.vue',
@@ -113,6 +113,52 @@ ruleTester.run('require-prop-types', rule, {
}
`,
parserOptions: { ecmaVersion: 6, sourceType: 'module' }
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default {
+ props: []
+ }
+ `,
+ parserOptions: { ecmaVersion: 6, sourceType: 'module' }
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default {
+ props: {}
+ }
+ `,
+ parserOptions: { ecmaVersion: 6, sourceType: 'module' }
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default (Vue as VueConstructor).extend({
+ props: {
+ foo: {
+ type: String
+ } as PropOptions
+ }
+ });
+ `,
+ parserOptions: { ecmaVersion: 6, sourceType: 'module' },
+ parser: 'typescript-eslint-parser'
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default Vue.extend({
+ props: {
+ foo: {
+ type: String
+ } as PropOptions
+ }
+ });
+ `,
+ parserOptions: { ecmaVersion: 6, sourceType: 'module' },
+ parser: 'typescript-eslint-parser'
}
],
@@ -121,12 +167,21 @@ ruleTester.run('require-prop-types', rule, {
filename: 'test.vue',
code: `
export default {
- props: ['foo']
+ props: ['foo', bar, \`baz\`, foo()]
}
`,
parserOptions: { ecmaVersion: 6, sourceType: 'module' },
errors: [{
- message: 'Props should at least define their types.',
+ message: 'Prop "foo" should define at least its type.',
+ line: 3
+ }, {
+ message: 'Prop "bar" should define at least its type.',
+ line: 3
+ }, {
+ message: 'Prop "baz" should define at least its type.',
+ line: 3
+ }, {
+ message: 'Prop "Unknown prop" should define at least its type.',
line: 3
}]
},
@@ -134,12 +189,21 @@ ruleTester.run('require-prop-types', rule, {
filename: 'test.js',
code: `
new Vue({
- props: ['foo']
+ props: ['foo', bar, \`baz\`, foo()]
})
`,
parserOptions: { ecmaVersion: 6, sourceType: 'module' },
errors: [{
- message: 'Props should at least define their types.',
+ message: 'Prop "foo" should define at least its type.',
+ line: 3
+ }, {
+ message: 'Prop "bar" should define at least its type.',
+ line: 3
+ }, {
+ message: 'Prop "baz" should define at least its type.',
+ line: 3
+ }, {
+ message: 'Prop "Unknown prop" should define at least its type.',
line: 3
}]
},
@@ -190,6 +254,38 @@ ruleTester.run('require-prop-types', rule, {
message: 'Prop "foo" should define at least its type.',
line: 4
}]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default Vue.extend({
+ props: {
+ foo: {} as PropOptions
+ }
+ });
+ `,
+ parserOptions: { ecmaVersion: 6, sourceType: 'module' },
+ parser: 'typescript-eslint-parser',
+ errors: [{
+ message: 'Prop "foo" should define at least its type.',
+ line: 4
+ }]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default (Vue as VueConstructor).extend({
+ props: {
+ foo: {} as PropOptions
+ }
+ });
+ `,
+ parserOptions: { ecmaVersion: 6, sourceType: 'module' },
+ parser: 'typescript-eslint-parser',
+ errors: [{
+ message: 'Prop "foo" should define at least its type.',
+ line: 4
+ }]
}
]
})
diff --git a/tests/lib/rules/require-render-return.js b/tests/lib/rules/require-render-return.js
index c12910ff7..e7fde4e60 100644
--- a/tests/lib/rules/require-render-return.js
+++ b/tests/lib/rules/require-render-return.js
@@ -12,9 +12,9 @@ const rule = require('../../../lib/rules/require-render-return')
const RuleTester = require('eslint').RuleTester
const parserOptions = {
- ecmaVersion: 6,
+ ecmaVersion: 2018,
sourceType: 'module',
- ecmaFeatures: { experimentalObjectRestSpread: true, jsx: true }
+ ecmaFeatures: { jsx: true }
}
// ------------------------------------------------------------------------------
@@ -70,6 +70,36 @@ ruleTester.run('require-render-return', rule, {
}`,
parserOptions
},
+ {
+ filename: 'test.vue',
+ code: `export default {
+ render() {
+ const foo = function () {}
+ return foo
+ }
+ }`,
+ parserOptions
+ },
+ {
+ filename: 'test.vue',
+ code: `export default {
+ render() {
+ if (a) {
+ if (b) {
+
+ }
+ if (c) {
+ return true
+ } else {
+ return foo
+ }
+ } else {
+ return foo
+ }
+ }
+ }`,
+ parserOptions
+ },
{
filename: 'test.vue',
code: `export default {
@@ -119,6 +149,22 @@ ruleTester.run('require-render-return', rule, {
line: 2
}]
},
+ {
+ filename: 'test.vue',
+ code: `export default {
+ render: function () {
+ if (foo) {
+ return h('div', 'hello')
+ }
+ }
+ }`,
+ parserOptions,
+ errors: [{
+ message: 'Expected to return a value in render function.',
+ type: 'Identifier',
+ line: 2
+ }]
+ },
{
code: `Vue.component('test', {
render: function () {
@@ -133,6 +179,38 @@ ruleTester.run('require-render-return', rule, {
type: 'Identifier',
line: 2
}]
+ },
+ {
+ code: `Vue.component('test2', {
+ render: function () {
+ if (a) {
+ return h('div', 'hello')
+ }
+ }
+ })`,
+ parserOptions,
+ errors: [{
+ message: 'Expected to return a value in render function.',
+ type: 'Identifier',
+ line: 2
+ }]
+ },
+ {
+ code: `Vue.component('test2', {
+ render: function () {
+ if (a) {
+
+ } else {
+ return h('div', 'hello')
+ }
+ }
+ })`,
+ parserOptions,
+ errors: [{
+ message: 'Expected to return a value in render function.',
+ type: 'Identifier',
+ line: 2
+ }]
}
]
})
diff --git a/tests/lib/rules/require-valid-default-prop.js b/tests/lib/rules/require-valid-default-prop.js
index b8794fbfb..4d434f2a8 100644
--- a/tests/lib/rules/require-valid-default-prop.js
+++ b/tests/lib/rules/require-valid-default-prop.js
@@ -12,9 +12,9 @@ const rule = require('../../../lib/rules/require-valid-default-prop')
const RuleTester = require('eslint').RuleTester
const parserOptions = {
- ecmaVersion: 6,
+ ecmaVersion: 2018,
sourceType: 'module',
- ecmaFeatures: { experimentalObjectRestSpread: true, jsx: true }
+ ecmaFeatures: { jsx: true }
}
function errorMessage (type) {
@@ -100,6 +100,21 @@ ruleTester.run('require-valid-default-prop', rule, {
}
})`,
parserOptions
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default (Vue as VueConstructor).extend({
+ props: {
+ foo: {
+ type: [Object, Number],
+ default: 10
+ } as PropOptions
hello
'
+ 'hello
',
+ '{{ }}
'
],
invalid: [
{
@@ -230,13 +231,6 @@ tester.run('no-parsing-error', rule, {
code: 'hello
',
errors: ['Parsing error: Unexpected token ;.']
},
- {
- filename: 'test.vue',
- code: '{{ }}
',
- errors: [
- { message: 'Parsing error: Expected to be an expression, but got empty.', column: 18 }
- ]
- },
{
filename: 'test.vue',
code: 'hello
',
diff --git a/tests/lib/rules/no-reserved-keys.js b/tests/lib/rules/no-reserved-keys.js
index cc116f1dc..08c7f244c 100644
--- a/tests/lib/rules/no-reserved-keys.js
+++ b/tests/lib/rules/no-reserved-keys.js
@@ -12,9 +12,8 @@ const rule = require('../../../lib/rules/no-reserved-keys')
const RuleTester = require('eslint').RuleTester
const parserOptions = {
- ecmaVersion: 7,
- sourceType: 'module',
- ecmaFeatures: { experimentalObjectRestSpread: true }
+ ecmaVersion: 2018,
+ sourceType: 'module'
}
// ------------------------------------------------------------------------------
diff --git a/tests/lib/rules/no-shared-component-data.js b/tests/lib/rules/no-shared-component-data.js
index d1616dd37..ce0e0062b 100644
--- a/tests/lib/rules/no-shared-component-data.js
+++ b/tests/lib/rules/no-shared-component-data.js
@@ -13,9 +13,8 @@ const rule = require('../../../lib/rules/no-shared-component-data')
const RuleTester = require('eslint').RuleTester
const parserOptions = {
- ecmaVersion: 7,
- sourceType: 'module',
- ecmaFeatures: { experimentalObjectRestSpread: true }
+ ecmaVersion: 2018,
+ sourceType: 'module'
}
// ------------------------------------------------------------------------------
diff --git a/tests/lib/rules/no-side-effects-in-computed-properties.js b/tests/lib/rules/no-side-effects-in-computed-properties.js
index 3eadc45c0..7367bc87a 100644
--- a/tests/lib/rules/no-side-effects-in-computed-properties.js
+++ b/tests/lib/rules/no-side-effects-in-computed-properties.js
@@ -12,9 +12,8 @@ const rule = require('../../../lib/rules/no-side-effects-in-computed-properties'
const RuleTester = require('eslint').RuleTester
const parserOptions = {
- ecmaVersion: 6,
- sourceType: 'module',
- ecmaFeatures: { experimentalObjectRestSpread: true }
+ ecmaVersion: 2018,
+ sourceType: 'module'
}
// ------------------------------------------------------------------------------
@@ -249,6 +248,24 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
line: 23,
message: 'Unexpected side effect in "test4" computed property.'
}]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default Vue.extend({
+ computed: {
+ test1() : string {
+ return this.something.reverse()
+ }
+ }
+ });
+ `,
+ parserOptions,
+ errors: [{
+ line: 5,
+ message: 'Unexpected side effect in "test1" computed property.'
+ }],
+ parser: 'typescript-eslint-parser'
}
]
})
diff --git a/tests/lib/rules/no-spaces-around-equal-signs-in-attribute.js b/tests/lib/rules/no-spaces-around-equal-signs-in-attribute.js
new file mode 100644
index 000000000..7bd090f3e
--- /dev/null
+++ b/tests/lib/rules/no-spaces-around-equal-signs-in-attribute.js
@@ -0,0 +1,234 @@
+/**
+ * @author Yosuke Ota
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const rule = require('../../../lib/rules/no-spaces-around-equal-signs-in-attribute')
+const RuleTester = require('eslint').RuleTester
+
+// ------------------------------------------------------------------------------
+// Tests
+// ------------------------------------------------------------------------------
+
+const tester = new RuleTester({
+ parser: 'vue-eslint-parser'
+})
+
+tester.run('no-spaces-around-equal-signs-in-attribute', rule, {
+ valid: [
+ '',
+ '',
+ '',
+ '',
+ '',
+ '',
+ `
+
+
+ `
+ ],
+ invalid: [
+ {
+ code: '',
+ output: '',
+ errors: [
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 1,
+ column: 20,
+ endLine: 1,
+ endColumn: 23
+ }
+ ]
+ },
+ {
+ code: '',
+ output: '',
+ errors: [
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 1,
+ column: 20,
+ endLine: 1,
+ endColumn: 23
+ }
+ ]
+ },
+ {
+ code: '',
+ output: '',
+ errors: [
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 1,
+ column: 20,
+ endLine: 1,
+ endColumn: 23
+ }
+ ]
+ },
+ {
+ code: '',
+ output: '',
+ errors: [
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 1,
+ column: 20,
+ endLine: 1,
+ endColumn: 23
+ }
+ ]
+ },
+ {
+ code: '',
+ output: '',
+ errors: [
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 1,
+ column: 20,
+ endLine: 3,
+ endColumn: 2
+ }
+ ]
+ },
+ {
+ code: '',
+ output: '',
+ errors: [
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 1,
+ column: 20,
+ endLine: 1,
+ endColumn: 22
+ }
+ ]
+ },
+ {
+ code: '',
+ output: '',
+ errors: [
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 1,
+ column: 20,
+ endLine: 1,
+ endColumn: 22
+ }
+ ]
+ },
+ {
+ code:
+ `
+
+
+ `,
+ output:
+ `
+
+
+ `,
+ errors: [
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 3,
+ column: 15,
+ endLine: 3,
+ endColumn: 18
+ },
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 4,
+ column: 18,
+ endLine: 4,
+ endColumn: 21
+ },
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 5,
+ column: 17,
+ endLine: 5,
+ endColumn: 20
+ },
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 7,
+ column: 15,
+ endLine: 7,
+ endColumn: 18
+ },
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 8,
+ column: 16,
+ endLine: 8,
+ endColumn: 19
+ },
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 9,
+ column: 20,
+ endLine: 9,
+ endColumn: 23
+ },
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 10,
+ column: 19,
+ endLine: 10,
+ endColumn: 22
+ },
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 11,
+ column: 19,
+ endLine: 11,
+ endColumn: 22
+ },
+ {
+ message: 'Unexpected spaces found around equal signs.',
+ line: 12,
+ column: 19,
+ endLine: 12,
+ endColumn: 22
+ }
+ ]
+ }
+ ]
+})
diff --git a/tests/lib/rules/no-template-shadow.js b/tests/lib/rules/no-template-shadow.js
new file mode 100644
index 000000000..c52f4ddb4
--- /dev/null
+++ b/tests/lib/rules/no-template-shadow.js
@@ -0,0 +1,213 @@
+/**
+ * @fileoverview Disallow variable declarations from shadowing variables declared in the outer scope.
+ * @author Armano
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const rule = require('../../../lib/rules/no-template-shadow')
+const RuleTester = require('eslint').RuleTester
+
+// ------------------------------------------------------------------------------
+// Tests
+// ------------------------------------------------------------------------------
+
+const ruleTester = new RuleTester({
+ parser: 'vue-eslint-parser',
+ parserOptions: {
+ ecmaVersion: 2018,
+ sourceType: 'module'
+ }
+})
+
+ruleTester.run('no-template-shadow', rule, {
+
+ valid: [
+ '',
+ '',
+ '',
+ '',
+ '',
+ '
- {{i}},{{j}}
',
+ errors: [{
+ message: "Variable 'i' is already declared in the upper scope.",
+ type: 'Identifier'
+ }]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+ `,
+ errors: [{
+ message: "Variable 'i' is already declared in the upper scope.",
+ type: 'Identifier',
+ line: 2
+ }]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `,
+ errors: [{
+ message: "Variable 'i' is already declared in the upper scope.",
+ type: 'Identifier',
+ line: 2
+ }, {
+ message: "Variable 'i' is already declared in the upper scope.",
+ type: 'Identifier',
+ line: 3
+ }]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+ `,
+ errors: [{
+ message: "Variable 'i' is already declared in the upper scope.",
+ type: 'Identifier',
+ line: 2
+ }, {
+ message: "Variable 'i' is already declared in the upper scope.",
+ type: 'Identifier',
+ line: 3
+ }]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `,
+ errors: [{
+ message: "Variable 'i' is already declared in the upper scope.",
+ type: 'Identifier',
+ line: 2
+ }, {
+ message: "Variable 'f' is already declared in the upper scope.",
+ type: 'Identifier',
+ line: 3
+ }]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+
+
+
+ `,
+ errors: [{
+ message: "Variable 'e' is already declared in the upper scope.",
+ type: 'Identifier',
+ line: 6
+ }, {
+ message: "Variable 'f' is already declared in the upper scope.",
+ type: 'Identifier',
+ line: 7
+ }]
+ }
+ ]
+})
diff --git a/tests/lib/rules/no-unused-components.js b/tests/lib/rules/no-unused-components.js
new file mode 100644
index 000000000..f62b83fb1
--- /dev/null
+++ b/tests/lib/rules/no-unused-components.js
@@ -0,0 +1,517 @@
+/**
+ * @fileoverview Report used components
+ * @author Michał Sajnóg
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const RuleTester = require('eslint').RuleTester
+const rule = require('../../../lib/rules/no-unused-components')
+
+// ------------------------------------------------------------------------------
+// Tests
+// ------------------------------------------------------------------------------
+
+const tester = new RuleTester({
+ parser: 'vue-eslint-parser',
+ parserOptions: {
+ ecmaVersion: 2018,
+ sourceType: 'module'
+ }
+})
+
+tester.run('no-unused-components', rule, {
+ valid: [
+ {
+ filename: 'test.vue',
+ code: `Lorem ipsum
`
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ Lorem ipsum
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ Lorem ipsum
+
+
+
+ `
+ },
+
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+
+
+
+ `,
+ options: [{ ignoreWhenBindingPresent: true }]
+ },
+
+ // Ignore when `render` is used instead of template
+ {
+ filename: 'test.vue',
+ code: `
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+ `
+ }
+ ],
+ invalid: [
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+
+ `,
+ errors: [{
+ message: 'The "TheButton" component has been registered but not used.',
+ line: 10
+ }]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+ Lorem ipsum
+
+
+
+
+ `,
+ errors: [{
+ message: 'The "TheButton" component has been registered but not used.',
+ line: 11
+ }]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+ Lorem ipsum
+
+
+
+
+ `,
+ errors: [{
+ message: 'The "the-button" component has been registered but not used.',
+ line: 11
+ }]
+ },
+ // Setting: ignoreWhenBindingPresent
+ {
+ filename: 'test.vue',
+ code: `
+
+ Lorem ipsum
+
+
+
+ `,
+ options: [{ ignoreWhenBindingPresent: false }],
+ errors: [{
+ message: 'The "Foo" component has been registered but not used.',
+ line: 13
+ }, {
+ message: 'The "Bar" component has been registered but not used.',
+ line: 14
+ }]
+ }
+ ]
+})
diff --git a/tests/lib/rules/no-unused-vars.js b/tests/lib/rules/no-unused-vars.js
index db67041e1..e033d4871 100644
--- a/tests/lib/rules/no-unused-vars.js
+++ b/tests/lib/rules/no-unused-vars.js
@@ -46,6 +46,9 @@ tester.run('no-unused-vars', rule, {
},
{
code: '{{c}}
'
+ },
+ {
+ code: '{{value | f(x)}}
'
}
],
invalid: [
@@ -80,6 +83,10 @@ tester.run('no-unused-vars', rule, {
{
code: '{{item.name}}
',
errors: ["'key' is defined but never used."]
+ },
+ {
+ code: '{{value | x}}
',
+ errors: ["'x' is defined but never used."]
}
]
})
diff --git a/tests/lib/rules/order-in-components.js b/tests/lib/rules/order-in-components.js
index 2eff407ec..b022508fa 100644
--- a/tests/lib/rules/order-in-components.js
+++ b/tests/lib/rules/order-in-components.js
@@ -10,9 +10,8 @@ const RuleTester = require('eslint').RuleTester
const ruleTester = new RuleTester()
const parserOptions = {
- ecmaVersion: 6,
- sourceType: 'module',
- ecmaFeatures: { experimentalObjectRestSpread: true }
+ ecmaVersion: 2018,
+ sourceType: 'module'
}
ruleTester.run('order-in-components', rule, {
@@ -375,7 +374,7 @@ ruleTester.run('order-in-components', rule, {
data() {
},
test: 'ok',
- name: 'burger',
+ name: 'burger'
};
`,
options: [{ order: ['data', 'test', 'name'] }],
@@ -402,7 +401,33 @@ ruleTester.run('order-in-components', rule, {
name: 'burger',
/** data provider */
data() {
- },
+ }
+ };
+ `,
+ errors: [{
+ message: 'The "name" property should be above the "data" property on line 4.',
+ line: 7
+ }]
+ },
+ {
+ filename: 'example.vue',
+ code: `
+ export default {
+ /** data provider */
+ data() {
+ }/*test*/,
+ /** name of vue component */
+ name: 'burger'
+ };
+ `,
+ parserOptions,
+ output: `
+ export default {
+ /** name of vue component */
+ name: 'burger',
+ /** data provider */
+ data() {
+ }/*test*/
};
`,
errors: [{
@@ -414,7 +439,7 @@ ruleTester.run('order-in-components', rule, {
filename: 'example.vue',
code: `export default {data(){},name:'burger'};`,
parserOptions,
- output: `export default {name:'burger',data(){},};`,
+ output: `export default {name:'burger',data(){}};`,
errors: [{
message: 'The "name" property should be above the "data" property on line 1.',
line: 1
diff --git a/tests/lib/rules/prop-name-casing.js b/tests/lib/rules/prop-name-casing.js
index cfeeaf8e8..842e07ab8 100644
--- a/tests/lib/rules/prop-name-casing.js
+++ b/tests/lib/rules/prop-name-casing.js
@@ -16,9 +16,8 @@ const RuleTester = require('eslint').RuleTester
// ------------------------------------------------------------------------------
const parserOptions = {
- ecmaVersion: 6,
- sourceType: 'module',
- ecmaFeatures: { experimentalObjectRestSpread: true }
+ ecmaVersion: 2018,
+ sourceType: 'module'
}
const ruleTester = new RuleTester()
@@ -68,7 +67,7 @@ ruleTester.run('prop-name-casing', rule, {
filename: 'test.vue',
code: `
export default {
- props: ['greetingText']
+ props: ['greeting_text']
}
`,
options: ['snake_case'],
@@ -339,6 +338,26 @@ ruleTester.run('prop-name-casing', rule, {
line: 4
}]
},
+ {
+ filename: 'test.vue',
+ code: `
+ export default {
+ props: ['greeting_text']
+ }
+ `,
+ options: ['camelCase'],
+ output: `
+ export default {
+ props: ['greetingText']
+ }
+ `,
+ parserOptions,
+ errors: [{
+ message: 'Prop "greeting_text" is not in camelCase.',
+ type: 'Literal',
+ line: 3
+ }]
+ },
{
filename: 'test.vue',
code: `
diff --git a/tests/lib/rules/require-default-prop.js b/tests/lib/rules/require-default-prop.js
index df5fc0afc..7eb689a3d 100644
--- a/tests/lib/rules/require-default-prop.js
+++ b/tests/lib/rules/require-default-prop.js
@@ -11,8 +11,7 @@
const rule = require('../../../lib/rules/require-default-prop')
const RuleTester = require('eslint').RuleTester
const parserOptions = {
- ecmaVersion: 6,
- ecmaFeatures: { experimentalObjectRestSpread: true },
+ ecmaVersion: 2018,
sourceType: 'module'
}
@@ -20,7 +19,7 @@ const parserOptions = {
// Tests
// ------------------------------------------------------------------------------
-const ruleTester = new RuleTester()
+const ruleTester = new RuleTester({ parserOptions })
ruleTester.run('require-default-prop', rule, {
valid: [
@@ -47,12 +46,27 @@ ruleTester.run('require-default-prop', rule, {
required: false,
'default': 'lorem'
},
+ e: {
+ type: Boolean
+ },
+ f: {
+ type: Boolean,
+ required: false
+ },
+ g: {
+ type: Boolean,
+ default: true
+ },
+ h: {
+ type: [Boolean]
+ },
+ i: Boolean,
+ j: [Boolean],
// eslint-disable-next-line require-default-prop
- e: Number
+ k: Number
}
}
- `,
- parserOptions
+ `
},
{
filename: 'test.vue',
@@ -71,8 +85,7 @@ ruleTester.run('require-default-prop', rule, {
}
}
}
- `,
- parserOptions
+ `
},
{
filename: 'test.vue',
@@ -99,8 +112,47 @@ ruleTester.run('require-default-prop', rule, {
}
}
}
+ `
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ export default (Vue as VueConstructor