Skip to content

Commit 37ded33

Browse files
author
Yannick Croissant
committed
Fix new indent errors found by ESLint
1 parent 8b4b0aa commit 37ded33

9 files changed

+101
-82
lines changed

‎tests/lib/rules/display-name.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -378,8 +378,7 @@ ruleTester.run('display-name', rule, {
378378
')'
379379
].join('\n'),
380380
parser: 'babel-eslint'
381-
}
382-
],
381+
}],
383382

384383
invalid: [{
385384
code: [
@@ -549,5 +548,5 @@ ruleTester.run('display-name', rule, {
549548
errors: [{
550549
message: 'Component definition is missing display name'
551550
}]
552-
}
553-
]});
551+
}]
552+
});

‎tests/lib/rules/jsx-boolean-value.js

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,17 @@ ruleTester.run('jsx-boolean-value', rule, {
2929
{code: '<App foo />;', parserOptions: parserOptions},
3030
{code: '<App foo={true} />;', options: ['always'], parserOptions: parserOptions}
3131
],
32-
invalid: [
33-
{code: '<App foo={true} />;', output: '<App foo />;', options: ['never'],
34-
errors: [{message: 'Value must be omitted for boolean attributes'}], parserOptions: parserOptions},
35-
{code: '<App foo={true} />;', output: '<App foo />;',
36-
errors: [{message: 'Value must be omitted for boolean attributes'}], parserOptions: parserOptions},
37-
{code: '<App foo = {true} />;', output: '<App foo />;',
38-
errors: [{message: 'Value must be omitted for boolean attributes'}], parserOptions: parserOptions},
39-
{code: '<App foo />;', output: '<App foo={true} />;', options: ['always'],
40-
errors: [{message: 'Value must be set for boolean attributes'}], parserOptions: parserOptions}
41-
]
32+
invalid: [{
33+
code: '<App foo={true} />;', output: '<App foo />;', options: ['never'],
34+
errors: [{message: 'Value must be omitted for boolean attributes'}], parserOptions: parserOptions
35+
}, {
36+
code: '<App foo={true} />;', output: '<App foo />;',
37+
errors: [{message: 'Value must be omitted for boolean attributes'}], parserOptions: parserOptions
38+
}, {
39+
code: '<App foo = {true} />;', output: '<App foo />;',
40+
errors: [{message: 'Value must be omitted for boolean attributes'}], parserOptions: parserOptions
41+
}, {
42+
code: '<App foo />;', output: '<App foo={true} />;', options: ['always'],
43+
errors: [{message: 'Value must be set for boolean attributes'}], parserOptions: parserOptions
44+
}]
4245
});

‎tests/lib/rules/jsx-key.js

Lines changed: 25 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -37,29 +37,29 @@ ruleTester.run('jsx-key', rule, {
3737
{code: '[1, 2, 3].map(function(x) { return; });', parserOptions: parserOptions},
3838
{code: 'foo(() => <div />);', parserOptions: parserOptions}
3939
],
40-
invalid: [
41-
{code: '[<App />];',
42-
errors: [{message: 'Missing "key" prop for element in array'}],
43-
parserOptions: parserOptions},
44-
45-
{code: '[<App {...key} />];',
46-
errors: [{message: 'Missing "key" prop for element in array'}],
47-
parserOptions: parserOptions},
48-
49-
{code: '[<App key={0}/>, <App />];',
50-
errors: [{message: 'Missing "key" prop for element in array'}],
51-
parserOptions: parserOptions},
52-
53-
{code: '[1, 2 ,3].map(function(x) { return <App /> });',
54-
errors: [{message: 'Missing "key" prop for element in iterator'}],
55-
parserOptions: parserOptions},
56-
57-
{code: '[1, 2 ,3].map(x => <App />);',
58-
errors: [{message: 'Missing "key" prop for element in iterator'}],
59-
parserOptions: parserOptions},
60-
61-
{code: '[1, 2 ,3].map(x => { return <App /> });',
62-
errors: [{message: 'Missing "key" prop for element in iterator'}],
63-
parserOptions: parserOptions}
64-
]
40+
invalid: [{
41+
code: '[<App />];',
42+
errors: [{message: 'Missing "key" prop for element in array'}],
43+
parserOptions: parserOptions
44+
}, {
45+
code: '[<App {...key} />];',
46+
errors: [{message: 'Missing "key" prop for element in array'}],
47+
parserOptions: parserOptions
48+
}, {
49+
code: '[<App key={0}/>, <App />];',
50+
errors: [{message: 'Missing "key" prop for element in array'}],
51+
parserOptions: parserOptions
52+
}, {
53+
code: '[1, 2 ,3].map(function(x) { return <App /> });',
54+
errors: [{message: 'Missing "key" prop for element in iterator'}],
55+
parserOptions: parserOptions
56+
}, {
57+
code: '[1, 2 ,3].map(x => <App />);',
58+
errors: [{message: 'Missing "key" prop for element in iterator'}],
59+
parserOptions: parserOptions
60+
}, {
61+
code: '[1, 2 ,3].map(x => { return <App /> });',
62+
errors: [{message: 'Missing "key" prop for element in iterator'}],
63+
parserOptions: parserOptions
64+
}]
6565
});

