Skip to content

Commit 46b23bd

Browse files
committed
Fix tests for eslint 4.x
1 parent 284c99f commit 46b23bd

File tree

4 files changed

+44
-148
lines changed

4 files changed

+44
-148
lines changed

tests/lib/rules/no-color-literals.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -181,9 +181,11 @@ const tests = {
181181

182182
const config = {
183183
parser: 'babel-eslint',
184-
ecmaFeatures: {
185-
classes: true,
186-
jsx: true,
184+
parserOptions: {
185+
ecmaFeatures: {
186+
classes: true,
187+
jsx: true,
188+
},
187189
},
188190
};
189191

tests/lib/rules/no-inline-styles.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -182,9 +182,11 @@ const tests = {
182182

183183
const config = {
184184
parser: 'babel-eslint',
185-
ecmaFeatures: {
186-
classes: true,
187-
jsx: true,
185+
parserOptions: {
186+
ecmaFeatures: {
187+
classes: true,
188+
jsx: true,
189+
},
188190
},
189191
};
190192

tests/lib/rules/no-unused-styles.js

Lines changed: 17 additions & 69 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,7 @@ require('babel-eslint');
1919
// ------------------------------------------------------------------------------
2020

2121
const ruleTester = new RuleTester();
22-
ruleTester.run('no-unused-styles', rule, {
23-
22+
const tests = {
2423
valid: [{
2524
code: [
2625
'const styles = StyleSheet.create({',
@@ -32,11 +31,6 @@ ruleTester.run('no-unused-styles', rule, {
3231
' }',
3332
'});',
3433
].join('\n'),
35-
parser: 'babel-eslint',
36-
ecmaFeatures: {
37-
classes: true,
38-
jsx: true,
39-
},
4034
}, {
4135
code: [
4236
'const Hello = React.createClass({',
@@ -48,11 +42,6 @@ ruleTester.run('no-unused-styles', rule, {
4842
' name: {}',
4943
'});',
5044
].join('\n'),
51-
parser: 'babel-eslint',
52-
ecmaFeatures: {
53-
classes: true,
54-
jsx: true,
55-
},
5645
}, {
5746
code: [
5847
'const styles = StyleSheet.create({',
@@ -64,11 +53,6 @@ ruleTester.run('no-unused-styles', rule, {
6453
' }',
6554
'});',
6655
].join('\n'),
67-
parser: 'babel-eslint',
68-
ecmaFeatures: {
69-
classes: true,
70-
jsx: true,
71-
},
7256
}, {
7357
code: [
7458
'const styles = StyleSheet.create({',
@@ -86,11 +70,6 @@ ruleTester.run('no-unused-styles', rule, {
8670
' }',
8771
'});',
8872
].join('\n'),
89-
parser: 'babel-eslint',
90-
ecmaFeatures: {
91-
classes: true,
92-
jsx: true,
93-
},
9473
}, {
9574
code: [
9675
'const styles = StyleSheet.create({',
@@ -105,11 +84,6 @@ ruleTester.run('no-unused-styles', rule, {
10584
' }',
10685
'});',
10786
].join('\n'),
108-
parser: 'babel-eslint',
109-
ecmaFeatures: {
110-
classes: true,
111-
jsx: true,
112-
},
11387
}, {
11488
code: [
11589
'const styles = StyleSheet.create({',
@@ -131,11 +105,6 @@ ruleTester.run('no-unused-styles', rule, {
131105
' }',
132106
'});',
133107
].join('\n'),
134-
parser: 'babel-eslint',
135-
ecmaFeatures: {
136-
classes: true,
137-
jsx: true,
138-
},
139108
}, {
140109
code: [
141110
'const styles = StyleSheet.create({',
@@ -158,11 +127,6 @@ ruleTester.run('no-unused-styles', rule, {
158127
' }',
159128
'});',
160129
].join('\n'),
161-
parser: 'babel-eslint',
162-
ecmaFeatures: {
163-
classes: true,
164-
jsx: true,
165-
},
166130
}, {
167131
code: [
168132
'const styles = StyleSheet.create({',
@@ -182,11 +146,6 @@ ruleTester.run('no-unused-styles', rule, {
182146
' }',
183147
'});',
184148
].join('\n'),
185-
parser: 'babel-eslint',
186-
ecmaFeatures: {
187-
classes: true,
188-
jsx: true,
189-
},
190149
}, {
191150
code: [
192151
'const styles = StyleSheet.create({',
@@ -206,22 +165,12 @@ ruleTester.run('no-unused-styles', rule, {
206165
' }',
207166
'}',
208167
].join('\n'),
209-
parser: 'babel-eslint',
210-
ecmaFeatures: {
211-
classes: true,
212-
jsx: true,
213-
},
214168
}, {
215169
code: [
216170
'const styles = StyleSheet.create({',
217171
' text: {}',
218172
'})',
219173
].join('\n'),
220-
parser: 'babel-eslint',
221-
ecmaFeatures: {
222-
classes: true,
223-
jsx: true,
224-
},
225174
}, {
226175
code: [
227176
'const Hello = React.createClass({',
@@ -242,11 +191,6 @@ ruleTester.run('no-unused-styles', rule, {
242191
' text2: {},',
243192
'});',
244193
].join('\n'),
245-
parser: 'babel-eslint',
246-
ecmaFeatures: {
247-
classes: true,
248-
jsx: true,
249-
},
250194
}, {
251195
code: [
252196
'const additionalStyles = {};',
@@ -260,12 +204,6 @@ ruleTester.run('no-unused-styles', rule, {
260204
' }',
261205
'});',
262206
].join('\n'),
263-
parser: 'babel-eslint',
264-
ecmaFeatures: {
265-
classes: true,
266-
jsx: true,
267-
spread: true,
268-
},
269207
}],
270208

271209
invalid: [{
@@ -279,13 +217,23 @@ ruleTester.run('no-unused-styles', rule, {
279217
' }',
280218
'});',
281219
].join('\n'),
282-
parser: 'babel-eslint',
283-
ecmaFeatures: {
284-
classes: true,
285-
jsx: true,
286-
},
287220
errors: [{
288221
message: 'Unused style detected: styles.text',
289222
}],
290223
}],
291-
});
224+
};
225+
226+
const config = {
227+
parser: 'babel-eslint',
228+
parserOptions: {
229+
ecmaFeatures: {
230+
classes: true,
231+
jsx: true,
232+
},
233+
},
234+
};
235+
236+
tests.valid.forEach(t => Object.assign(t, config));
237+
tests.invalid.forEach(t => Object.assign(t, config));
238+
239+
ruleTester.run('split-platform-components', rule, tests);

0 commit comments

Comments
 (0)