Fix prop-types crash when propTypes definition is invalid (fixes #95)

Author: Yannick Croissant

lib/rules/prop-types.js

@@ -162,7 +162,7 @@ module.exports = function(context) {
     var declaredPropTypes = component && component.declaredPropTypes || [];
     var ignorePropsValidation = false;
 
-    switch (propTypes.type) {
+    switch (propTypes && propTypes.type) {
       case 'ObjectExpression':
         for (var i = 0, j = propTypes.properties.length; i < j; i++) {
           var key = propTypes.properties[i].key;
@@ -172,6 +172,8 @@ module.exports = function(context) {
       case 'MemberExpression':
         declaredPropTypes.push(propTypes.property.name);
         break;
+      case null:
+        break;
       default:
         ignorePropsValidation = true;
         break;

tests/lib/rules/prop-types.js

@@ -359,6 +359,25 @@ eslintTester.addRuleTest('lib/rules/prop-types', {
       errors: [{
         message: '\'lastname\' is missing in props validation for Hello'
       }]
+    }, {
+      code: [
+        'class Hello extends React.Component {',
+        '  static propTypes: { ',
+        '    firstname: React.PropTypes.string',
+        '  }',
+        '  render() {',
+        '    return <div>Hello {this.props.firstname}</div>;',
+        '  }',
+        '}'
+      ].join('\n'),
+      parser: 'babel-eslint',
+      ecmaFeatures: {
+        classes: true,
+        jsx: true
+      },
+      errors: [{
+        message: '\'firstname\' is missing in props validation for Hello'
+      }]
     }
   ]
 });