Merge pull request #1375 from jseminck/more-robust-no-typos-fix

More robust no typos fix

Author: yannickcr

lib/rules/no-typos.js

@@ -117,10 +117,6 @@ module.exports = {
       },
 
       MemberExpression: function(node) {
-        if (node.parent.type !== 'AssignmentExpression') {
-          return;
-        }
-
         const relatedComponent = utils.getRelatedComponent(node);
 
         if (

lib/util/Components.js

@@ -215,7 +215,16 @@ function componentRule(rule, context) {
      * @returns {Boolean} True if the node is explicitly declared as a descendant of a React Component, false if not
      */
     isExplicitComponent: function(node) {
-      const comment = sourceCode.getJSDocComment(node);
+      let comment;
+      // Sometimes the passed node may not have been parsed yet by eslint, and this function call crashes.
+      // Can be removed when eslint sets "parent" property for all nodes on initial AST traversal: https://github.com/eslint/eslint-scope/issues/27
+      // eslint-disable-next-line no-warning-comments
+      // FIXME: Remove try/catch when https://github.com/eslint/eslint-scope/issues/27 is implemented.
+      try {
+        comment = sourceCode.getJSDocComment(node);
+      } catch (e) {
+        comment = null;
+      }
 
       if (comment === null) {
         return false;

tests/lib/rules/no-typos.js

@@ -238,6 +238,15 @@ ruleTester.run('no-typos', rule, {
       '}'
     ].join('\n'),
     parserOptions: parserOptions
+  }, {
+    // PropTypes declared on a component that is detected through JSDoc comments and is
+    // declared AFTER the PropTypes assignment does not work.
+    code: `
+      MyComponent.PROPTYPES = {}
+      /** @extends React.Component */
+      class MyComponent extends BaseComponent {}
+    `,
+    parserOptions: parserOptions
   }, {
     // https://github.com/yannickcr/eslint-plugin-react/issues/1353
     code: `
@@ -513,6 +522,21 @@ ruleTester.run('no-typos', rule, {
     ].join('\n'),
     parserOptions: parserOptions,
     errors: [{message: ERROR_MESSAGE}]
+  }, {
+    code: [
+      'Component.defaultprops = {}',
+      'class Component extends React.Component {}'
+    ].join('\n'),
+    parserOptions: parserOptions,
+    errors: [{message: ERROR_MESSAGE}]
+  }, {
+    code: `
+      /** @extends React.Component */
+      class MyComponent extends BaseComponent {}
+      MyComponent.PROPTYPES = {}
+    `,
+    parserOptions: parserOptions,
+    errors: [{message: ERROR_MESSAGE}]
   }, {
     code: [
       'class Hello extends React.Component {',