Fixes #1369

Author: Ev Haus

lib/rules/boolean-prop-naming.js

@@ -1,6 +1,6 @@
 /**
  * @fileoverview Enforces consistent naming for boolean props
- * @author Evgueni Naverniouk
+ * @author Ev Haus
  */
 'use strict';
 
@@ -57,7 +57,7 @@ module.exports = {
     function isPropTypesDeclaration(node) {
       // Special case for class properties
       // (babel-eslint does not expose property name so we have to rely on tokens)
-      if (node.type === 'ClassProperty') {
+      if (node && node.type === 'ClassProperty') {
         const tokens = context.getFirstTokens(node, 2);
         if (tokens[0].value === 'propTypes' || (tokens[1] && tokens[1].value === 'propTypes')) {
           return true;

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

@@ -1,6 +1,6 @@
 /**
  * @fileoverview Enforces consistent naming for boolean props
- * @author Evgueni Naverniouk
+ * @author Ev Haus
  */
 'use strict';
 
@@ -247,6 +247,16 @@ ruleTester.run('boolean-prop-naming', rule, {
       rule: '^is[A-Z]([A-Za-z0-9]?)+'
     }],
     parser: 'babel-eslint'
+  }, {
+    // Ensure rule doesn't crash on destructured objects [Issue #1369]
+    code: [
+      'var x = {a: 1}',
+      'var y = {...x}'
+    ].join('\n'),
+    options: [{
+      rule: '^is[A-Z]([A-Za-z0-9]?)+'
+    }],
+    parser: 'babel-eslint'
   }],
 
   invalid: [{