Skip to content

Commit b9bb564

Browse files
authored
Merge pull request jsx-eslint#1358 from dustinsoftware/issue1354
Fix propTypes wrapper crash in boolean-prop-naming
2 parents dae6574 + 99fd813 commit b9bb564

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

lib/rules/boolean-prop-naming.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ module.exports = {
180180
return;
181181
}
182182
const component = utils.getRelatedComponent(node);
183-
if (!component) {
183+
if (!component || !node.parent.right.properties) {
184184
return;
185185
}
186186
validatePropNaming(component.node, node.parent.right.properties);

tests/lib/rules/boolean-prop-naming.js

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,16 @@ ruleTester.run('boolean-prop-naming', rule, {
101101
options: [{
102102
rule: '^is[A-Z]([A-Za-z0-9]?)+'
103103
}]
104+
}, {
105+
code: [
106+
'class Hello extends React.Component {',
107+
' render () { return <div />; }',
108+
'}',
109+
'Hello.propTypes = wrap({ a: PropTypes.bool })'
110+
].join('\n'),
111+
options: [{
112+
rule: '^is[A-Z]([A-Za-z0-9]?)+'
113+
}]
104114
}, {
105115
// ES6 components as React.Component with non-boolean PropTypes
106116
code: [

0 commit comments

Comments
 (0)