‎tests/lib/rules/jsx-no-target-blank.js

Lines changed: 24 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -35,18 +35,29 @@ ruleTester.run('jsx-no-target-blank', rule, {
3535
{code: '<a target="_blank" rel={relValue}></a>', parserOptions: parserOptions},
3636
{code: '<a target={targetValue} rel="noopener noreferrer"></a>', parserOptions: parserOptions}
3737
],
38-
invalid: [
39-
{code: '<a target="_blank"></a>', parserOptions: parserOptions,
40-
errors: [{message: 'Using target="_blank" without rel="noopener noreferrer" is a security risk:' +
41-
' see https://mathiasbynens.github.io/rel-noopener'}]},
42-
{code: '<a target="_blank" rel=""></a>', parserOptions: parserOptions,
43-
errors: [{message: 'Using target="_blank" without rel="noopener noreferrer" is a security risk:' +
44-
' see https://mathiasbynens.github.io/rel-noopener'}]},
45-
{code: '<a target="_blank" rel="noopenernoreferrer"></a>', parserOptions: parserOptions,
46-
errors: [{message: 'Using target="_blank" without rel="noopener noreferrer" is a security risk:' +
47-
' see https://mathiasbynens.github.io/rel-noopener'}]},
48-
{code: '<a target="_BLANK"></a>', parserOptions: parserOptions,
49-
errors: [{message: 'Using target="_blank" without rel="noopener noreferrer" is a security risk:' +
50-
' see https://mathiasbynens.github.io/rel-noopener'}]}
38+
invalid: [{
39+
code: '<a target="_blank"></a>', parserOptions: parserOptions,
40+
errors: [{
41+
message: 'Using target="_blank" without rel="noopener noreferrer" is a security risk:' +
42+
' see https://mathiasbynens.github.io/rel-noopener'
43+
}]
44+
}, {
45+
code: '<a target="_blank" rel=""></a>', parserOptions: parserOptions,
46+
errors: [{
47+
message: 'Using target="_blank" without rel="noopener noreferrer" is a security risk:' +
48+
' see https://mathiasbynens.github.io/rel-noopener'
49+
}]
50+
}, {
51+
code: '<a target="_blank" rel="noopenernoreferrer"></a>', parserOptions: parserOptions,
52+
errors: [{
53+
message: 'Using target="_blank" without rel="noopener noreferrer" is a security risk:' +
54+
' see https://mathiasbynens.github.io/rel-noopener'
55+
}]
56+
}, {
57+
code: '<a target="_BLANK"></a>', parserOptions: parserOptions,
58+
errors: [{
59+
message: 'Using target="_blank" without rel="noopener noreferrer" is a security risk:' +
60+
' see https://mathiasbynens.github.io/rel-noopener'
61+
}]}
5162
]
5263
});

