Skip to content

Commit b2655ae

Browse files
committed
Add more tests
1 parent 5075192 commit b2655ae

File tree

1 file changed

+69
-79
lines changed

1 file changed

+69
-79
lines changed

tests/lib/utils/vue-component.js

Lines changed: 69 additions & 79 deletions
Original file line numberDiff line numberDiff line change
@@ -39,98 +39,72 @@ function makeError (line) {
3939
}
4040
}
4141

42-
// ------------------------------------------------------------------------------
43-
// Tests
44-
// ------------------------------------------------------------------------------
45-
46-
const ruleTester = new RuleTester()
47-
ruleTester.run('return-in-computed-property', rule, {
48-
49-
valid: [
50-
{
51-
filename: 'test.js',
52-
code: `export default { }`,
53-
parserOptions
54-
},
55-
{
56-
filename: 'test.js',
57-
code: `const foo = {}`,
58-
parserOptions
59-
},
42+
function notFoundTests (ext) {
43+
return [
6044
{
61-
filename: 'test.js',
45+
filename: `test.${ext}`,
6246
code: `export const foo = {}`,
6347
parserOptions
6448
},
6549
{
66-
filename: 'test.js',
67-
code: `var foo = {}`,
50+
filename: `test.${ext}`,
51+
code: `export var foo = {}`,
6852
parserOptions
6953
},
7054
{
71-
filename: 'test.js',
72-
code: `foo({ })`,
55+
filename: `test.${ext}`,
56+
code: `const foo = {}`,
7357
parserOptions
7458
},
7559
{
76-
filename: 'test.js',
77-
code: `Vue.component('async-example', function (resolve, reject) { })`,
60+
filename: `test.${ext}`,
61+
code: `var foo = {}`,
7862
parserOptions
7963
},
8064
{
81-
filename: 'test.vue',
82-
code: `export const a = { }`,
65+
filename: `test.${ext}`,
66+
code: `let foo = {}`,
8367
parserOptions
8468
},
8569
{
86-
filename: 'test.vue',
70+
filename: `test.${ext}`,
8771
code: `foo({ })`,
8872
parserOptions
8973
},
9074
{
91-
filename: 'test.vue',
75+
filename: `test.${ext}`,
9276
code: `foo(() => { return {} })`,
9377
parserOptions
9478
},
9579
{
96-
filename: 'test.jsx',
80+
filename: `test.${ext}`,
9781
code: `Vue.component('async-example', function (resolve, reject) { })`,
9882
parserOptions
9983
},
10084
{
101-
filename: 'test.jsx',
102-
code: `export const a = { }`,
103-
parserOptions
104-
},
105-
{
106-
filename: 'test.jsx',
107-
code: `foo({ })`,
108-
parserOptions
109-
},
110-
{
111-
filename: 'test.jsx',
112-
code: `foo(() => { return {} })`,
85+
filename: `test.${ext}`,
86+
code: `Vue.component('async-example', function (resolve, reject) { resolve({}) })`,
11387
parserOptions
11488
},
11589
{
116-
filename: 'test.js',
90+
filename: `test.${ext}`,
11791
code: `new Vue({ })`,
11892
parserOptions
11993
},
12094
{
121-
filename: 'test.jsx',
122-
code: `new Vue({ })`,
123-
parserOptions
124-
},
125-
{
126-
filename: 'test.vue',
127-
code: `new Vue({ })`,
95+
filename: `test.${ext}`,
96+
code: `{
97+
foo: {}
98+
}`,
12899
parserOptions
129100
}
130-
],
131-
invalid: [
101+
]
102+
}
103+
104+
function foundTests (ext) {
105+
return [
132106
{
133-
filename: 'test.vue',
107+
filename: `test.${ext}`,
134108
code: `Vue.component('async-example', function (resolve, reject) {
135109
// @vue/component
136110
resolve({})
@@ -139,39 +113,27 @@ ruleTester.run('return-in-computed-property', rule, {
139113
errors: [makeError(2)]
140114
},
141115
{
142-
filename: 'test.js',
116+
filename: `test.${ext}`,
143117
code: `Vue.component({})`,
144118
parserOptions,
145119
errors: [makeError(1)]
146120
},
147121
{
148-
filename: 'test.vue',
149-
code: `export default { }`,
150-
parserOptions,
151-
errors: [makeError(1)]
152-
},
153-
{
154-
filename: 'test.jsx',
155-
code: `export default { }`,
156-
parserOptions,
157-
errors: [makeError(1)]
158-
},
159-
{
160-
filename: 'test.js',
122+
filename: `test.${ext}`,
161123
code: `// @vue/component
162124
export default { }`,
163125
parserOptions,
164126
errors: [makeError(2)]
165127
},
166128
{
167-
filename: 'test.js',
129+
filename: `test.${ext}`,
168130
code: `/* @vue/component */
169131
export default { }`,
170132
parserOptions,
171133
errors: [makeError(2)]
172134
},
173135
{
174-
filename: 'test.js',
136+
filename: `test.${ext}`,
175137
code: `
176138
/*
177139
* @vue/component
@@ -181,7 +143,7 @@ ruleTester.run('return-in-computed-property', rule, {
181143
errors: [makeError(5)]
182144
},
183145
{
184-
filename: 'test.js',
146+
filename: `test.${ext}`,
185147
code: `// @vue/component
186148
export default { }
187149
// @vue/component
@@ -190,7 +152,7 @@ ruleTester.run('return-in-computed-property', rule, {
190152
errors: [makeError(2), makeError(4)]
191153
},
192154
{
193-
filename: 'test.js',
155+
filename: `test.${ext}`,
194156
code: `/* @vue/component */
195157
export const a = { }
196158
/* @vue/component */
@@ -199,23 +161,23 @@ ruleTester.run('return-in-computed-property', rule, {
199161
errors: [makeError(2), makeError(4)]
200162
},
201163
{
202-
filename: 'test.js',
164+
filename: `test.${ext}`,
203165
code: `export default { }
204166
// @vue/component
205167
export let b = { }`,
206168
parserOptions,
207169
errors: [makeError(3)]
208170
},
209171
{
210-
filename: 'test.js',
172+
filename: `test.${ext}`,
211173
code: `let b = { }
212174
// @vue/component
213175
export let b = { }`,
214176
parserOptions,
215177
errors: [makeError(3)]
216178
},
217179
{
218-
filename: 'test.js',
180+
filename: `test.${ext}`,
219181
code: `export var b = { }
220182
// @vue/component
221183
foo({ })
@@ -224,21 +186,19 @@ ruleTester.run('return-in-computed-property', rule, {
224186
errors: [makeError(3)]
225187
},
226188
{
227-
filename: 'test.js',
189+
filename: `test.${ext}`,
228190
code: `foo({ })
229191
export default {
230192
test: {}
231193
// @vue/component
232-
foo: {
233-
234-
}
194+
foo: { }
235195
}
236196
bar({ })`,
237197
parserOptions,
238198
errors: [makeError(5)]
239199
},
240200
{
241-
filename: 'test.js',
201+
filename: `test.${ext}`,
242202
code: `export default {
243203
bar () {
244204
return {}
@@ -252,4 +212,34 @@ ruleTester.run('return-in-computed-property', rule, {
252212
errors: [makeError(7)]
253213
}
254214
]
215+
}
216+
217+
// ------------------------------------------------------------------------------
218+
// Tests
219+
// ------------------------------------------------------------------------------
220+
221+
const ruleTester = new RuleTester()
222+
ruleTester.run('return-in-computed-property', rule, {
223+
224+
valid: [
225+
{
226+
filename: 'test.js',
227+
code: `export default { }`,
228+
parserOptions
229+
}
230+
].concat(notFoundTests('js')).concat(notFoundTests('jsx')).concat(notFoundTests('vue')),
231+
invalid: [
232+
{
233+
filename: 'test.vue',
234+
code: `export default { }`,
235+
parserOptions,
236+
errors: [makeError(1)]
237+
},
238+
{
239+
filename: 'test.jsx',
240+
code: `export default { }`,
241+
parserOptions,
242+
errors: [makeError(1)]
243+
}
244+
].concat(foundTests('js')).concat(foundTests('jsx')).concat(foundTests('vue'))
255245
})

0 commit comments

Comments
 (0)