jsx-uses-react: support shorthand fragments

Author: alexzherdev

lib/rules/jsx-uses-react.js

@@ -25,16 +25,16 @@ module.exports = {
   create: function(context) {
     const pragma = pragmaUtil.getFromContext(context);
 
+    function handleOpeningElement() {
+      context.markVariableAsUsed(pragma);
+    }
     // --------------------------------------------------------------------------
     // Public
     // --------------------------------------------------------------------------
 
     return {
-
-      JSXOpeningElement: function() {
-        context.markVariableAsUsed(pragma);
-      }
-
+      JSXOpeningElement: handleOpeningElement,
+      JSXOpeningFragment: handleOpeningElement
     };
   }
 };

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

@@ -39,7 +39,8 @@ ruleTester.run('no-unused-vars', rule, {
     {code: '/*eslint jsx-uses-react:1*/ var React; <div />;'},
     {code: '/*eslint jsx-uses-react:1*/ var React; (function () { <div /> })();'},
     {code: '/*eslint jsx-uses-react:1*/ /** @jsx Foo */ var Foo; <div />;'},
-    {code: '/*eslint jsx-uses-react:1*/ var Foo; <div />;', settings: settings}
+    {code: '/*eslint jsx-uses-react:1*/ var Foo; <div />;', settings: settings},
+    {code: '/*eslint jsx-uses-react:1*/ var React; <></>;', parser: 'babel-eslint'}
   ],
   invalid: [{
     code: '/*eslint jsx-uses-react:1*/ var React;',
@@ -49,6 +50,12 @@ ruleTester.run('no-unused-vars', rule, {
     errors: [{message: '\'React\' is defined but never used.'}]
   }, {
     code: '/*eslint jsx-uses-react:1*/ var React; <div />;',
-    errors: [{message: '\'React\' is defined but never used.'}], settings: settings
+    errors: [{message: '\'React\' is defined but never used.'}],
+    settings: settings
+  }, {
+    code: '/*eslint jsx-uses-react:1*/ var React; <></>;',
+    parser: 'babel-eslint',
+    errors: [{message: '\'React\' is defined but never used.'}],
+    settings: settings
   }]
 });