jsx-no-duplicate-props: TypeError: name.toLowerCase is not a function jsx-eslint#969

- recommended improvement
- add test

Author: marcelmokos

lib/rules/jsx-no-duplicate-props.js

@@ -45,7 +45,11 @@ module.exports = {
 
           let name = decl.name.name;
 
-          if (ignoreCase && typeof name === 'string') {
+          if (typeof name !== 'string') {
+            return;
+          }
+
+          if (ignoreCase) {
             name = name.toLowerCase();
           }
 

tests/lib/rules/jsx-no-duplicate-props.js

@@ -49,7 +49,8 @@ ruleTester.run('jsx-no-duplicate-props', rule, {
     {code: '<App c="a" {...this.props} a="c" b="b" />;'},
     {code: '<App A a />;'},
     {code: '<App A b a />;'},
-    {code: '<App A="a" b="b" B="B" />;'}
+    {code: '<App A="a" b="b" B="B" />;'},
+    {code: '<App a:b="c" />;', options: ignoreCaseArgs}
   ],
   invalid: [
     {code: '<App a a />;', errors: [expectedError]},