Skip to content

Commit 6ae315b

Browse files
[fix] fix test case
1 parent 00921f7 commit 6ae315b

File tree

1 file changed

+20
-38
lines changed

1 file changed

+20
-38
lines changed

tests/lib/rules/destructuring-assignment.js

Lines changed: 20 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ const parserOptions = {
1717
}
1818
};
1919

20-
const ruleTester = new RuleTester({ parserOptions });
20+
const ruleTester = new RuleTester({parserOptions});
2121
ruleTester.run('destructuring-assignment', rule, {
2222
valid: [{
2323
code: `const MyComponent = ({ id, className }) => (
@@ -163,7 +163,7 @@ ruleTester.run('destructuring-assignment', rule, {
163163
bar = this.props.bar
164164
}
165165
`,
166-
options: ['always', { ignoreClassFields: true }],
166+
options: ['always', {ignoreClassFields: true}],
167167
parser: parsers.BABEL_ESLINT
168168
}, {
169169
code: [
@@ -174,7 +174,7 @@ ruleTester.run('destructuring-assignment', rule, {
174174
' }',
175175
'}'
176176
].join('\n'),
177-
options: ['always', { ignoreClassFields: true }],
177+
options: ['always', {ignoreClassFields: true}],
178178
parser: parsers.BABEL_ESLINT
179179
}, {
180180
code: `const MyComponent = (props) => {
@@ -194,23 +194,23 @@ ruleTester.run('destructuring-assignment', rule, {
194194
return (<div id={props.id} />)
195195
};`,
196196
errors: [
197-
{ message: 'Must use destructuring props assignment' }
197+
{message: 'Must use destructuring props assignment'}
198198
]
199199
}, {
200200
code: `const MyComponent = ({ id, className }) => (
201201
<div id={id} className={className} />
202202
);`,
203203
options: ['never'],
204204
errors: [
205-
{ message: 'Must never use destructuring props assignment in SFC argument' }
205+
{message: 'Must never use destructuring props assignment in SFC argument'}
206206
]
207207
}, {
208208
code: `const MyComponent = (props, { color }) => (
209209
<div id={props.id} className={props.className} />
210210
);`,
211211
options: ['never'],
212212
errors: [
213-
{ message: 'Must never use destructuring context assignment in SFC argument' }
213+
{message: 'Must never use destructuring context assignment in SFC argument'}
214214
]
215215
}, {
216216
code: `const Foo = class extends React.PureComponent {
@@ -219,7 +219,7 @@ ruleTester.run('destructuring-assignment', rule, {
219219
}
220220
};`,
221221
errors: [
222-
{ message: 'Must use destructuring props assignment' }
222+
{message: 'Must use destructuring props assignment'}
223223
]
224224
}, {
225225
code: `const Foo = class extends React.PureComponent {
@@ -228,7 +228,7 @@ ruleTester.run('destructuring-assignment', rule, {
228228
}
229229
};`,
230230
errors: [
231-
{ message: 'Must use destructuring state assignment' }
231+
{message: 'Must use destructuring state assignment'}
232232
]
233233
}, {
234234
code: `const Foo = class extends React.PureComponent {
@@ -237,7 +237,7 @@ ruleTester.run('destructuring-assignment', rule, {
237237
}
238238
};`,
239239
errors: [
240-
{ message: 'Must use destructuring context assignment' }
240+
{message: 'Must use destructuring context assignment'}
241241
]
242242
}, {
243243
code: `class Foo extends React.Component {
@@ -247,7 +247,7 @@ ruleTester.run('destructuring-assignment', rule, {
247247
}
248248
}`,
249249
errors: [
250-
{ message: 'Must use destructuring props assignment' }
250+
{message: 'Must use destructuring props assignment'}
251251
]
252252
}, {
253253
code: `var Hello = React.createClass({
@@ -256,7 +256,7 @@ ruleTester.run('destructuring-assignment', rule, {
256256
}
257257
});`,
258258
errors: [
259-
{ message: 'Must use destructuring props assignment' }
259+
{message: 'Must use destructuring props assignment'}
260260
]
261261
}, {
262262
code: `
@@ -266,21 +266,21 @@ ruleTester.run('destructuring-assignment', rule, {
266266
}
267267
}
268268
`,
269-
errors: [{ message: 'Must use destructuring props assignment' }]
269+
errors: [{message: 'Must use destructuring props assignment'}]
270270
}, {
271271
code: `
272272
export default function Foo(props) {
273273
return <p>{props.a}</p>;
274274
}
275275
`,
276-
errors: [{ message: 'Must use destructuring props assignment' }]
276+
errors: [{message: 'Must use destructuring props assignment'}]
277277
}, {
278278
code: `
279279
function hof() {
280280
return (props) => <p>{props.a}</p>;
281281
}
282282
`,
283-
errors: [{ message: 'Must use destructuring props assignment' }]
283+
errors: [{message: 'Must use destructuring props assignment'}]
284284
}, {
285285
code: `const Foo = class extends React.PureComponent {
286286
render() {
@@ -289,7 +289,7 @@ ruleTester.run('destructuring-assignment', rule, {
289289
}
290290
};`,
291291
errors: [
292-
{ message: 'Must use destructuring props assignment' }
292+
{message: 'Must use destructuring props assignment'}
293293
]
294294
}, {
295295
code: `const Foo = class extends React.PureComponent {
@@ -301,7 +301,7 @@ ruleTester.run('destructuring-assignment', rule, {
301301
options: ['never'],
302302
parser: parsers.BABEL_ESLINT,
303303
errors: [
304-
{ message: 'Must never use destructuring props assignment' }
304+
{message: 'Must never use destructuring props assignment'}
305305
]
306306
}, {
307307
code: `const MyComponent = (props) => {
@@ -311,7 +311,7 @@ ruleTester.run('destructuring-assignment', rule, {
311311
options: ['never'],
312312
parser: parsers.BABEL_ESLINT,
313313
errors: [
314-
{ message: 'Must never use destructuring props assignment' }
314+
{message: 'Must never use destructuring props assignment'}
315315
]
316316
}, {
317317
code: `const Foo = class extends React.PureComponent {
@@ -323,7 +323,7 @@ ruleTester.run('destructuring-assignment', rule, {
323323
options: ['never'],
324324
parser: parsers.BABEL_ESLINT,
325325
errors: [
326-
{ message: 'Must never use destructuring state assignment' }
326+
{message: 'Must never use destructuring state assignment'}
327327
]
328328
}, {
329329
code: `const MyComponent = (props) => {
@@ -332,7 +332,7 @@ ruleTester.run('destructuring-assignment', rule, {
332332
};`,
333333
options: ['always'],
334334
errors: [
335-
{ message: 'Must use destructuring foo assignment' }
335+
{message: 'Must use destructuring foo assignment'}
336336
]
337337
}, {
338338
code: `const MyComponent = (props) => {
@@ -341,25 +341,7 @@ ruleTester.run('destructuring-assignment', rule, {
341341
};`,
342342
options: ['never'],
343343
errors: [
344-
{ message: 'Must never use destructuring useContext assignment' }
345-
]
346-
}, {
347-
code: `const MyComponent = (props) => {
348-
const foo = useContext(aContext);
349-
return <div>{foo.test}</div>
350-
};`,
351-
options: ['always'],
352-
errors: [
353-
{ message: 'Must use destructuring foo assignment' }
354-
]
355-
}, {
356-
code: `const MyComponent = (props) => {
357-
const {foo} = useContext(aContext);
358-
return <div>{foo}</div>
359-
};`,
360-
options: ['never'],
361-
errors: [
362-
{ message: 'Must never use destructuring useContext assignment' }
344+
{message: 'Must never use destructuring useContext assignment'}
363345
]
364346
}]
365347
});

0 commit comments

Comments
 (0)