Skip to content

Commit

Permalink
Rewrite no-set-state tests with template literals
Browse files Browse the repository at this point in the history
  • Loading branch information
jseminck committed Oct 3, 2017
1 parent f18b857 commit a0056b3
Showing 1 changed file with 78 additions and 78 deletions.
156 changes: 78 additions & 78 deletions tests/lib/rules/no-set-state.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,106 +28,106 @@ const ruleTester = new RuleTester({parserOptions});
ruleTester.run('no-set-state', rule, {

valid: [{
code: [
'var Hello = function() {',
' this.setState({})',
'};'
].join('\n')
code: `
var Hello = function() {
this.setState({})
};
`
}, {
code: [
'var Hello = createReactClass({',
' render: function() {',
' return <div>Hello {this.props.name}</div>;',
' }',
'});'
].join('\n')
code: `
var Hello = createReactClass({
render: function() {
return <div>Hello {this.props.name}</div>;
}
});
`
}, {
code: [
'var Hello = createReactClass({',
' componentDidUpdate: function() {',
' someNonMemberFunction(arg);',
' this.someHandler = this.setState;',
' },',
' render: function() {',
' return <div>Hello {this.props.name}</div>;',
' }',
'});'
].join('\n')
code: `
var Hello = createReactClass({
componentDidUpdate: function() {
someNonMemberFunction(arg);
this.someHandler = this.setState;
},
render: function() {
return <div>Hello {this.props.name}</div>;
}
});
`
}],

invalid: [{
code: [
'var Hello = createReactClass({',
' componentDidUpdate: function() {',
' this.setState({',
' name: this.props.name.toUpperCase()',
' });',
' },',
' render: function() {',
' return <div>Hello {this.state.name}</div>;',
' }',
'});'
].join('\n'),
code: `
var Hello = createReactClass({
componentDidUpdate: function() {
this.setState({
name: this.props.name.toUpperCase()
});
},
render: function() {
return <div>Hello {this.state.name}</div>;
}
});
`,
errors: [{
message: 'Do not use setState'
}]
}, {
code: [
'var Hello = createReactClass({',
' someMethod: function() {',
' this.setState({',
' name: this.props.name.toUpperCase()',
' });',
' },',
' render: function() {',
' return <div onClick={this.someMethod.bind(this)}>Hello {this.state.name}</div>;',
' }',
'});'
].join('\n'),
code: `
var Hello = createReactClass({
someMethod: function() {
this.setState({
name: this.props.name.toUpperCase()
});
},
render: function() {
return <div onClick={this.someMethod.bind(this)}>Hello {this.state.name}</div>;
}
});
`,
errors: [{
message: 'Do not use setState'
}]
}, {
code: [
'class Hello extends React.Component {',
' someMethod() {',
' this.setState({',
' name: this.props.name.toUpperCase()',
' });',
' }',
' render() {',
' return <div onClick={this.someMethod.bind(this)}>Hello {this.state.name}</div>;',
' }',
'};'
].join('\n'),
code: `
class Hello extends React.Component {
someMethod() {
this.setState({
name: this.props.name.toUpperCase()
});
}
render() {
return <div onClick={this.someMethod.bind(this)}>Hello {this.state.name}</div>;
}
};
`,
errors: [{
message: 'Do not use setState'
}]
}, {
code: [
'class Hello extends React.Component {',
' someMethod = () => {',
' this.setState({',
' name: this.props.name.toUpperCase()',
' });',
' }',
' render() {',
' return <div onClick={this.someMethod.bind(this)}>Hello {this.state.name}</div>;',
' }',
'};'
].join('\n'),
code: `
class Hello extends React.Component {
someMethod = () => {
this.setState({
name: this.props.name.toUpperCase()
});
}
render() {
return <div onClick={this.someMethod.bind(this)}>Hello {this.state.name}</div>;
}
};
`,
parser: 'babel-eslint',
errors: [{
message: 'Do not use setState'
}]
}, {
code: [
'class Hello extends React.Component {',
' render() {',
' return <div onMouseEnter={() => this.setState({dropdownIndex: index})} />;',
' }',
'};'
].join('\n'),
code: `
class Hello extends React.Component {
render() {
return <div onMouseEnter={() => this.setState({dropdownIndex: index})} />;
}
};
`,
parser: 'babel-eslint',
errors: [{
message: 'Do not use setState'
Expand Down

0 comments on commit a0056b3

Please sign in to comment.