[Fix] no-typos: Compilation error when method name is string instead of identifier

Author: 李凤宝

lib/rules/no-typos.js

@@ -144,7 +144,11 @@ module.exports = {
 
     function reportErrorIfLifecycleMethodCasingTypo(node) {
       LIFECYCLE_METHODS.forEach((method) => {
-        if (method.toLowerCase() === node.key.name.toLowerCase() && method !== node.key.name) {
+        let nodeKeyName = node.key.name;
+        if (node.key.type === 'Literal') {
+          nodeKeyName = node.key.value;
+        }
+        if (method.toLowerCase() === nodeKeyName.toLowerCase() && method !== nodeKeyName) {
           context.report({
             node,
             message: 'Typo in component lifecycle method declaration'

tests/lib/rules/no-typos.js

@@ -203,6 +203,14 @@ ruleTester.run('no-typos', rule, {
       }
     `,
     parserOptions
+  }, {
+    code: `
+      class Hello extends React.Component {
+        "componentDidMount"() { }
+        "my-method"() { }
+      }
+    `,
+    parserOptions
   }, {
     code: `
       class MyClass {