Merge pull request #69 from burnnat/fix-jsx-props

Only mark component types as used, not prop names.

Author: yannickcr

lib/rules/jsx-uses-vars.js

@@ -20,8 +20,8 @@ module.exports = function(context) {
       variableUtil.markVariableAsUsed(context, node.expression.name);
     },
 
-    JSXIdentifier: function(node) {
-      variableUtil.markVariableAsUsed(context, node.name);
+    JSXOpeningElement: function(node) {
+      variableUtil.markVariableAsUsed(context, node.name.name);
     }
 
   };

tests/lib/rules/jsx-uses-vars.js

@@ -65,7 +65,18 @@ eslintTester.addRuleTest('node_modules/eslint/lib/rules/no-unused-vars', {
   invalid: [
     {
       code: '/*eslint jsx-uses-vars:1*/ var App;',
-      errors: [{message: 'App is defined but never used'}], ecmaFeatures: {jsx: true}
+      errors: [{message: 'App is defined but never used'}],
+      ecmaFeatures: {jsx: true}
+    }, {
+      code: '\
+        /*eslint jsx-uses-vars:1*/\
+        var App;\
+        var unused;\
+        React.render(<App unused=""/>);',
+      errors: [{message: 'unused is defined but never used'}],
+      ecmaFeatures: {
+        jsx: true
+      }
     }
   ]
 });