‎tests/lib/rules/jsx-uses-react.js

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,14 @@ ruleTester.run('no-unused-vars', rule, {
3939
{code: '/*eslint jsx-uses-react:1*/ /** @jsx Foo */ var Foo; <div />;', parserOptions: parserOptions},
4040
{code: '/*eslint jsx-uses-react:1*/ var Foo; <div />;', settings: settings, parserOptions: parserOptions}
4141
],
42-
invalid: [
43-
{code: '/*eslint jsx-uses-react:1*/ var React;',
44-
errors: [{message: '\'React\' is defined but never used.'}], parserOptions: parserOptions},
45-
{code: '/*eslint jsx-uses-react:1*/ /** @jsx Foo */ var React; <div />;',
46-
errors: [{message: '\'React\' is defined but never used.'}], parserOptions: parserOptions},
47-
{code: '/*eslint jsx-uses-react:1*/ var React; <div />;',
48-
errors: [{message: '\'React\' is defined but never used.'}], settings: settings, parserOptions: parserOptions}
49-
]
42+
invalid: [{
43+
code: '/*eslint jsx-uses-react:1*/ var React;',
44+
errors: [{message: '\'React\' is defined but never used.'}], parserOptions: parserOptions
45+
}, {
46+
code: '/*eslint jsx-uses-react:1*/ /** @jsx Foo */ var React; <div />;',
47+
errors: [{message: '\'React\' is defined but never used.'}], parserOptions: parserOptions
48+
}, {
49+
code: '/*eslint jsx-uses-react:1*/ var React; <div />;',
50+
errors: [{message: '\'React\' is defined but never used.'}], settings: settings, parserOptions: parserOptions
51+
}]
5052
});

‎tests/lib/rules/no-string-refs.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,5 +110,5 @@ ruleTester.run('no-refs', rule, {
110110
}, {
111111
message: 'Using string literals in ref attributes is deprecated.'
112112
}]
113-
}
114-
]});
113+
}]
114+
});

‎tests/lib/rules/prefer-es6-class.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,5 +115,5 @@ ruleTester.run('prefer-es6-class', rule, {
115115
errors: [{
116116
message: 'Component should use createClass instead of es6 class'
117117
}]
118-
}
119-
]});
118+
}]
119+
});

‎tests/lib/rules/react-in-jsx-scope.js

Lines changed: 20 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -51,22 +51,26 @@ ruleTester.run('react-in-jsx-scope', rule, {
5151
' }',
5252
'});',
5353
'export default Button;'
54-
].join('\n'),
55-
parserOptions: parserOptions},
54+
].join('\n'), parserOptions: parserOptions},
5655
{code: 'var Foo, App; <App />;', settings: settings, parserOptions: parserOptions}
5756
],
58-
invalid: [
59-
{code: 'var App, a = <App />;',
60-
errors: [{message: '\'React\' must be in scope when using JSX'}], parserOptions: parserOptions},
61-
{code: 'var a = <App />;',
62-
errors: [{message: '\'React\' must be in scope when using JSX'}], parserOptions: parserOptions},
63-
{code: 'var a = <img />;',
64-
errors: [{message: '\'React\' must be in scope when using JSX'}], parserOptions: parserOptions},
65-
{code: '/** @jsx React.DOM */ var a = <img />;',
66-
errors: [{message: '\'React\' must be in scope when using JSX'}], parserOptions: parserOptions},
67-
{code: '/** @jsx Foo.bar */ var React, a = <img />;',
68-
errors: [{message: '\'Foo\' must be in scope when using JSX'}], parserOptions: parserOptions},
69-
{code: 'var React, a = <img />;',
70-
errors: [{message: '\'Foo\' must be in scope when using JSX'}], settings: settings, parserOptions: parserOptions}
71-
]
57+
invalid: [{
58+
code: 'var App, a = <App />;',
59+
errors: [{message: '\'React\' must be in scope when using JSX'}], parserOptions: parserOptions
60+
}, {
61+
code: 'var a = <App />;',
62+
errors: [{message: '\'React\' must be in scope when using JSX'}], parserOptions: parserOptions
63+
}, {
64+
code: 'var a = <img />;',
65+
errors: [{message: '\'React\' must be in scope when using JSX'}], parserOptions: parserOptions
66+
}, {
67+
code: '/** @jsx React.DOM */ var a = <img />;',
68+
errors: [{message: '\'React\' must be in scope when using JSX'}], parserOptions: parserOptions
69+
}, {
70+
code: '/** @jsx Foo.bar */ var React, a = <img />;',
71+
errors: [{message: '\'Foo\' must be in scope when using JSX'}], parserOptions: parserOptions
72+
}, {
73+
code: 'var React, a = <img />;',
74+
errors: [{message: '\'Foo\' must be in scope when using JSX'}], settings: settings, parserOptions: parserOptions
75+
}]
7276
});

‎tests/lib/rules/require-render-return.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -203,5 +203,5 @@ ruleTester.run('require-render-return', rule, {
203203
errors: [{
204204
message: 'Your render method should have return statement'
205205
}]
206-
}
207-
]});
206+
}]
207+
});

0 commit comments

Comments
 (0)