',
- errors: ['Parsing error: x-invalid-namespace.']
+ errors: [
+ {
+ message: 'Parsing error: x-invalid-namespace.',
+ line: 1,
+ column: 16
+ }
+ ]
},
//style vars
diff --git a/tests/lib/rules/no-potential-component-option-typo.js b/tests/lib/rules/no-potential-component-option-typo.js
index 0b9fad541..63a6a42cf 100644
--- a/tests/lib/rules/no-potential-component-option-typo.js
+++ b/tests/lib/rules/no-potential-component-option-typo.js
@@ -183,6 +183,8 @@ tester.run('no-potential-component-option-typo', rule, {
message: "'dat' may be a typo, which is similar to option [data].",
line: 4,
column: 9,
+ endLine: 4,
+ endColumn: 12,
suggestions: [
{
desc: `Replace property 'dat' to 'data'`,
@@ -200,6 +202,8 @@ tester.run('no-potential-component-option-typo', rule, {
message: `'method' may be a typo, which is similar to option [methods].`,
line: 5,
column: 9,
+ endLine: 5,
+ endColumn: 15,
suggestions: [
{
desc: `Replace property 'method' to 'methods'`,
@@ -235,6 +239,8 @@ tester.run('no-potential-component-option-typo', rule, {
message: "'dat' may be a typo, which is similar to option [data].",
line: 4,
column: 9,
+ endLine: 4,
+ endColumn: 12,
suggestions: [
{
desc: `Replace property 'dat' to 'data'`,
@@ -253,6 +259,8 @@ tester.run('no-potential-component-option-typo', rule, {
message: `'method' may be a typo, which is similar to option [methods].`,
line: 5,
column: 9,
+ endLine: 5,
+ endColumn: 15,
suggestions: [
{
desc: `Replace property 'method' to 'methods'`,
@@ -271,6 +279,8 @@ tester.run('no-potential-component-option-typo', rule, {
message: `'custo' may be a typo, which is similar to option [custom].`,
line: 6,
column: 9,
+ endLine: 6,
+ endColumn: 14,
suggestions: [
{
desc: `Replace property 'custo' to 'custom'`,
@@ -303,6 +313,8 @@ tester.run('no-potential-component-option-typo', rule, {
message: "'dat' may be a typo, which is similar to option [data].",
line: 4,
column: 9,
+ endLine: 4,
+ endColumn: 12,
suggestions: [
{
desc: `Replace property 'dat' to 'data'`,
@@ -320,6 +332,8 @@ tester.run('no-potential-component-option-typo', rule, {
message: `'method' may be a typo, which is similar to option [methods].`,
line: 5,
column: 9,
+ endLine: 5,
+ endColumn: 15,
suggestions: [
{
desc: `Replace property 'method' to 'methods'`,
@@ -352,6 +366,8 @@ tester.run('no-potential-component-option-typo', rule, {
message: "'dat' may be a typo, which is similar to option [data].",
line: 4,
column: 9,
+ endLine: 4,
+ endColumn: 12,
suggestions: [
{
desc: `Replace property 'dat' to 'data'`,
@@ -371,6 +387,8 @@ tester.run('no-potential-component-option-typo', rule, {
"'beforeRouteEntr' may be a typo, which is similar to option [beforeRouteEnter].",
line: 5,
column: 9,
+ endLine: 5,
+ endColumn: 24,
suggestions: [
{
desc: `Replace property 'beforeRouteEntr' to 'beforeRouteEnter'`,
@@ -389,6 +407,8 @@ tester.run('no-potential-component-option-typo', rule, {
message: `'method' may be a typo, which is similar to option [methods].`,
line: 6,
column: 9,
+ endLine: 6,
+ endColumn: 15,
suggestions: [
{
desc: `Replace property 'method' to 'methods'`,
@@ -420,6 +440,8 @@ tester.run('no-potential-component-option-typo', rule, {
message: `'method' may be a typo, which is similar to option [methods,data].`,
line: 4,
column: 9,
+ endLine: 4,
+ endColumn: 15,
suggestions: [
{
desc: `Replace property 'method' to 'methods'`,
diff --git a/tests/lib/rules/no-ref-as-operand.js b/tests/lib/rules/no-ref-as-operand.js
index 9db2ab3a0..86cd7bd54 100644
--- a/tests/lib/rules/no-ref-as-operand.js
+++ b/tests/lib/rules/no-ref-as-operand.js
@@ -490,7 +490,10 @@ tester.run('no-ref-as-operand', rule, {
errors: [
{
messageId: 'requireDotValue',
- line: 4
+ line: 4,
+ column: 11,
+ endLine: 4,
+ endColumn: 14
}
]
},
@@ -512,7 +515,10 @@ tester.run('no-ref-as-operand', rule, {
errors: [
{
messageId: 'requireDotValue',
- line: 4
+ line: 4,
+ column: 15,
+ endLine: 4,
+ endColumn: 18
}
]
},
@@ -536,19 +542,31 @@ tester.run('no-ref-as-operand', rule, {
errors: [
{
messageId: 'requireDotValue',
- line: 4
+ line: 4,
+ column: 16,
+ endLine: 4,
+ endColumn: 19
},
{
messageId: 'requireDotValue',
- line: 5
+ line: 5,
+ column: 16,
+ endLine: 5,
+ endColumn: 19
},
{
messageId: 'requireDotValue',
- line: 6
+ line: 6,
+ column: 16,
+ endLine: 6,
+ endColumn: 19
},
{
messageId: 'requireDotValue',
- line: 7
+ line: 7,
+ column: 16,
+ endLine: 7,
+ endColumn: 19
}
]
},
@@ -572,19 +590,31 @@ tester.run('no-ref-as-operand', rule, {
errors: [
{
messageId: 'requireDotValue',
- line: 4
+ line: 4,
+ column: 7,
+ endLine: 4,
+ endColumn: 10
},
{
messageId: 'requireDotValue',
- line: 5
+ line: 5,
+ column: 7,
+ endLine: 5,
+ endColumn: 10
},
{
messageId: 'requireDotValue',
- line: 6
+ line: 6,
+ column: 14,
+ endLine: 6,
+ endColumn: 17
},
{
messageId: 'requireDotValue',
- line: 7
+ line: 7,
+ column: 14,
+ endLine: 7,
+ endColumn: 17
}
]
},
@@ -604,11 +634,17 @@ tester.run('no-ref-as-operand', rule, {
errors: [
{
messageId: 'requireDotValue',
- line: 4
+ line: 4,
+ column: 15,
+ endLine: 4,
+ endColumn: 18
},
{
messageId: 'requireDotValue',
- line: 5
+ line: 5,
+ column: 15,
+ endLine: 5,
+ endColumn: 18
}
]
},
@@ -626,7 +662,10 @@ tester.run('no-ref-as-operand', rule, {
errors: [
{
messageId: 'requireDotValue',
- line: 4
+ line: 4,
+ column: 15,
+ endLine: 4,
+ endColumn: 18
}
]
},
@@ -666,15 +705,24 @@ tester.run('no-ref-as-operand', rule, {
errors: [
{
messageId: 'requireDotValue',
- line: 7
+ line: 7,
+ column: 13,
+ endLine: 7,
+ endColumn: 18
},
{
messageId: 'requireDotValue',
- line: 8
+ line: 8,
+ column: 25,
+ endLine: 8,
+ endColumn: 30
},
{
messageId: 'requireDotValue',
- line: 9
+ line: 9,
+ column: 29,
+ endLine: 9,
+ endColumn: 34
}
]
},
@@ -763,27 +811,42 @@ tester.run('no-ref-as-operand', rule, {
{
message:
'Must use `.value` to read or write the value wrapped by `ref()`.',
- line: 33
+ line: 33,
+ column: 9,
+ endLine: 33,
+ endColumn: 14
},
{
message:
'Must use `.value` to read or write the value wrapped by `computed()`.',
- line: 34
+ line: 34,
+ column: 9,
+ endLine: 34,
+ endColumn: 15
},
{
message:
'Must use `.value` to read or write the value wrapped by `toRef()`.',
- line: 36
+ line: 36,
+ column: 22,
+ endLine: 36,
+ endColumn: 28
},
{
message:
'Must use `.value` to read or write the value wrapped by `customRef()`.',
- line: 36
+ line: 36,
+ column: 34,
+ endLine: 36,
+ endColumn: 38
},
{
message:
'Must use `.value` to read or write the value wrapped by `shallowRef()`.',
- line: 38
+ line: 38,
+ column: 19,
+ endLine: 38,
+ endColumn: 22
}
]
},
@@ -804,7 +867,11 @@ tester.run('no-ref-as-operand', rule, {
`,
errors: [
{
- messageId: 'requireDotValue'
+ messageId: 'requireDotValue',
+ line: 5,
+ column: 9,
+ endLine: 5,
+ endColumn: 12
}
]
},
@@ -825,7 +892,11 @@ tester.run('no-ref-as-operand', rule, {
`,
errors: [
{
- messageId: 'requireDotValue'
+ messageId: 'requireDotValue',
+ line: 5,
+ column: 21,
+ endLine: 5,
+ endColumn: 24
}
]
},
@@ -859,7 +930,9 @@ tester.run('no-ref-as-operand', rule, {
message:
'Must use `.value` to read or write the value wrapped by `ref()`.',
line: 10,
- column: 7
+ column: 7,
+ endLine: 10,
+ endColumn: 10
}
]
},
@@ -892,12 +965,18 @@ tester.run('no-ref-as-operand', rule, {
{
message:
'Must use `.value` to read or write the value wrapped by `defineModel()`.',
- line: 6
+ line: 6,
+ column: 13,
+ endLine: 6,
+ endColumn: 18
},
{
message:
'Must use `.value` to read or write the value wrapped by `defineModel()`.',
- line: 9
+ line: 9,
+ column: 9,
+ endLine: 9,
+ endColumn: 14
}
]
},
@@ -930,12 +1009,18 @@ tester.run('no-ref-as-operand', rule, {
{
message:
'Must use `.value` to read or write the value wrapped by `defineModel()`.',
- line: 6
+ line: 6,
+ column: 13,
+ endLine: 6,
+ endColumn: 18
},
{
message:
'Must use `.value` to read or write the value wrapped by `defineModel()`.',
- line: 9
+ line: 9,
+ column: 9,
+ endLine: 9,
+ endColumn: 14
}
]
},
@@ -967,7 +1052,9 @@ tester.run('no-ref-as-operand', rule, {
message:
'Must use `.value` to read or write the value wrapped by `ref()`.',
line: 8,
- endLine: 8
+ column: 23,
+ endLine: 8,
+ endColumn: 28
}
]
},
@@ -1013,7 +1100,9 @@ tester.run('no-ref-as-operand', rule, {
message:
'Must use `.value` to read or write the value wrapped by `ref()`.',
line: 10,
- endLine: 10
+ column: 35,
+ endLine: 10,
+ endColumn: 42
}
]
},
@@ -1059,7 +1148,9 @@ tester.run('no-ref-as-operand', rule, {
message:
'Must use `.value` to read or write the value wrapped by `ref()`.',
line: 10,
- endLine: 10
+ column: 31,
+ endLine: 10,
+ endColumn: 38
}
]
},
@@ -1105,7 +1196,9 @@ tester.run('no-ref-as-operand', rule, {
message:
'Must use `.value` to read or write the value wrapped by `ref()`.',
line: 10,
- endLine: 10
+ column: 38,
+ endLine: 10,
+ endColumn: 45
}
]
},
diff --git a/tests/lib/rules/no-reserved-component-names.js b/tests/lib/rules/no-reserved-component-names.js
index 809d35103..09d9a6f64 100644
--- a/tests/lib/rules/no-reserved-component-names.js
+++ b/tests/lib/rules/no-reserved-component-names.js
@@ -608,8 +608,10 @@ ruleTester.run('no-reserved-component-names', rule, {
? 'reservedInHtml'
: 'reserved',
data: { name },
- type: 'Literal',
- line: 3
+ line: 3,
+ column: 19,
+ endLine: 3,
+ endColumn: 19 + name.length + 2
}
]
})),
@@ -623,8 +625,10 @@ ruleTester.run('no-reserved-component-names', rule, {
? 'reservedInHtml'
: 'reserved',
data: { name },
- type: 'Literal',
- line: 1
+ line: 1,
+ column: 15,
+ endLine: 1,
+ endColumn: 15 + name.length + 2
}
]
})),
@@ -638,8 +642,10 @@ ruleTester.run('no-reserved-component-names', rule, {
? 'reservedInHtml'
: 'reserved',
data: { name },
- type: 'Literal',
- line: 1
+ line: 1,
+ column: 15,
+ endLine: 1,
+ endColumn: 15 + name.length + 2
}
]
})),
@@ -653,8 +659,10 @@ ruleTester.run('no-reserved-component-names', rule, {
? 'reservedInHtml'
: 'reserved',
data: { name },
- type: 'TemplateLiteral',
- line: 1
+ line: 1,
+ column: 15,
+ endLine: 1,
+ endColumn: 15 + name.length + 2
}
]
})),
@@ -668,8 +676,10 @@ ruleTester.run('no-reserved-component-names', rule, {
? 'reservedInHtml'
: 'reserved',
data: { name },
- type: 'TemplateLiteral',
- line: 1
+ line: 1,
+ column: 15,
+ endLine: 1,
+ endColumn: 15 + name.length + 2
}
]
})),
@@ -687,8 +697,10 @@ ruleTester.run('no-reserved-component-names', rule, {
? 'reservedInHtml'
: 'reserved',
data: { name },
- type: 'Property',
- line: 3
+ line: 3,
+ column: 13,
+ endLine: 3,
+ endColumn: 13 + name.length + 6
}
]
})),
@@ -705,7 +717,10 @@ ruleTester.run('no-reserved-component-names', rule, {
? 'reservedInHtml'
: 'reserved',
data: { name },
- line: 1
+ line: 1,
+ column: 37,
+ endLine: 1,
+ endColumn: 37 + name.length + 2
}
]
})),
@@ -723,7 +738,10 @@ ruleTester.run('no-reserved-component-names', rule, {
? 'reservedInHtml'
: 'reserved',
data: { name },
- line: 1
+ line: 1,
+ column: 37,
+ endLine: 1,
+ endColumn: 37 + name.length + 2
}
]
})),
@@ -740,8 +758,10 @@ ruleTester.run('no-reserved-component-names', rule, {
{
messageId: 'reservedInVue',
data: { name },
- type: 'Literal',
- line: 3
+ line: 3,
+ column: 19,
+ endLine: 3,
+ endColumn: 19 + name.length + 2
}
]
})),
@@ -758,8 +778,10 @@ ruleTester.run('no-reserved-component-names', rule, {
{
messageId: 'reservedInVue',
data: { name },
- type: 'Literal',
- line: 3
+ line: 3,
+ column: 19,
+ endLine: 3,
+ endColumn: 19 + name.length + 2
}
]
})),
@@ -776,8 +798,10 @@ ruleTester.run('no-reserved-component-names', rule, {
{
messageId: 'reservedInVue3',
data: { name },
- type: 'Literal',
- line: 3
+ line: 3,
+ column: 19,
+ endLine: 3,
+ endColumn: 19 + name.length + 2
}
]
}))
diff --git a/tests/lib/rules/no-reserved-keys.js b/tests/lib/rules/no-reserved-keys.js
index 277c12c92..7ece338f1 100644
--- a/tests/lib/rules/no-reserved-keys.js
+++ b/tests/lib/rules/no-reserved-keys.js
@@ -4,7 +4,6 @@
*/
'use strict'
-const semver = require('semver')
const rule = require('../../../lib/rules/no-reserved-keys')
const RuleTester = require('../../eslint-compat').RuleTester
@@ -293,15 +292,9 @@ ruleTester.run('no-reserved-keys', rule, {
}
]
},
- ...(semver.lt(
- require('@typescript-eslint/parser/package.json').version,
- '4.0.0'
- )
- ? []
- : [
- {
- filename: 'test.vue',
- code: `
+ {
+ filename: 'test.vue',
+ code: `
`,
- languageOptions: {
- parser: require('vue-eslint-parser'),
- ecmaVersion: 6,
- parserOptions: {
- parser: require.resolve('@typescript-eslint/parser')
- }
- },
- errors: [
- {
- message: "Key '$el' is reserved.",
- line: 4
- }
- ]
- },
- {
- filename: 'test.vue',
- code: `
+ languageOptions: {
+ parser: require('vue-eslint-parser'),
+ ecmaVersion: 6,
+ parserOptions: {
+ parser: require.resolve('@typescript-eslint/parser')
+ }
+ },
+ errors: [
+ {
+ message: "Key '$el' is reserved.",
+ line: 4
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
`,
- languageOptions: {
- parser: require('vue-eslint-parser'),
- ecmaVersion: 6,
- parserOptions: {
- parser: require.resolve('@typescript-eslint/parser')
- }
- },
- errors: [
- {
- message: "Key '$el' is reserved.",
- line: 4
- }
- ]
- }
- ])
+ languageOptions: {
+ parser: require('vue-eslint-parser'),
+ ecmaVersion: 6,
+ parserOptions: {
+ parser: require.resolve('@typescript-eslint/parser')
+ }
+ },
+ errors: [
+ {
+ message: "Key '$el' is reserved.",
+ line: 4
+ }
+ ]
+ }
]
})
diff --git a/tests/lib/rules/no-restricted-class.js b/tests/lib/rules/no-restricted-class.js
index 97cc0b4b1..ae94d4af1 100644
--- a/tests/lib/rules/no-restricted-class.js
+++ b/tests/lib/rules/no-restricted-class.js
@@ -49,7 +49,10 @@ ruleTester.run('no-restricted-class', rule, {
errors: [
{
message: "'forbidden' class is not allowed.",
- type: 'VAttribute'
+ line: 1,
+ column: 22,
+ endLine: 1,
+ endColumn: 41
}
]
},
@@ -59,7 +62,10 @@ ruleTester.run('no-restricted-class', rule, {
errors: [
{
message: "'forbidden' class is not allowed.",
- type: 'Literal'
+ line: 1,
+ column: 24,
+ endLine: 1,
+ endColumn: 35
}
]
},
@@ -69,7 +75,10 @@ ruleTester.run('no-restricted-class', rule, {
errors: [
{
message: "'forbidden' class is not allowed.",
- type: 'Literal'
+ line: 1,
+ column: 25,
+ endLine: 1,
+ endColumn: 36
}
]
},
@@ -79,7 +88,10 @@ ruleTester.run('no-restricted-class', rule, {
errors: [
{
message: "'forbidden' class is not allowed.",
- type: 'Identifier'
+ line: 1,
+ column: 25,
+ endLine: 1,
+ endColumn: 34
}
]
},
@@ -89,7 +101,10 @@ ruleTester.run('no-restricted-class', rule, {
errors: [
{
message: "'forbidden' class is not allowed.",
- type: 'TemplateElement'
+ line: 1,
+ column: 24,
+ endLine: 1,
+ endColumn: 37
}
]
},
@@ -99,7 +114,10 @@ ruleTester.run('no-restricted-class', rule, {
errors: [
{
message: "'forbidden' class is not allowed.",
- type: 'Literal'
+ line: 1,
+ column: 24,
+ endLine: 1,
+ endColumn: 35
}
]
},
@@ -109,7 +127,10 @@ ruleTester.run('no-restricted-class', rule, {
errors: [
{
message: "'forbidden' class is not allowed.",
- type: 'Literal'
+ line: 1,
+ column: 25,
+ endLine: 1,
+ endColumn: 36
}
]
},
@@ -119,7 +140,10 @@ ruleTester.run('no-restricted-class', rule, {
errors: [
{
message: "'forbidden' class is not allowed.",
- type: 'Literal'
+ line: 1,
+ column: 25,
+ endLine: 1,
+ endColumn: 44
}
]
},
@@ -129,7 +153,10 @@ ruleTester.run('no-restricted-class', rule, {
errors: [
{
message: "'forbidden' class is not allowed.",
- type: 'VAttribute'
+ line: 1,
+ column: 22,
+ endLine: 1,
+ endColumn: 41
}
]
}
diff --git a/tests/lib/rules/no-restricted-props.js b/tests/lib/rules/no-restricted-props.js
index a91615e15..e1d55d090 100644
--- a/tests/lib/rules/no-restricted-props.js
+++ b/tests/lib/rules/no-restricted-props.js
@@ -3,7 +3,6 @@
*/
'use strict'
-const semver = require('semver')
const RuleTester = require('../../eslint-compat').RuleTester
const rule = require('../../../lib/rules/no-restricted-props')
const {
@@ -414,15 +413,9 @@ tester.run('no-restricted-props', rule, {
}
]
},
- ...(semver.lt(
- require('@typescript-eslint/parser/package.json').version,
- '4.0.0'
- )
- ? []
- : [
- {
- filename: 'test.vue',
- code: `
+ {
+ filename: 'test.vue',
+ code: `
`,
- languageOptions: {
- parserOptions: {
- parser: require.resolve('@typescript-eslint/parser')
- }
- },
- options: [{ name: 'foo', suggest: 'Foo' }],
- errors: [
- {
- message: 'Using `foo` props is not allowed.',
- line: 4,
- suggestions: [
- {
- desc: 'Instead, change to `Foo`.',
- output: `
+ options: [{ name: 'foo', suggest: 'Foo' }],
+ languageOptions: {
+ parserOptions: {
+ parser: require.resolve('@typescript-eslint/parser')
+ }
+ },
+ errors: [
+ {
+ message: 'Using `foo` props is not allowed.',
+ line: 4,
+ suggestions: [
+ {
+ desc: 'Instead, change to `Foo`.',
+ output: `
`
- }
- ]
- }
- ]
- },
- {
- filename: 'test.vue',
- code: `
+ }
+ ]
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
`,
- languageOptions: {
- parserOptions: {
- parser: require.resolve('@typescript-eslint/parser')
- }
- },
- options: [{ name: 'foo', suggest: 'Foo' }],
- errors: [
- {
- message: 'Using `foo` props is not allowed.',
- line: 4,
- suggestions: [
- {
- desc: 'Instead, change to `Foo`.',
- output: `
+ options: [{ name: 'foo', suggest: 'Foo' }],
+ languageOptions: {
+ parserOptions: {
+ parser: require.resolve('@typescript-eslint/parser')
+ }
+ },
+ errors: [
+ {
+ message: 'Using `foo` props is not allowed.',
+ line: 4,
+ suggestions: [
+ {
+ desc: 'Instead, change to `Foo`.',
+ output: `
`
- }
- ]
- }
- ]
- },
- {
- filename: 'test.vue',
- code: `
+ }
+ ]
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
@@ -517,20 +510,20 @@ tester.run('no-restricted-props', rule, {
)
`,
- languageOptions: {
- parserOptions: {
- parser: require.resolve('@typescript-eslint/parser')
- }
- },
- options: [{ name: 'foo', suggest: 'Foo' }],
- errors: [
- {
- message: 'Using `foo` props is not allowed.',
- line: 7,
- suggestions: [
- {
- desc: 'Instead, change to `Foo`.',
- output: `
+ options: [{ name: 'foo', suggest: 'Foo' }],
+ languageOptions: {
+ parserOptions: {
+ parser: require.resolve('@typescript-eslint/parser')
+ }
+ },
+ errors: [
+ {
+ message: 'Using `foo` props is not allowed.',
+ line: 7,
+ suggestions: [
+ {
+ desc: 'Instead, change to `Foo`.',
+ output: `
@@ -545,14 +538,14 @@ tester.run('no-restricted-props', rule, {
)
`
- }
- ]
- }
- ]
- },
- {
- filename: 'test.vue',
- code: `
+ }
+ ]
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
`,
- languageOptions: {
- parserOptions: {
- parser: require.resolve('@typescript-eslint/parser')
- }
- },
- options: [{ name: 'foo', suggest: 'Foo' }],
- errors: [
- {
- message: 'Using `foo` props is not allowed.',
- line: 9,
- suggestions: [
- {
- desc: 'Instead, change to `Foo`.',
- output: `
+ options: [{ name: 'foo', suggest: 'Foo' }],
+ languageOptions: {
+ parserOptions: {
+ parser: require.resolve('@typescript-eslint/parser')
+ }
+ },
+ errors: [
+ {
+ message: 'Using `foo` props is not allowed.',
+ line: 9,
+ suggestions: [
+ {
+ desc: 'Instead, change to `Foo`.',
+ output: `
`
- }
- ]
- }
- ]
- },
- {
- code: `
-
- `,
- ...getTypeScriptFixtureTestOptions(),
- options: [{ name: 'foo', suggest: 'Foo' }],
- errors: [
- {
- message: 'Using `foo` props is not allowed.',
- line: 4,
- suggestions: null
- }
- ]
- }
- ]),
+ }
+ ]
+ }
+ ]
+ },
+ {
+ code: `
+
+ `,
+ options: [{ name: 'foo', suggest: 'Foo' }],
+ errors: [
+ {
+ message: 'Using `foo` props is not allowed.',
+ line: 4,
+ suggestions: null
+ }
+ ],
+ ...getTypeScriptFixtureTestOptions()
+ },
{
filename: 'test.vue',
code: `
diff --git a/tests/lib/rules/no-setup-props-reactivity-loss.js b/tests/lib/rules/no-setup-props-reactivity-loss.js
index fce921bfe..4a5f1bc4f 100644
--- a/tests/lib/rules/no-setup-props-reactivity-loss.js
+++ b/tests/lib/rules/no-setup-props-reactivity-loss.js
@@ -203,13 +203,7 @@ tester.run('no-setup-props-reactivity-loss', rule, {
const count = props.count
})
- `,
- errors: [
- {
- messageId: 'getProperty',
- line: 4
- }
- ]
+ `
},
{
filename: 'test.vue',
@@ -223,13 +217,7 @@ tester.run('no-setup-props-reactivity-loss', rule, {
count = props.count
})
- `,
- errors: [
- {
- messageId: 'getProperty',
- line: 4
- }
- ]
+ `
},
{
filename: 'test.vue',
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 6cb6d6284..7ba23f875 100644
--- a/tests/lib/rules/no-side-effects-in-computed-properties.js
+++ b/tests/lib/rules/no-side-effects-in-computed-properties.js
@@ -259,6 +259,7 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
})
const test18 = computed(() => (console.log('a'), true))
const test19 = computed(() => utils.reverse(foo.array))
+ const test20 = computed(() => Object.assign({}, foo.data, { extra: 'value' }))
}
}
`
@@ -296,32 +297,53 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
})`,
errors: [
{
+ message: 'Unexpected side effect in "test1" computed property.',
line: 4,
- message: 'Unexpected side effect in "test1" computed property.'
+ column: 13,
+ endLine: 4,
+ endColumn: 37
},
{
+ message: 'Unexpected side effect in "test2" computed property.',
line: 9,
- message: 'Unexpected side effect in "test2" computed property.'
+ column: 13,
+ endLine: 9,
+ endColumn: 28
},
{
+ message: 'Unexpected side effect in "test2" computed property.',
line: 10,
- message: 'Unexpected side effect in "test2" computed property.'
+ column: 13,
+ endLine: 10,
+ endColumn: 25
},
{
+ message: 'Unexpected side effect in "test3" computed property.',
line: 14,
- message: 'Unexpected side effect in "test3" computed property.'
+ column: 20,
+ endLine: 14,
+ endColumn: 44
},
{
+ message: 'Unexpected side effect in "test4" computed property.',
line: 17,
- message: 'Unexpected side effect in "test4" computed property.'
+ column: 26,
+ endLine: 17,
+ endColumn: 64
},
{
+ message: 'Unexpected side effect in "test5" computed property.',
line: 21,
- message: 'Unexpected side effect in "test5" computed property.'
+ column: 13,
+ endLine: 21,
+ endColumn: 49
},
{
+ message: 'Unexpected side effect in "test6" computed property.',
line: 25,
- message: 'Unexpected side effect in "test6" computed property.'
+ column: 20,
+ endLine: 25,
+ endColumn: 46
}
]
},
@@ -359,24 +381,39 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
})`,
errors: [
{
+ message: 'Unexpected side effect in "test1" computed property.',
line: 5,
- message: 'Unexpected side effect in "test1" computed property.'
+ column: 15,
+ endLine: 5,
+ endColumn: 39
},
{
+ message: 'Unexpected side effect in "test2" computed property.',
line: 11,
- message: 'Unexpected side effect in "test2" computed property.'
+ column: 15,
+ endLine: 11,
+ endColumn: 30
},
{
+ message: 'Unexpected side effect in "test2" computed property.',
line: 12,
- message: 'Unexpected side effect in "test2" computed property.'
+ column: 15,
+ endLine: 12,
+ endColumn: 27
},
{
+ message: 'Unexpected side effect in "test3" computed property.',
line: 18,
- message: 'Unexpected side effect in "test3" computed property.'
+ column: 22,
+ endLine: 18,
+ endColumn: 46
},
{
+ message: 'Unexpected side effect in "test4" computed property.',
line: 23,
- message: 'Unexpected side effect in "test4" computed property.'
+ column: 28,
+ endLine: 23,
+ endColumn: 66
}
]
},
@@ -394,8 +431,11 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
languageOptions: { parser: require('@typescript-eslint/parser') },
errors: [
{
+ message: 'Unexpected side effect in "test1" computed property.',
line: 5,
- message: 'Unexpected side effect in "test1" computed property.'
+ column: 22,
+ endLine: 5,
+ endColumn: 46
}
]
},
@@ -412,8 +452,11 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
})`,
errors: [
{
+ message: 'Unexpected side effect in "test1" computed property.',
line: 4,
- message: 'Unexpected side effect in "test1" computed property.'
+ column: 13,
+ endLine: 4,
+ endColumn: 37
}
]
},
@@ -432,9 +475,27 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
}
})`,
errors: [
- 'Unexpected side effect in "test1" computed property.',
- 'Unexpected side effect in "test2" computed property.',
- 'Unexpected side effect in "test3" computed property.'
+ {
+ message: 'Unexpected side effect in "test1" computed property.',
+ line: 4,
+ column: 20,
+ endLine: 4,
+ endColumn: 48
+ },
+ {
+ message: 'Unexpected side effect in "test2" computed property.',
+ line: 7,
+ column: 20,
+ endLine: 7,
+ endColumn: 50
+ },
+ {
+ message: 'Unexpected side effect in "test3" computed property.',
+ line: 10,
+ column: 20,
+ endLine: 10,
+ endColumn: 50
+ }
]
},
{
@@ -450,12 +511,18 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
})`,
errors: [
{
+ message: 'Unexpected side effect in "fooBar" computed property.',
line: 4,
- message: 'Unexpected side effect in "fooBar" computed property.'
+ column: 18,
+ endLine: 4,
+ endColumn: 22
},
{
+ message: 'Unexpected side effect in "fooBar" computed property.',
line: 5,
- message: 'Unexpected side effect in "fooBar" computed property.'
+ column: 17,
+ endLine: 5,
+ endColumn: 20
}
]
},
@@ -514,52 +581,88 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
`,
errors: [
{
+ message: 'Unexpected side effect in computed function.',
line: 12,
- message: 'Unexpected side effect in computed function.'
+ column: 13,
+ endLine: 12,
+ endColumn: 36
},
{
+ message: 'Unexpected side effect in computed function.',
line: 13,
- message: 'Unexpected side effect in computed function.'
+ column: 13,
+ endLine: 13,
+ endColumn: 34
},
{
+ message: 'Unexpected side effect in computed function.',
line: 17,
- message: 'Unexpected side effect in computed function.'
+ column: 13,
+ endLine: 17,
+ endColumn: 27
},
{
+ message: 'Unexpected side effect in computed function.',
line: 18,
- message: 'Unexpected side effect in computed function.'
+ column: 13,
+ endLine: 18,
+ endColumn: 24
},
{
+ message: 'Unexpected side effect in computed function.',
line: 21,
- message: 'Unexpected side effect in computed function.'
+ column: 40,
+ endLine: 21,
+ endColumn: 63
},
{
+ message: 'Unexpected side effect in computed function.',
line: 23,
- message: 'Unexpected side effect in computed function.'
+ column: 26,
+ endLine: 23,
+ endColumn: 63
},
{
+ message: 'Unexpected side effect in computed function.',
line: 27,
- message: 'Unexpected side effect in computed function.'
+ column: 13,
+ endLine: 27,
+ endColumn: 52
},
{
+ message: 'Unexpected side effect in computed function.',
line: 30,
- message: 'Unexpected side effect in computed function.'
+ column: 40,
+ endLine: 30,
+ endColumn: 65
},
{
+ message: 'Unexpected side effect in computed function.',
line: 33,
- message: 'Unexpected side effect in computed function.'
+ column: 22,
+ endLine: 33,
+ endColumn: 45
},
{
+ message: 'Unexpected side effect in computed function.',
line: 37,
- message: 'Unexpected side effect in computed function.'
+ column: 13,
+ endLine: 37,
+ endColumn: 24
},
{
+ message: 'Unexpected side effect in computed function.',
line: 40,
- message: 'Unexpected side effect in computed function.'
+ column: 13,
+ endLine: 40,
+ endColumn: 24
},
{
+ message: 'Unexpected side effect in computed function.',
line: 42,
- message: 'Unexpected side effect in computed function.'
+ column: 42,
+ endLine: 42,
+ endColumn: 52
}
]
},
@@ -579,8 +682,11 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
`,
errors: [
{
+ message: 'Unexpected side effect in computed function.',
line: 8,
- message: 'Unexpected side effect in computed function.'
+ column: 40,
+ endLine: 8,
+ endColumn: 53
}
]
},
@@ -600,8 +706,11 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
`,
errors: [
{
+ message: 'Unexpected side effect in computed function.',
line: 8,
- message: 'Unexpected side effect in computed function.'
+ column: 40,
+ endLine: 8,
+ endColumn: 63
}
]
},
@@ -657,52 +766,88 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
`,
errors: [
{
+ message: 'Unexpected side effect in computed function.',
line: 10,
- message: 'Unexpected side effect in computed function.'
+ column: 9,
+ endLine: 10,
+ endColumn: 32
},
{
+ message: 'Unexpected side effect in computed function.',
line: 11,
- message: 'Unexpected side effect in computed function.'
+ column: 9,
+ endLine: 11,
+ endColumn: 30
},
{
+ message: 'Unexpected side effect in computed function.',
line: 15,
- message: 'Unexpected side effect in computed function.'
+ column: 9,
+ endLine: 15,
+ endColumn: 23
},
{
+ message: 'Unexpected side effect in computed function.',
line: 16,
- message: 'Unexpected side effect in computed function.'
+ column: 9,
+ endLine: 16,
+ endColumn: 20
},
{
+ message: 'Unexpected side effect in computed function.',
line: 19,
- message: 'Unexpected side effect in computed function.'
+ column: 36,
+ endLine: 19,
+ endColumn: 59
},
{
+ message: 'Unexpected side effect in computed function.',
line: 21,
- message: 'Unexpected side effect in computed function.'
+ column: 22,
+ endLine: 21,
+ endColumn: 59
},
{
+ message: 'Unexpected side effect in computed function.',
line: 25,
- message: 'Unexpected side effect in computed function.'
+ column: 9,
+ endLine: 25,
+ endColumn: 48
},
{
+ message: 'Unexpected side effect in computed function.',
line: 28,
- message: 'Unexpected side effect in computed function.'
+ column: 36,
+ endLine: 28,
+ endColumn: 61
},
{
+ message: 'Unexpected side effect in computed function.',
line: 31,
- message: 'Unexpected side effect in computed function.'
+ column: 18,
+ endLine: 31,
+ endColumn: 41
},
{
+ message: 'Unexpected side effect in computed function.',
line: 35,
- message: 'Unexpected side effect in computed function.'
+ column: 9,
+ endLine: 35,
+ endColumn: 20
},
{
+ message: 'Unexpected side effect in computed function.',
line: 38,
- message: 'Unexpected side effect in computed function.'
+ column: 9,
+ endLine: 38,
+ endColumn: 20
},
{
+ message: 'Unexpected side effect in computed function.',
line: 40,
- message: 'Unexpected side effect in computed function.'
+ column: 38,
+ endLine: 40,
+ endColumn: 48
}
]
},
@@ -718,8 +863,11 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
`,
errors: [
{
+ message: 'Unexpected side effect in computed function.',
line: 6,
- message: 'Unexpected side effect in computed function.'
+ column: 36,
+ endLine: 6,
+ endColumn: 49
}
]
},
@@ -735,8 +883,59 @@ ruleTester.run('no-side-effects-in-computed-properties', rule, {
`,
errors: [
{
+ message: 'Unexpected side effect in computed function.',
line: 6,
- message: 'Unexpected side effect in computed function.'
+ column: 36,
+ endLine: 6,
+ endColumn: 59
+ }
+ ]
+ },
+ {
+ // Object.assign mutating the prop as first argument in computed properties
+ filename: 'test.vue',
+ code: `
+
+ `,
+ errors: [
+ {
+ message: 'Unexpected side effect in computed function.',
+ line: 8,
+ column: 40,
+ endLine: 8,
+ endColumn: 83
+ },
+ {
+ message: 'Unexpected side effect in computed function.',
+ line: 10,
+ column: 20,
+ endLine: 10,
+ endColumn: 56
+ },
+ {
+ message: 'Unexpected side effect in computed function.',
+ line: 14,
+ column: 15,
+ endLine: 14,
+ endColumn: 61
}
]
}
diff --git a/tests/lib/rules/no-template-shadow.js b/tests/lib/rules/no-template-shadow.js
index 3cc5218d1..6cbd94c58 100644
--- a/tests/lib/rules/no-template-shadow.js
+++ b/tests/lib/rules/no-template-shadow.js
@@ -180,7 +180,10 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'i' is already declared in the upper scope.",
- type: 'Identifier'
+ line: 1,
+ column: 43,
+ endLine: 1,
+ endColumn: 44
}
]
},
@@ -199,8 +202,10 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'i' is already declared in the upper scope.",
- type: 'Identifier',
- line: 2
+ line: 2,
+ column: 21,
+ endLine: 2,
+ endColumn: 22
}
]
},
@@ -220,13 +225,17 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'i' is already declared in the upper scope.",
- type: 'Identifier',
- line: 2
+ line: 2,
+ column: 21,
+ endLine: 2,
+ endColumn: 22
},
{
message: "Variable 'i' is already declared in the upper scope.",
- type: 'Identifier',
- line: 3
+ line: 3,
+ column: 21,
+ endLine: 3,
+ endColumn: 22
}
]
},
@@ -247,13 +256,17 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'i' is already declared in the upper scope.",
- type: 'Identifier',
- line: 2
+ line: 2,
+ column: 21,
+ endLine: 2,
+ endColumn: 22
},
{
message: "Variable 'i' is already declared in the upper scope.",
- type: 'Identifier',
- line: 3
+ line: 3,
+ column: 23,
+ endLine: 3,
+ endColumn: 24
}
]
},
@@ -274,8 +287,10 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'i' is already declared in the upper scope.",
- type: 'Identifier',
- line: 2
+ line: 2,
+ column: 21,
+ endLine: 2,
+ endColumn: 22
}
]
},
@@ -298,13 +313,17 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'i' is already declared in the upper scope.",
- type: 'Identifier',
- line: 2
+ line: 2,
+ column: 21,
+ endLine: 2,
+ endColumn: 22
},
{
message: "Variable 'f' is already declared in the upper scope.",
- type: 'Identifier',
- line: 3
+ line: 3,
+ column: 21,
+ endLine: 3,
+ endColumn: 22
}
]
},
@@ -337,13 +356,17 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'e' is already declared in the upper scope.",
- type: 'Identifier',
- line: 6
+ line: 6,
+ column: 21,
+ endLine: 6,
+ endColumn: 22
},
{
message: "Variable 'f' is already declared in the upper scope.",
- type: 'Identifier',
- line: 7
+ line: 7,
+ column: 21,
+ endLine: 7,
+ endColumn: 22
}
]
},
@@ -376,13 +399,17 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'e' is already declared in the upper scope.",
- type: 'Identifier',
- line: 6
+ line: 6,
+ column: 21,
+ endLine: 6,
+ endColumn: 22
},
{
message: "Variable 'f' is already declared in the upper scope.",
- type: 'Identifier',
- line: 7
+ line: 7,
+ column: 21,
+ endLine: 7,
+ endColumn: 22
}
]
},
@@ -413,13 +440,17 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'e' is already declared in the upper scope.",
- type: 'Identifier',
- line: 6
+ line: 6,
+ column: 21,
+ endLine: 6,
+ endColumn: 22
},
{
message: "Variable 'f' is already declared in the upper scope.",
- type: 'Identifier',
- line: 7
+ line: 7,
+ column: 21,
+ endLine: 7,
+ endColumn: 22
}
]
},
@@ -445,7 +476,10 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'j' is already declared in the upper scope.",
- line: 4
+ line: 4,
+ column: 23,
+ endLine: 4,
+ endColumn: 24
}
]
},
@@ -465,7 +499,10 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'j' is already declared in the upper scope.",
- line: 4
+ line: 4,
+ column: 23,
+ endLine: 4,
+ endColumn: 24
}
]
},
@@ -485,7 +522,10 @@ ruleTester.run('no-template-shadow', rule, {
errors: [
{
message: "Variable 'j' is already declared in the upper scope.",
- line: 4
+ line: 4,
+ column: 23,
+ endLine: 4,
+ endColumn: 24
}
]
}
diff --git a/tests/lib/rules/no-undef-components.js b/tests/lib/rules/no-undef-components.js
index 37a3bec68..f87f9dcd5 100644
--- a/tests/lib/rules/no-undef-components.js
+++ b/tests/lib/rules/no-undef-components.js
@@ -6,7 +6,6 @@
const RuleTester = require('../../eslint-compat').RuleTester
const rule = require('../../../lib/rules/no-undef-components')
-const semver = require('semver')
const tester = new RuleTester({
languageOptions: {
@@ -746,14 +745,9 @@ tester.run('no-undef-components', rule, {
}
]
},
- ...(semver.gte(
- require('@typescript-eslint/parser/package.json').version,
- '5.0.0'
- )
- ? [
- {
- 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: `
+
+
+
+
+ `,
+ options: [{ ignore: ['/foo/'] }]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+ `,
+ options: [{ ignore: ['foo-bar'] }]
+ }
+ ],
+ invalid: [
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `,
+ errors: [
+ {
+ message: "The 'v-focus' directive has been used, but not defined.",
+ line: 3,
+ column: 14,
+ endLine: 3,
+ endColumn: 21
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+ `,
+ errors: [
+ {
+ message: "The 'v-focus' directive has been used, but not defined.",
+ line: 6,
+ column: 14,
+ endLine: 6,
+ endColumn: 21
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+ `,
+ errors: [
+ {
+ message: "The 'v-foo' directive has been used, but not defined.",
+ line: 6,
+ column: 14,
+ endLine: 6,
+ endColumn: 19
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+ `,
+ errors: [
+ {
+ message: "The 'v-focus' directive has been used, but not defined.",
+ line: 6,
+ column: 14,
+ endLine: 6,
+ endColumn: 21
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+ `,
+ errors: [
+ {
+ message:
+ "The 'v-clickoutside' directive has been used, but not defined.",
+ line: 6,
+ column: 14,
+ endLine: 6,
+ endColumn: 28
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+ `,
+ errors: [
+ {
+ message:
+ "The 'v-click-outside' directive has been used, but not defined.",
+ line: 6,
+ column: 14,
+ endLine: 6,
+ endColumn: 29
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `,
+ errors: [
+ {
+ message: "The 'v-foo' directive has been used, but not defined.",
+ line: 3,
+ column: 14,
+ endLine: 3,
+ endColumn: 25
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `,
+ errors: [
+ {
+ message: "The 'v-foo' directive has been used, but not defined.",
+ line: 3,
+ column: 14,
+ endLine: 3,
+ endColumn: 23
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `,
+ errors: [
+ {
+ message: "The 'v-foo' directive has been used, but not defined.",
+ line: 3,
+ column: 14,
+ endLine: 3,
+ endColumn: 27
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+ `,
+ errors: [
+ {
+ message: "The 'v-Focus' directive has been used, but not defined.",
+ line: 10,
+ column: 14,
+ endLine: 10,
+ endColumn: 21
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+ `,
+ errors: [
+ {
+ message: "The 'v-foo-bar' directive has been used, but not defined.",
+ line: 10,
+ column: 14,
+ endLine: 10,
+ endColumn: 23
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `,
+ options: [{ ignore: ['baz'] }],
+ errors: [
+ {
+ message: "The 'v-foo-bar' directive has been used, but not defined.",
+ line: 3,
+ column: 14,
+ endLine: 3,
+ endColumn: 23
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+ `,
+ options: [{ ignore: ['/^bar/'] }],
+ errors: [
+ {
+ message: "The 'v-foo-bar' directive has been used, but not defined.",
+ line: 3,
+ column: 14,
+ endLine: 3,
+ endColumn: 23
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+
+
+
+ `,
+ options: [{ ignore: ['foo'] }],
+ errors: [
+ {
+ message:
+ "The 'v-barfoobaz' directive has been used, but not defined.",
+ line: 4,
+ column: 14,
+ endLine: 4,
+ endColumn: 25
+ }
+ ]
+ }
+ ]
+})
diff --git a/tests/lib/rules/no-v-html.js b/tests/lib/rules/no-v-html.js
index d9ae67ca1..16b606a63 100644
--- a/tests/lib/rules/no-v-html.js
+++ b/tests/lib/rules/no-v-html.js
@@ -28,23 +28,71 @@ ruleTester.run('no-v-html', rule, {
{
filename: 'test.vue',
code: '
'
+ },
+ {
+ filename: 'test.vue',
+ code: '
',
+ options: [{ ignorePattern: '^html' }]
+ },
+ {
+ filename: 'test.vue',
+ code: '
',
+ options: [{ ignorePattern: String.raw`^\$sanitize\(` }]
}
],
invalid: [
{
filename: 'test.vue',
code: '
',
- errors: ["'v-html' directive can lead to XSS attack."]
+ errors: [
+ {
+ message: "'v-html' directive can lead to XSS attack.",
+ line: 1,
+ column: 16,
+ endLine: 1,
+ endColumn: 28
+ }
+ ]
},
{
filename: 'test.vue',
code: '
',
- errors: ["'v-html' directive can lead to XSS attack."]
+ errors: [
+ {
+ message: "'v-html' directive can lead to XSS attack.",
+ line: 1,
+ column: 15,
+ endLine: 1,
+ endColumn: 36
+ }
+ ]
},
{
filename: 'test.vue',
code: '
',
- errors: ["'v-html' directive can lead to XSS attack."]
+ errors: [
+ {
+ message: "'v-html' directive can lead to XSS attack.",
+ line: 1,
+ column: 20,
+ endLine: 1,
+ endColumn: 26
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: '
',
+ options: [{ ignorePattern: '^html' }],
+ errors: [
+ {
+ message: "'v-html' directive can lead to XSS attack.",
+ line: 1,
+ column: 16,
+ endLine: 1,
+ endColumn: 37
+ }
+ ]
}
]
})
diff --git a/tests/lib/rules/prop-name-casing.js b/tests/lib/rules/prop-name-casing.js
index a83893a5a..ea15dddb4 100644
--- a/tests/lib/rules/prop-name-casing.js
+++ b/tests/lib/rules/prop-name-casing.js
@@ -4,9 +4,11 @@
*/
'use strict'
-const semver = require('semver')
const rule = require('../../../lib/rules/prop-name-casing')
const RuleTester = require('../../eslint-compat').RuleTester
+const {
+ getTypeScriptFixtureTestOptions
+} = require('../../test-utils/typescript')
const languageOptions = {
ecmaVersion: 2018,
@@ -376,6 +378,16 @@ ruleTester.run('prop-name-casing', rule, {
{ ignoreProps: ['ignored_prop', '/^ignored-pattern-/'] }
],
languageOptions
+ },
+ {
+ code: `
+
+ `,
+ ...getTypeScriptFixtureTestOptions()
}
],
@@ -393,8 +405,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greeting_text" is not in camelCase.',
- type: 'Property',
- line: 4
+ line: 4,
+ column: 13,
+ endLine: 4,
+ endColumn: 34
}
]
},
@@ -412,8 +426,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greeting_text" is not in camelCase.',
- type: 'Property',
- line: 4
+ line: 4,
+ column: 13,
+ endLine: 4,
+ endColumn: 34
}
]
},
@@ -429,8 +445,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greeting_text" is not in camelCase.',
- type: 'Literal',
- line: 3
+ line: 3,
+ column: 19,
+ endLine: 3,
+ endColumn: 34
}
]
},
@@ -448,8 +466,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greetingText" is not in snake_case.',
- type: 'Property',
- line: 4
+ line: 4,
+ column: 13,
+ endLine: 4,
+ endColumn: 33
}
]
},
@@ -467,8 +487,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greeting-text" is not in camelCase.',
- type: 'Property',
- line: 4
+ line: 4,
+ column: 13,
+ endLine: 4,
+ endColumn: 36
}
]
},
@@ -486,8 +508,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greeting-text" is not in snake_case.',
- type: 'Property',
- line: 4
+ line: 4,
+ column: 13,
+ endLine: 4,
+ endColumn: 36
}
]
},
@@ -504,8 +528,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greeting_text" is not in camelCase.',
- type: 'Property',
- line: 4
+ line: 4,
+ column: 13,
+ endLine: 4,
+ endColumn: 36
}
]
},
@@ -523,8 +549,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greeting-text" is not in camelCase.',
- type: 'Property',
- line: 4
+ line: 4,
+ column: 13,
+ endLine: 4,
+ endColumn: 38
}
]
},
@@ -542,8 +570,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greeting_text" is not in camelCase.',
- type: 'Property',
- line: 4
+ line: 4,
+ column: 13,
+ endLine: 4,
+ endColumn: 26
}
]
},
@@ -560,8 +590,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "abc-123-def" is not in camelCase.',
- type: 'Property',
- line: 4
+ line: 4,
+ column: 13,
+ endLine: 4,
+ endColumn: 34
}
]
},
@@ -579,8 +611,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greeting-text" is not in camelCase.',
- type: 'Property',
- line: 4
+ line: 4,
+ column: 13,
+ endLine: 4,
+ endColumn: 40
}
]
},
@@ -651,7 +685,10 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greeting_text" is not in camelCase.',
- line: 4
+ line: 4,
+ column: 9,
+ endLine: 4,
+ endColumn: 30
}
]
},
@@ -669,41 +706,40 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "greeting_text" is not in camelCase.',
- line: 3
+ line: 3,
+ column: 20,
+ endLine: 3,
+ endColumn: 35
+ }
+ ]
+ },
+ {
+ filename: 'test.vue',
+ code: `
+
+ `,
+ languageOptions: {
+ parser: require('vue-eslint-parser'),
+ ...languageOptions,
+ parserOptions: {
+ parser: require.resolve('@typescript-eslint/parser')
+ }
+ },
+ errors: [
+ {
+ message: 'Prop "greeting_text" is not in camelCase.',
+ line: 4,
+ column: 9,
+ endLine: 4,
+ endColumn: 30
}
]
},
- ...(semver.lt(
- require('@typescript-eslint/parser/package.json').version,
- '4.0.0'
- )
- ? []
- : [
- {
- filename: 'test.vue',
- code: `
-
- `,
- languageOptions: {
- parser: require('vue-eslint-parser'),
- ...languageOptions,
- parserOptions: {
- parser: require.resolve('@typescript-eslint/parser')
- }
- },
- errors: [
- {
- message: 'Prop "greeting_text" is not in camelCase.',
- line: 4
- }
- ]
- }
- ]),
{
filename: 'test.vue',
code: `
@@ -720,13 +756,17 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "notIgnored_prop" is not in camelCase.',
- type: 'Property',
- line: 4
+ line: 4,
+ column: 13,
+ endLine: 4,
+ endColumn: 36
},
{
message: 'Prop "other-pattern" is not in camelCase.',
- type: 'Property',
- line: 5
+ line: 5,
+ column: 13,
+ endLine: 5,
+ endColumn: 36
}
]
},
@@ -742,15 +782,52 @@ ruleTester.run('prop-name-casing', rule, {
errors: [
{
message: 'Prop "notIgnored_prop" is not in camelCase.',
- type: 'Literal',
- line: 3
+ line: 3,
+ column: 19,
+ endLine: 3,
+ endColumn: 36
},
{
message: 'Prop "pattern_invalid" is not in camelCase.',
- type: 'Literal',
- line: 3
+ line: 3,
+ column: 38,
+ endLine: 3,
+ endColumn: 55
}
]
+ },
+ {
+ code: `
+
+ `,
+ errors: [
+ {
+ message: 'Prop "snake_case" is not in camelCase.',
+ line: 5,
+ column: 19,
+ endLine: 5,
+ endColumn: 24
+ },
+ {
+ message: 'Prop "kebab-case" is not in camelCase.',
+ line: 5,
+ column: 19,
+ endLine: 5,
+ endColumn: 24
+ },
+ {
+ message: 'Prop "PascalCase" is not in camelCase.',
+ line: 5,
+ column: 19,
+ endLine: 5,
+ endColumn: 24
+ }
+ ],
+ ...getTypeScriptFixtureTestOptions()
}
]
})
diff --git a/tests/lib/rules/require-default-prop.js b/tests/lib/rules/require-default-prop.js
index 43160dfda..cbca0ee19 100644
--- a/tests/lib/rules/require-default-prop.js
+++ b/tests/lib/rules/require-default-prop.js
@@ -4,7 +4,6 @@
*/
'use strict'
-const semver = require('semver')
const rule = require('../../../lib/rules/require-default-prop')
const RuleTester = require('../../eslint-compat').RuleTester
const languageOptions = {
@@ -655,37 +654,30 @@ ruleTester.run('require-default-prop', rule, {
}
]
},
- ...(semver.lt(
- require('@typescript-eslint/parser/package.json').version,
- '4.0.0'
- )
- ? []
- : [
- {
- filename: 'test.vue',
- code: `
-
- `,
- languageOptions: {
- parser: require('vue-eslint-parser'),
- ...languageOptions,
- parserOptions: {
- parser: require.resolve('@typescript-eslint/parser')
- }
- },
- errors: [
- {
- message: "Prop 'foo' requires default value to be set.",
- line: 4
- }
- ]
- }
- ]),
+ {
+ filename: 'test.vue',
+ code: `
+
+ `,
+ languageOptions: {
+ parser: require('vue-eslint-parser'),
+ ...languageOptions,
+ parserOptions: {
+ parser: require.resolve('@typescript-eslint/parser')
+ }
+ },
+ errors: [
+ {
+ message: "Prop 'foo' requires default value to be set.",
+ line: 4
+ }
+ ]
+ },
{
filename: 'test.vue',
code: `
diff --git a/tests/lib/rules/require-direct-export.js b/tests/lib/rules/require-direct-export.js
index cec06c7a0..0a5816e94 100644
--- a/tests/lib/rules/require-direct-export.js
+++ b/tests/lib/rules/require-direct-export.js
@@ -94,8 +94,10 @@ ruleTester.run('require-direct-export', rule, {
errors: [
{
message: 'Expected the component literal to be directly exported.',
- type: 'ExportDefaultDeclaration',
- line: 3
+ line: 3,
+ column: 7,
+ endLine: 3,
+ endColumn: 23
}
]
},
@@ -109,8 +111,10 @@ ruleTester.run('require-direct-export', rule, {
errors: [
{
message: 'Expected the component literal to be directly exported.',
- type: 'ExportDefaultDeclaration',
- line: 5
+ line: 5,
+ column: 7,
+ endLine: 5,
+ endColumn: 23
}
]
},
@@ -120,8 +124,10 @@ ruleTester.run('require-direct-export', rule, {
errors: [
{
message: 'Expected the component literal to be directly exported.',
- type: 'ExportDefaultDeclaration',
- line: 1
+ line: 1,
+ column: 1,
+ endLine: 1,
+ endColumn: 38
}
]
},
@@ -131,8 +137,10 @@ ruleTester.run('require-direct-export', rule, {
errors: [
{
message: 'Expected the component literal to be directly exported.',
- type: 'ExportDefaultDeclaration',
- line: 1
+ line: 1,
+ column: 1,
+ endLine: 1,
+ endColumn: 30
}
]
},
@@ -142,8 +150,10 @@ ruleTester.run('require-direct-export', rule, {
errors: [
{
message: 'Expected the component literal to be directly exported.',
- type: 'ExportDefaultDeclaration',
- line: 1
+ line: 1,
+ column: 1,
+ endLine: 1,
+ endColumn: 24
}
]
},
@@ -157,8 +167,10 @@ ruleTester.run('require-direct-export', rule, {
errors: [
{
message: 'Expected the component literal to be directly exported.',
- type: 'ExportDefaultDeclaration',
- line: 1
+ line: 1,
+ column: 1,
+ endLine: 5,
+ endColumn: 8
}
]
},
@@ -170,8 +182,10 @@ ruleTester.run('require-direct-export', rule, {
errors: [
{
message: 'Expected the component literal to be directly exported.',
- type: 'ExportDefaultDeclaration',
- line: 1
+ line: 1,
+ column: 1,
+ endLine: 3,
+ endColumn: 8
}
]
},
@@ -186,8 +200,10 @@ ruleTester.run('require-direct-export', rule, {
errors: [
{
message: 'Expected the component literal to be directly exported.',
- type: 'ExportDefaultDeclaration',
- line: 5
+ line: 5,
+ column: 7,
+ endLine: 5,
+ endColumn: 23
}
]
},
diff --git a/tests/lib/rules/require-name-property.js b/tests/lib/rules/require-name-property.js
index c61c46759..517e8df58 100644
--- a/tests/lib/rules/require-name-property.js
+++ b/tests/lib/rules/require-name-property.js
@@ -80,7 +80,10 @@ ruleTester.run('require-name-property', rule, {
errors: [
{
message: 'Required name property is not set.',
- type: 'ObjectExpression',
+ line: 2,
+ column: 24,
+ endLine: 3,
+ endColumn: 10,
suggestions: [
{
desc: 'Add name property to component.',
@@ -104,7 +107,10 @@ ruleTester.run('require-name-property', rule, {
errors: [
{
message: 'Required name property is not set.',
- type: 'ObjectExpression',
+ line: 2,
+ column: 40,
+ endLine: 3,
+ endColumn: 10,
suggestions: [
{
desc: 'Add name property to component.',
@@ -127,7 +133,10 @@ ruleTester.run('require-name-property', rule, {
errors: [
{
message: 'Required name property is not set.',
- type: 'ObjectExpression',
+ line: 2,
+ column: 40,
+ endLine: 2,
+ endColumn: 43,
suggestions: [
{
desc: 'Add name property to component.',
@@ -150,7 +159,10 @@ ruleTester.run('require-name-property', rule, {
errors: [
{
message: 'Required name property is not set.',
- type: 'ObjectExpression',
+ line: 2,
+ column: 24,
+ endLine: 2,
+ endColumn: 27,
suggestions: [
{
desc: 'Add name property to component.',
@@ -175,7 +187,10 @@ ruleTester.run('require-name-property', rule, {
errors: [
{
message: 'Required name property is not set.',
- type: 'ObjectExpression',
+ line: 2,
+ column: 24,
+ endLine: 4,
+ endColumn: 10,
suggestions: [
{
desc: 'Add name property to component.',
@@ -201,7 +216,10 @@ ruleTester.run('require-name-property', rule, {
errors: [
{
message: 'Required name property is not set.',
- type: 'ObjectExpression',
+ line: 2,
+ column: 40,
+ endLine: 4,
+ endColumn: 10,
suggestions: [
{
desc: 'Add name property to component.',
@@ -230,7 +248,10 @@ ruleTester.run('require-name-property', rule, {
errors: [
{
message: 'Required name property is not set.',
- type: 'ObjectExpression',
+ line: 2,
+ column: 24,
+ endLine: 6,
+ endColumn: 10,
suggestions: [
{
desc: 'Add name property to component.',
@@ -258,7 +279,10 @@ ruleTester.run('require-name-property', rule, {
errors: [
{
message: 'Required name property is not set.',
- type: 'ObjectExpression',
+ line: 2,
+ column: 24,
+ endLine: 4,
+ endColumn: 10,
suggestions: [
{
desc: 'Add name property to component.',
diff --git a/tests/lib/rules/require-render-return.js b/tests/lib/rules/require-render-return.js
index d2aad3c01..0fa628302 100644
--- a/tests/lib/rules/require-render-return.js
+++ b/tests/lib/rules/require-render-return.js
@@ -140,8 +140,10 @@ ruleTester.run('require-render-return', rule, {
errors: [
{
message: 'Expected to return a value in render function.',
- type: 'Identifier',
- line: 2
+ line: 2,
+ column: 9,
+ endLine: 2,
+ endColumn: 15
}
]
},
@@ -158,8 +160,10 @@ ruleTester.run('require-render-return', rule, {
errors: [
{
message: 'Expected to return a value in render function.',
- type: 'Identifier',
- line: 2
+ line: 2,
+ column: 9,
+ endLine: 2,
+ endColumn: 15
}
]
},
@@ -175,8 +179,10 @@ ruleTester.run('require-render-return', rule, {
errors: [
{
message: 'Expected to return a value in render function.',
- type: 'Identifier',
- line: 2
+ line: 2,
+ column: 9,
+ endLine: 2,
+ endColumn: 15
}
]
},
@@ -192,8 +198,10 @@ ruleTester.run('require-render-return', rule, {
errors: [
{
message: 'Expected to return a value in render function.',
- type: 'Identifier',
- line: 2
+ line: 2,
+ column: 9,
+ endLine: 2,
+ endColumn: 15
}
]
},
@@ -209,8 +217,10 @@ ruleTester.run('require-render-return', rule, {
errors: [
{
message: 'Expected to return a value in render function.',
- type: 'Identifier',
- line: 2
+ line: 2,
+ column: 9,
+ endLine: 2,
+ endColumn: 15
}
]
},
@@ -228,8 +238,10 @@ ruleTester.run('require-render-return', rule, {
errors: [
{
message: 'Expected to return a value in render function.',
- type: 'Identifier',
- line: 2
+ line: 2,
+ column: 9,
+ endLine: 2,
+ endColumn: 15
}
]
}
diff --git a/tests/lib/rules/require-typed-ref.js b/tests/lib/rules/require-typed-ref.js
index 6f3b359d6..ec42c0751 100644
--- a/tests/lib/rules/require-typed-ref.js
+++ b/tests/lib/rules/require-typed-ref.js
@@ -242,7 +242,6 @@ tester.run('require-typed-ref', rule, {
}
}
- }
`,
languageOptions: { parser: require('vue-eslint-parser') },
errors: [
diff --git a/tests/lib/rules/singleline-html-element-content-newline.js b/tests/lib/rules/singleline-html-element-content-newline.js
index 52c01392b..83366266b 100644
--- a/tests/lib/rules/singleline-html-element-content-newline.js
+++ b/tests/lib/rules/singleline-html-element-content-newline.js
@@ -251,7 +251,6 @@ content
'Expected 1 line break after opening tag (`
`), but no line breaks found.',
line: 3,
column: 30,
- type: 'HTMLTagClose',
endLine: 3,
endColumn: 30
},
@@ -260,7 +259,6 @@ content
'Expected 1 line break before closing tag (`
`), but no line breaks found.',
line: 3,
column: 37,
- type: 'HTMLEndTagOpen',
endLine: 3,
endColumn: 37
}
diff --git a/tests/lib/rules/space-in-parens.js b/tests/lib/rules/space-in-parens.js
index d6218216d..9b1509f07 100644
--- a/tests/lib/rules/space-in-parens.js
+++ b/tests/lib/rules/space-in-parens.js
@@ -3,21 +3,9 @@
*/
'use strict'
-const { RuleTester, ESLint } = require('../../eslint-compat')
-const semver = require('semver')
+const { RuleTester } = require('../../eslint-compat')
const rule = require('../../../lib/rules/space-in-parens')
-const errorMessage = semver.lt(ESLint.version, '6.4.0')
- ? (obj) => {
- const messageId = obj.messageId
- delete obj.messageId
- obj.message = messageId.startsWith('missing')
- ? 'There must be a space inside this paren.'
- : 'There should be no spaces inside this paren.'
- return obj
- }
- : (obj) => obj
-
const tester = new RuleTester({
languageOptions: { parser: require('vue-eslint-parser'), ecmaVersion: 2015 }
})
@@ -76,14 +64,14 @@ tester.run('space-in-parens', rule, {
/>
`,
errors: [
- errorMessage({
+ {
messageId: 'rejectedOpeningSpace',
line: 4
- }),
- errorMessage({
+ },
+ {
messageId: 'rejectedClosingSpace',
line: 4
- })
+ }
]
},
{
@@ -101,14 +89,14 @@ tester.run('space-in-parens', rule, {
`,
options: ['always'],
errors: [
- errorMessage({
+ {
messageId: 'missingOpeningSpace',
line: 4
- }),
- errorMessage({
+ },
+ {
messageId: 'missingClosingSpace',
line: 4
- })
+ }
]
},
{
@@ -125,14 +113,14 @@ tester.run('space-in-parens', rule, {
>
`,
errors: [
- errorMessage({
+ {
messageId: 'rejectedOpeningSpace',
line: 4
- }),
- errorMessage({
+ },
+ {
messageId: 'rejectedClosingSpace',
line: 4
- })
+ }
]
},
{
@@ -150,14 +138,14 @@ tester.run('space-in-parens', rule, {
`,
options: ['always'],
errors: [
- errorMessage({
+ {
messageId: 'missingOpeningSpace',
line: 4
- }),
- errorMessage({
+ },
+ {
messageId: 'missingClosingSpace',
line: 4
- })
+ }
]
},
{
@@ -174,14 +162,14 @@ tester.run('space-in-parens', rule, {
>
`,
errors: [
- errorMessage({
+ {
messageId: 'rejectedOpeningSpace',
line: 4
- }),
- errorMessage({
+ },
+ {
messageId: 'rejectedClosingSpace',
line: 4
- })
+ }
]
},
{
@@ -199,14 +187,14 @@ tester.run('space-in-parens', rule, {
`,
options: ['always'],
errors: [
- errorMessage({
+ {
messageId: 'missingOpeningSpace',
line: 4
- }),
- errorMessage({
+ },
+ {
messageId: 'missingClosingSpace',
line: 4
- })
+ }
]
},
@@ -225,14 +213,14 @@ tester.run('space-in-parens', rule, {
}
`,
errors: [
- errorMessage({
+ {
messageId: 'rejectedOpeningSpace',
line: 4
- }),
- errorMessage({
+ },
+ {
messageId: 'rejectedClosingSpace',
line: 4
- })
+ }
]
}
]
diff --git a/tests/lib/rules/space-infix-ops.js b/tests/lib/rules/space-infix-ops.js
index 0325884a7..d05e25170 100644
--- a/tests/lib/rules/space-infix-ops.js
+++ b/tests/lib/rules/space-infix-ops.js
@@ -3,17 +3,14 @@
*/
'use strict'
-const { RuleTester, ESLint } = require('../../eslint-compat')
-const semver = require('semver')
+const { RuleTester } = require('../../eslint-compat')
const rule = require('../../../lib/rules/space-infix-ops')
const tester = new RuleTester({
languageOptions: { parser: require('vue-eslint-parser'), ecmaVersion: 2015 }
})
-const message = semver.lt(ESLint.version, '5.10.0')
- ? () => 'Infix operators must be spaced.'
- : (operator) => `Operator '${operator}' must be spaced.`
+const message = (operator) => `Operator '${operator}' must be spaced.`
tester.run('space-infix-ops', rule, {
valid: [
diff --git a/tests/lib/rules/static-class-names-order.js b/tests/lib/rules/static-class-names-order.js
index 0f293f5e9..748cd6603 100644
--- a/tests/lib/rules/static-class-names-order.js
+++ b/tests/lib/rules/static-class-names-order.js
@@ -42,7 +42,10 @@ tester.run('static-class-names-order', rule, {
errors: [
{
message: 'Classes should be ordered alphabetically.',
- type: 'VAttribute'
+ line: 1,
+ column: 16,
+ endLine: 1,
+ endColumn: 27
}
]
},
@@ -59,7 +62,10 @@ tester.run('static-class-names-order', rule, {
errors: [
{
message: 'Classes should be ordered alphabetically.',
- type: 'VAttribute'
+ line: 2,
+ column: 16,
+ endLine: 2,
+ endColumn: 29
}
]
}
diff --git a/tests/lib/rules/valid-define-options.js b/tests/lib/rules/valid-define-options.js
index 867bcb569..330bc819f 100644
--- a/tests/lib/rules/valid-define-options.js
+++ b/tests/lib/rules/valid-define-options.js
@@ -74,6 +74,14 @@ tester.run('valid-define-options', rule, {
defineOptions(def);
`
+ },
+ {
+ filename: 'test.vue',
+ code: `
+ `
}
],
invalid: [
diff --git a/tests/lib/utils/selector.js b/tests/lib/utils/selector.js
index b5b0ee986..4fff7cfa9 100644
--- a/tests/lib/utils/selector.js
+++ b/tests/lib/utils/selector.js
@@ -45,9 +45,7 @@ function extractElements(code, inputSelector) {
VElement(node) {
if (parsed.test(node)) {
matches.push(
- context
- .getSourceCode()
- .text.slice(...node.startTag.range)
+ context.sourceCode.text.slice(...node.startTag.range)
)
}
}
diff --git a/tests/lib/utils/vue-component.js b/tests/lib/utils/vue-component.js
index 12fcd904f..f2cce4331 100644
--- a/tests/lib/utils/vue-component.js
+++ b/tests/lib/utils/vue-component.js
@@ -26,11 +26,13 @@ const languageOptions = {
sourceType: 'module'
}
-function makeError(line) {
+function makeError(line, column, endLine, endColumn) {
return {
message: 'Component detected.',
line,
- type: 'ObjectExpression'
+ column,
+ endLine,
+ endColumn
}
}
@@ -132,37 +134,37 @@ function invalidTests(ext) {
// ${ext}
`,
languageOptions,
- errors: [makeError(4)]
+ errors: [makeError(4, 19, 4, 21)]
},
{
filename: `test.${ext}`,
code: `Vue.component({})`,
languageOptions,
- errors: [makeError(1)]
+ errors: [makeError(1, 15, 1, 17)]
},
{
filename: `test.${ext}`,
code: `Vue.mixin({})`,
languageOptions,
- errors: [makeError(1)]
+ errors: [makeError(1, 11, 1, 13)]
},
{
filename: `test.${ext}`,
code: `Vue.extend({})`,
languageOptions,
- errors: [makeError(1)]
+ errors: [makeError(1, 12, 1, 14)]
},
{
filename: `test.${ext}`,
code: `app.component('name', {})`,
languageOptions,
- errors: [makeError(1)]
+ errors: [makeError(1, 23, 1, 25)]
},
{
filename: `test.${ext}`,
code: `app.mixin({})`,
languageOptions,
- errors: [makeError(1)]
+ errors: [makeError(1, 11, 1, 13)]
},
{
filename: `test.${ext}`,
@@ -171,7 +173,7 @@ function invalidTests(ext) {
...languageOptions,
parser: require('@typescript-eslint/parser')
},
- errors: [makeError(1)]
+ errors: [makeError(1, 52, 1, 54)]
},
{
filename: `test.${ext}`,
@@ -180,7 +182,7 @@ function invalidTests(ext) {
...languageOptions,
parser: require('@typescript-eslint/parser')
},
- errors: [makeError(1)]
+ errors: [makeError(1, 27, 1, 29)]
},
{
filename: `test.${ext}`,
@@ -189,13 +191,13 @@ function invalidTests(ext) {
...languageOptions,
parser: require('@typescript-eslint/parser')
},
- errors: [makeError(1)]
+ errors: [makeError(1, 27, 1, 29)]
},
{
filename: `test.${ext}`,
code: `createApp({})`,
languageOptions,
- errors: [makeError(1)]
+ errors: [makeError(1, 11, 1, 13)]
},
{
filename: `test.${ext}`,
@@ -205,7 +207,7 @@ function invalidTests(ext) {
// ${ext}
`,
languageOptions,
- errors: [makeError(3)]
+ errors: [makeError(3, 24, 3, 27)]
},
{
filename: `test.${ext}`,
@@ -215,7 +217,7 @@ function invalidTests(ext) {
// ${ext}
`,
languageOptions,
- errors: [makeError(3)]
+ errors: [makeError(3, 24, 3, 27)]
},
{
filename: `test.${ext}`,
@@ -228,7 +230,7 @@ function invalidTests(ext) {
// ${ext}
`,
languageOptions,
- errors: [makeError(6)]
+ errors: [makeError(6, 24, 6, 27)]
},
{
filename: `test.${ext}`,
@@ -240,7 +242,7 @@ function invalidTests(ext) {
// ${ext}
`,
languageOptions,
- errors: [makeError(3), makeError(5)]
+ errors: [makeError(3, 24, 3, 27), makeError(5, 24, 5, 27)]
},
{
filename: `test.${ext}`,
@@ -252,7 +254,7 @@ function invalidTests(ext) {
// ${ext}
`,
languageOptions,
- errors: [makeError(3), makeError(5)]
+ errors: [makeError(3, 28, 3, 31), makeError(5, 24, 5, 27)]
},
{
filename: `test.${ext}`,
@@ -263,7 +265,10 @@ function invalidTests(ext) {
// ${ext}
`,
languageOptions,
- errors: [...(ext === 'js' ? [] : [makeError(2)]), makeError(4)]
+ errors: [
+ ...(ext === 'js' ? [] : [makeError(2, 24, 2, 27)]),
+ makeError(4, 26, 4, 29)
+ ]
},
{
filename: `test.${ext}`,
@@ -274,7 +279,7 @@ function invalidTests(ext) {
// ${ext}
`,
languageOptions,
- errors: [makeError(4)]
+ errors: [makeError(4, 26, 4, 29)]
},
{
filename: `test.${ext}`,
@@ -286,7 +291,7 @@ function invalidTests(ext) {
// ${ext}
`,
languageOptions,
- errors: [makeError(4)]
+ errors: [makeError(4, 13, 4, 16)]
},
{
filename: `test.${ext}`,
@@ -301,7 +306,10 @@ function invalidTests(ext) {
// ${ext}
`,
languageOptions,
- errors: [...(ext === 'js' ? [] : [makeError(3)]), makeError(6)]
+ errors: [
+ ...(ext === 'js' ? [] : [makeError(3, 24, 7, 10)]),
+ makeError(6, 16, 6, 19)
+ ]
},
{
filename: `test.${ext}`,
@@ -318,19 +326,22 @@ function invalidTests(ext) {
// ${ext}
`,
languageOptions,
- errors: [...(ext === 'js' ? [] : [makeError(2)]), makeError(8)]
+ errors: [
+ ...(ext === 'js' ? [] : [makeError(2, 24, 10, 10)]),
+ makeError(8, 20, 8, 22)
+ ]
},
{
filename: `test.${ext}`,
code: `export default defineComponent({})`,
languageOptions,
- errors: [makeError(1)]
+ errors: [makeError(1, 32, 1, 34)]
},
{
filename: `test.${ext}`,
code: `export default defineNuxtComponent({})`,
languageOptions,
- errors: [makeError(1)]
+ errors: [makeError(1, 36, 1, 38)]
}
]
}
@@ -353,13 +364,13 @@ ruleTester.run('vue-component', rule, {
filename: 'test.vue',
code: `export default { }`,
languageOptions,
- errors: [makeError(1)]
+ errors: [makeError(1, 16, 1, 19)]
},
{
filename: 'test.jsx',
code: `export default { }`,
languageOptions,
- errors: [makeError(1)]
+ errors: [makeError(1, 16, 1, 19)]
},
...invalidTests('js'),
...invalidTests('jsx'),
diff --git a/tests/test-utils/eslint-stylistic.js b/tests/test-utils/eslint-stylistic.js
new file mode 100644
index 000000000..81fc774f3
--- /dev/null
+++ b/tests/test-utils/eslint-stylistic.js
@@ -0,0 +1,16 @@
+const { existsSync, readFileSync } = require('node:fs')
+const path = require('node:path')
+
+const eslintStylisticPackagePath = path.join(
+ __dirname,
+ '../..',
+ 'node_modules',
+ '@stylistic',
+ 'eslint-plugin',
+ 'package.json'
+)
+const eslintStylisticVersion = existsSync(eslintStylisticPackagePath)
+ ? JSON.parse(readFileSync(eslintStylisticPackagePath, 'utf8')).version
+ : undefined
+
+module.exports = { eslintStylisticVersion }
diff --git a/tools/update-html-resources.js b/tools/update-html-resources.js
index fcbdacbb0..aba02d13f 100644
--- a/tools/update-html-resources.js
+++ b/tools/update-html-resources.js
@@ -21,9 +21,8 @@ async function main() {
// Update deprecated-html-elements.json
// ------------------------------------------------------------------------------
function updateDeprecatedHTMLElements() {
- const DEPRECATED_HTML_ELEMENTS_PATH = require.resolve(
- '../lib/utils/deprecated-html-elements.json'
- )
+ const DEPRECATED_HTML_ELEMENTS_PATH =
+ require.resolve('../lib/utils/deprecated-html-elements.json')
const elements = new Set()
const domDl = jsdom.JSDOM.fragment(obsoleteHtml).querySelector(
@@ -49,9 +48,8 @@ async function main() {
// Update html-elements.json
// ------------------------------------------------------------------------------
function updateHTMLElements() {
- const HTML_ELEMENTS_PATH = require.resolve(
- '../lib/utils/html-elements.json'
- )
+ const HTML_ELEMENTS_PATH =
+ require.resolve('../lib/utils/html-elements.json')
const elements = new Set()
const deprecatedHtmlElements = new Set(
require('../lib/utils/deprecated-html-elements.json')
diff --git a/tools/update-lib-flat-configs.js b/tools/update-lib-flat-configs.js
index e55fb2d01..bc103cad1 100644
--- a/tools/update-lib-flat-configs.js
+++ b/tools/update-lib-flat-configs.js
@@ -60,7 +60,7 @@ module.exports = [
name: 'vue/base/setup',
plugins: {
get vue() {
- return require('../../index')
+ return require('../../plugin')
}
},
languageOptions: {
@@ -72,7 +72,7 @@ module.exports = [
files: ['*.vue', '**/*.vue'],
plugins: {
get vue() {
- return require('../../index')
+ return require('../../plugin')
}
},
languageOptions: {
diff --git a/tools/update-lib-index.js b/tools/update-lib-index.js
deleted file mode 100644
index 5e9dd6187..000000000
--- a/tools/update-lib-index.js
+++ /dev/null
@@ -1,76 +0,0 @@
-/**
- * @author Toru Nagashima
- * @copyright 2017 Toru Nagashima. All rights reserved.
- * See LICENSE file in root directory for full license.
- */
-'use strict'
-
-/*
-This script updates `lib/index.js` file from rule's meta data.
-*/
-
-const fs = require('fs')
-const path = require('path')
-const { FlatESLint } = require('eslint/use-at-your-own-risk')
-const rules = require('./lib/rules')
-
-// Update files.
-const filePath = path.resolve(__dirname, '../lib/index.js')
-const content = `/*
- * IMPORTANT!
- * This file has been automatically generated,
- * in order to update its content execute "npm run update"
- */
-'use strict'
-
-const plugin = {
- meta: require('./meta'),
- configs: {
- // eslintrc configs
- base: require('./configs/base'),
-
- 'vue2-essential': require('./configs/vue2-essential'),
- 'vue2-strongly-recommended': require('./configs/vue2-strongly-recommended'),
- 'vue2-recommended': require('./configs/vue2-recommended'),
-
- essential: require('./configs/vue3-essential'),
- 'strongly-recommended': require('./configs/vue3-strongly-recommended'),
- recommended: require('./configs/vue3-recommended'),
-
- // flat configs
- 'flat/base': require('./configs/flat/base.js'),
-
- 'flat/vue2-essential': require('./configs/flat/vue2-essential.js'),
- 'flat/vue2-strongly-recommended': require('./configs/flat/vue2-strongly-recommended.js'),
- 'flat/vue2-recommended': require('./configs/flat/vue2-recommended.js'),
-
- 'flat/essential': require('./configs/flat/vue3-essential.js'),
- 'flat/strongly-recommended': require('./configs/flat/vue3-strongly-recommended.js'),
- 'flat/recommended': require('./configs/flat/vue3-recommended.js'),
-
- // config-format-agnostic configs
- 'no-layout-rules': require('./configs/no-layout-rules'),
- },
- rules: {
- ${rules
- .map((rule) => `'${rule.name}': require('./rules/${rule.name}')`)
- .join(',\n')}
- },
- processors: {
- '.vue': require('./processor'),
- 'vue': require('./processor')
- }
-}
-
-module.exports = plugin
-`
-fs.writeFileSync(filePath, content)
-
-// Format files.
-async function format() {
- const linter = new FlatESLint({ fix: true })
- const report = await linter.lintFiles([filePath])
- FlatESLint.outputFixes(report)
-}
-
-format()
diff --git a/tools/update-lib-plugin.js b/tools/update-lib-plugin.js
new file mode 100644
index 000000000..b9fa03870
--- /dev/null
+++ b/tools/update-lib-plugin.js
@@ -0,0 +1,50 @@
+/**
+ * @author Toru Nagashima
+ * @copyright 2017 Toru Nagashima. All rights reserved.
+ * See LICENSE file in root directory for full license.
+ */
+'use strict'
+
+/*
+This script updates `lib/plugin.js` file from rule's meta data.
+*/
+
+const fs = require('fs')
+const path = require('path')
+const { FlatESLint } = require('eslint/use-at-your-own-risk')
+const rules = require('./lib/rules')
+
+// Update files.
+const filePath = path.resolve(__dirname, '../lib/plugin.js')
+const content = `/*
+ * IMPORTANT!
+ * This file has been automatically generated,
+ * in order to update its content execute "npm run update"
+ */
+'use strict'
+
+const plugin = {
+ meta: require('./meta'),
+ rules: {
+ ${rules
+ .map((rule) => `'${rule.name}': require('./rules/${rule.name}')`)
+ .join(',\n')}
+ },
+ processors: {
+ '.vue': require('./processor'),
+ 'vue': require('./processor')
+ }
+}
+
+module.exports = plugin
+`
+fs.writeFileSync(filePath, content)
+
+// Format files.
+async function format() {
+ const linter = new FlatESLint({ fix: true })
+ const report = await linter.lintFiles([filePath])
+ FlatESLint.outputFixes(report)
+}
+
+format()
diff --git a/tools/update.js b/tools/update.js
index 64da1be1b..f1a77c9a0 100644
--- a/tools/update.js
+++ b/tools/update.js
@@ -8,6 +8,6 @@
require('./update-no-layout-rules-config')
require('./update-lib-configs')
require('./update-lib-flat-configs')
-require('./update-lib-index')
+require('./update-lib-plugin')
require('./update-docs')
require('./update-docs-rules-index')
diff --git a/tsconfig.json b/tsconfig.json
index b488a6b93..2008bef78 100644
--- a/tsconfig.json
+++ b/tsconfig.json
@@ -4,6 +4,9 @@
"module": "node16",
"moduleResolution": "Node16",
"lib": ["es2020"],
+ "types": [
+ "vitest/globals"
+ ],
"allowJs": true,
"checkJs": true,
"noEmit": true,
@@ -25,6 +28,7 @@
},
"include": [
"lib/**/*",
+ "tests/**/*.ts",
"typings/eslint-plugin-vue/global.d.ts",
"docs/.vitepress/**/*.ts",
"docs/.vitepress/**/*.mts"
diff --git a/typings/eslint-plugin-vue/util-types/ast/es-ast.ts b/typings/eslint-plugin-vue/util-types/ast/es-ast.ts
index 3205dc74e..010945c5b 100644
--- a/typings/eslint-plugin-vue/util-types/ast/es-ast.ts
+++ b/typings/eslint-plugin-vue/util-types/ast/es-ast.ts
@@ -227,8 +227,7 @@ interface BasePropertyDefinition extends HasParentNode {
static: boolean
parent: ClassBody
}
-export interface PropertyDefinitionNonComputedName
- extends BasePropertyDefinition {
+export interface PropertyDefinitionNonComputedName extends BasePropertyDefinition {
computed: false
key: Identifier | Literal
}
@@ -351,7 +350,7 @@ export interface PrivateIdentifier extends HasParentNode {
}
export interface Literal extends HasParentNode {
type: 'Literal'
- value: string | boolean | null | number | RegExp | BigInt
+ value: string | boolean | null | number | RegExp | bigint
raw: string
regex?: {
pattern: string
@@ -627,8 +626,7 @@ interface BaseAssignmentProperty extends HasParentNode {
value: Pattern
parent: ObjectPattern
}
-export interface AssignmentPropertyNonComputedName
- extends BaseAssignmentProperty {
+export interface AssignmentPropertyNonComputedName extends BaseAssignmentProperty {
computed: false
key: Identifier | Literal
}
diff --git a/typings/eslint-plugin-vue/util-types/parser-services.ts b/typings/eslint-plugin-vue/util-types/parser-services.ts
index 6a9ccc776..5f2b5a24c 100644
--- a/typings/eslint-plugin-vue/util-types/parser-services.ts
+++ b/typings/eslint-plugin-vue/util-types/parser-services.ts
@@ -7,8 +7,7 @@ type TemplateListenerBase = {
[T in keyof VAST.VNodeListenerMap]?: (node: VAST.VNodeListenerMap[T]) => void
}
export interface TemplateListener
- extends TemplateListenerBase,
- eslint.Rule.NodeListener {
+ extends TemplateListenerBase, eslint.Rule.NodeListener {
[key: string]: ((node: VAST.ParamNode) => void) | undefined
}
diff --git a/typings/eslint/index.d.ts b/typings/eslint/index.d.ts
index 0558e5511..f45f216d8 100644
--- a/typings/eslint/index.d.ts
+++ b/typings/eslint/index.d.ts
@@ -18,8 +18,7 @@ export namespace Scope {
scopes: Scope[]
globalScope: Scope | null
acquire(node: VAST.ESNode | VAST.Program, inner?: boolean): Scope | null
- /** @since ESLint v8.38.0 */
- getDeclaredVariables?(node: VAST.ESNode): Variable[]
+ getDeclaredVariables(node: VAST.ESNode): Variable[]
}
interface Scope {
type:
@@ -232,10 +231,8 @@ export class SourceCode /*extends ESLintSourceCode*/ {
getCommentsAfter(nodeOrToken: VNODE.HasLocation): VNODE.Comment[]
getCommentsInside(node: VNODE.HasLocation): VNODE.Comment[]
- /** @since ESLint v8.39.0 */
- markVariableAsUsed?(name: string, node?: VNODE.HasLocation): void
- /** @since ESLint v8.37.0 */
- getScope?(node: VNODE.HasLocation): Scope.Scope
+ markVariableAsUsed(name: string, node?: VNODE.HasLocation): void
+ getScope(node: VNODE.HasLocation): Scope.Scope
}
export namespace SourceCode {
interface Config {
@@ -323,9 +320,9 @@ export namespace Rule {
id: string
options: ESLintRule.RuleContext['options']
settings: { [name: string]: any }
- /** @deprecated removed in ESLint v10? */
+ /** @deprecated removed in ESLint v10 */
parserPath?: string
- /** @deprecated removed in ESLint v10? */
+ /** @deprecated removed in ESLint v10 */
parserOptions?: ESLintLinter.ParserOptions
/** For flat config */
languageOptions?: ESLintLinter.FlatConfig['languageOptions']
@@ -336,22 +333,24 @@ export namespace Rule {
getAncestors?(): VAST.ESNode[]
/** @deprecated removed in ESLint v9 */
getDeclaredVariables?(node: VAST.ESNode): Scope.Variable[]
- getFilename(): string
- /** @since ESLint v8.40.0 */
- filename?: string
+ /** @deprecated removed in ESLint v10 */
+ getFilename?(): string
+ filename: string
+ /** @deprecated removed in ESLint v10 */
+ getPhysicalFilename?(): string
+ physicalFilename: string
/** @deprecated removed in ESLint v9 */
getScope?(): Scope.Scope
- getSourceCode(): SourceCode
- /** @since ESLint v8.40.0 */
- sourceCode?: SourceCode
+ /** @deprecated removed in ESLint v10 */
+ getSourceCode?(): SourceCode
+ sourceCode: SourceCode
/** @deprecated removed in ESLint v9 */
markVariableAsUsed?(name: string): boolean
report(descriptor: ReportDescriptor): void
- // eslint@6 does not have this method.
+ /** @deprecated removed in ESLint v10 */
getCwd?: () => string
- /** @since ESLint v8.40.0 */
- cwd?: string
+ cwd: string
}
type ReportDescriptor =
diff --git a/vitest.config.ts b/vitest.config.mts
similarity index 65%
rename from vitest.config.ts
rename to vitest.config.mts
index eeef751d6..be1e63448 100644
--- a/vitest.config.ts
+++ b/vitest.config.mts
@@ -1,11 +1,8 @@
+import { defineConfig } from 'vitest/config'
-import { defineConfig } from 'vitest/config';
export default defineConfig({
test: {
- include: [
- 'tests/lib/**/*.js',
- 'tests/integrations/**/*.js'
- ],
+ include: ['tests/lib/**/*.js', 'tests/integrations/**/*.js'],
exclude: [
'**/node_modules/**',
'**/dist/**',
@@ -18,15 +15,10 @@ export default defineConfig({
coverage: {
provider: 'v8',
include: ['lib/**/*.js'],
- exclude: [
- 'tests/**',
- 'dist/**',
- 'tools/**',
- 'node_modules/**'
- ],
+ exclude: ['tests/**', 'dist/**', 'tools/**', 'node_modules/**'],
reporter: ['text', 'lcov', 'json-summary', 'html'],
all: true,
reportsDirectory: './coverage'
- },
- },
-});
+ }
+ }
+})