Skip to content

Commit d9d2193

Browse files
chiawendtljharb
chiawendt
authored and
ljharb
committed
[eslint] enable eslint-plugin-eslint-plugin
1 parent 39e4396 commit d9d2193

17 files changed

+63
-150
lines changed

‎.eslintrc‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
{
22
"root": true,
3-
"extends": "airbnb-base",
3+
"extends": ["airbnb-base", "plugin:eslint-plugin/recommended"],
4+
"plugins": ["eslint-plugin"],
45
"env": {
56
"es6": true,
67
"node": true

‎lib/rules/jsx-child-element-spacing.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ module.exports = {
4545
recommended: false,
4646
url: docsUrl('jsx-child-element-spacing')
4747
},
48-
fixable: false,
48+
fixable: null,
4949
schema: [
5050
{
5151
type: 'object',

‎package.json‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
"dependencies": {
2929
"array-includes": "^3.0.3",
3030
"doctrine": "^2.1.0",
31+
"eslint-plugin-eslint-plugin": "^2.1.0",
3132
"has": "^1.0.3",
3233
"jsx-ast-utils": "^2.2.1",
3334
"object.entries": "^1.1.0",

‎tests/lib/rules/default-props-match-prop-types.js‎

Lines changed: 24 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,8 @@ ruleTester.run('default-props-match-prop-types', rule, {
268268
'Greeting.defaultProps = {',
269269
' foo: "foo"',
270270
'};'
271-
].join('\n')
271+
].join('\n'),
272+
parser: parsers.BABEL_ESLINT
272273
},
273274
{
274275
code: [
@@ -530,7 +531,8 @@ ruleTester.run('default-props-match-prop-types', rule, {
530531
' ...defaults,',
531532
' bar: "bar"',
532533
'};'
533-
].join('\n')
534+
].join('\n'),
535+
parser: parsers.BABEL_ESLINT
534536
},
535537

536538
//
@@ -845,6 +847,26 @@ ruleTester.run('default-props-match-prop-types', rule, {
845847
column: 3
846848
}]
847849
},
850+
{
851+
code: [
852+
'function MyStatelessComponent({ foo, bar }) {',
853+
' return <div>{foo}{bar}</div>;',
854+
'}',
855+
'MyStatelessComponent.propTypes = {',
856+
' foo: React.PropTypes.string,',
857+
' bar: React.PropTypes.string.isRequired',
858+
'};',
859+
'MyStatelessComponent.defaultProps = {',
860+
' baz: "baz"',
861+
'};'
862+
].join('\n'),
863+
errors: [{
864+
message: 'defaultProp "baz" has no corresponding propTypes declaration.',
865+
line: 9,
866+
column: 3
867+
}],
868+
parser: parsers.BABEL_ESLINT
869+
},
848870
{
849871
code: [
850872
'function MyStatelessComponent({ foo, bar }) {',
@@ -1348,25 +1370,6 @@ ruleTester.run('default-props-match-prop-types', rule, {
13481370
column: 5
13491371
}]
13501372
},
1351-
{
1352-
code: [
1353-
'function MyStatelessComponent({ foo, bar }) {',
1354-
' return <div>{foo}{bar}</div>;',
1355-
'}',
1356-
'MyStatelessComponent.propTypes = {',
1357-
' foo: React.PropTypes.string,',
1358-
' bar: React.PropTypes.string.isRequired',
1359-
'};',
1360-
'MyStatelessComponent.defaultProps = {',
1361-
' baz: "baz"',
1362-
'};'
1363-
].join('\n'),
1364-
errors: [{
1365-
message: 'defaultProp "baz" has no corresponding propTypes declaration.',
1366-
line: 9,
1367-
column: 3
1368-
}]
1369-
},
13701373
{
13711374
code: [
13721375
'class Greeting extends React.Component {',

‎tests/lib/rules/destructuring-assignment.js‎

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -21,15 +21,6 @@ const parserOptions = {
2121
const ruleTester = new RuleTester({parserOptions});
2222
ruleTester.run('destructuring-assignment', rule, {
2323
valid: [{
24-
code: `const Foo = class extends React.PureComponent {
25-
render() {
26-
const { foo } = this.props;
27-
return <div>{foo}</div>;
28-
}
29-
};`,
30-
options: ['always'],
31-
parser: parsers.BABEL_ESLINT
32-
}, {
3324
code: `const MyComponent = ({ id, className }) => (
3425
<div id={id} className={className} />
3526
);`
@@ -102,15 +93,16 @@ ruleTester.run('destructuring-assignment', rule, {
10293
return <div>{foo}</div>;
10394
}
10495
};`,
105-
options: ['always'],
106-
parser: parsers.BABEL_ESLINT
96+
options: ['always']
10797
}, {
10898
code: `const Foo = class extends React.PureComponent {
10999
render() {
110100
const { foo } = this.props;
111101
return <div>{foo}</div>;
112102
}
113-
};`
103+
};`,
104+
options: ['always'],
105+
parser: parsers.BABEL_ESLINT
114106
}, {
115107
code: `const Foo = class extends React.PureComponent {
116108
render() {
@@ -119,7 +111,7 @@ ruleTester.run('destructuring-assignment', rule, {
119111
}
120112
};`,
121113
options: ['always'],
122-
parser: parsers.BABEL_ESLINT
114+
parser: parsers.TYPESCRIPT_ESLINT
123115
}, {
124116
code: `const MyComponent = (props) => {
125117
const { h, i } = hi;

‎tests/lib/rules/jsx-curly-brace-presence.js‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -508,7 +508,8 @@ ruleTester.run('jsx-curly-brace-presence', rule, {
508508
code: `<MyComponent prop="foo 'bar'">foo</MyComponent>`,
509509
output: `<MyComponent prop={"foo 'bar'"}>foo</MyComponent>`,
510510
options: [{props: 'always'}],
511-
errors: [{message: missingCurlyMessage}]
511+
errors: [{message: missingCurlyMessage}],
512+
parser: parsers.BABEL_ESLINT
512513
},
513514
{
514515
code: '<MyComponent>foo bar </MyComponent>',

‎tests/lib/rules/jsx-curly-spacing.js‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -405,7 +405,8 @@ ruleTester.run('jsx-curly-spacing', rule, {
405405
'...bar',
406406
'} />;'
407407
].join('\n'),
408-
options: [{attributes: {when: 'never'}}]
408+
options: [{attributes: {when: 'never'}}],
409+
parser: parsers.BABEL_ESLINT
409410
}, {
410411
code: [
411412
'<App {',
@@ -486,9 +487,6 @@ ruleTester.run('jsx-curly-spacing', rule, {
486487
}, {
487488
code: '<App foo={ bar }>{bar}</App>',
488489
options: [{attributes: {when: 'always'}}]
489-
}, {
490-
code: '<App foo={ bar }>{bar}</App>',
491-
options: [{attributes: {when: 'always'}}]
492490
}, {
493491
code: [
494492
'<App foo={ 42 } { ...bar } baz={{ 4: 2 }}>',
@@ -630,7 +628,8 @@ ruleTester.run('jsx-curly-spacing', rule, {
630628
'...bar',
631629
'} />;'
632630
].join('\n'),
633-
options: ['always']
631+
options: ['always'],
632+
parser: parsers.BABEL_ESLINT
634633
}, {
635634
code: [
636635
'<App {',

‎tests/lib/rules/jsx-no-undef.js‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,8 @@ ruleTester.run('jsx-no-undef', rule, {
3535
valid: [{
3636
code: '/*eslint no-undef:1*/ var React, App; React.render(<App />);'
3737
}, {
38-
code: '/*eslint no-undef:1*/ var React, App; React.render(<App />);'
38+
code: '/*eslint no-undef:1*/ var React, App; React.render(<App />);',
39+
parser: parsers.BABEL_ESLINT
3940
}, {
4041
code: '/*eslint no-undef:1*/ var React; React.render(<img />);'
4142
}, {

‎tests/lib/rules/jsx-one-expression-per-line.js‎

Lines changed: 2 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -522,23 +522,6 @@ ruleTester.run('jsx-one-expression-per-line', rule, {
522522
].join('\n'),
523523
errors: [{message: '` bar` must be placed on a new line'}],
524524
parserOptions
525-
}, {
526-
code: [
527-
'<div>',
528-
' foo {"bar"}',
529-
'</div>'
530-
].join('\n'),
531-
output: [
532-
'<div>',
533-
' foo ',
534-
'{\' \'}',
535-
'{"bar"}',
536-
'</div>'
537-
].join('\n'),
538-
errors: [
539-
{message: '`{"bar"}` must be placed on a new line'}
540-
],
541-
parserOptions
542525
}, {
543526
code: [
544527
'<div>',
@@ -705,7 +688,8 @@ ruleTester.run('jsx-one-expression-per-line', rule, {
705688
'</App>'
706689
].join('\n'),
707690
errors: [{message: '`Foo` must be placed on a new line'}],
708-
parserOptions
691+
parserOptions,
692+
parser: parsers.BABEL_ESLINT
709693
}, {
710694
code: [
711695
'<App>',

‎tests/lib/rules/jsx-pascal-case.js‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@
1212
const RuleTester = require('eslint').RuleTester;
1313
const rule = require('../../../lib/rules/jsx-pascal-case');
1414

15+
const parsers = require('../../helpers/parsers');
16+
1517
const parserOptions = {
1618
ecmaVersion: 2018,
1719
sourceType: 'module',
@@ -46,6 +48,9 @@ ruleTester.run('jsx-pascal-case', rule, {
4648
code: '<T3stComp0nent />'
4749
}, {
4850
code: '<T />'
51+
}, {
52+
code: '<T />',
53+
parser: parsers.BABEL_ESLINT
4954
}, {
5055
code: '<YMCA />',
5156
options: [{allowAllCaps: true}]
@@ -59,8 +64,6 @@ ruleTester.run('jsx-pascal-case', rule, {
5964
}, {
6065
code: '<IGNORED />',
6166
options: [{ignore: ['IGNORED']}]
62-
}, {
63-
code: '<T />'
6467
}, {
6568
code: '<$ />'
6669
}, {

0 commit comments

Comments
 (0)