fix: mapmapStateToProps-no-Store bug getting body of a var (#5)

Author: DianaSuvorova

lib/rules/mapStateToProps-no-store.js

@@ -44,7 +44,9 @@ module.exports = function (context) {
     CallExpression(node) {
       if (isReactReduxConnect(node)) {
         const mapStateToProps = node.arguments && node.arguments[0];
-        checkFunction(context, mapStateToProps.body, getFirstParamName(mapStateToProps));
+        if (mapStateToProps.body) {
+          checkFunction(context, mapStateToProps.body, getFirstParamName(mapStateToProps));
+        }
       }
     },
   };

tests/lib/rules/mapStateToProps-no-store.js

@@ -15,6 +15,9 @@ const ruleTester = new RuleTester({ parserOptions });
 
 ruleTester.run('mapStateToProps-no-store', rule, {
   valid: [
+    'export default connect(() => {})(Alert)',
+    'export default connect(() => {})(Alert)',
+    'export default connect(null, null)(Alert)',
     'connect((state) => ({isActive: state.isActive}), null)(App)',
     `connect(
           (state) => {
@@ -41,6 +44,8 @@ ruleTester.run('mapStateToProps-no-store', rule, {
     }`,
     'const mapStateToProps = (state, ownProps) => {}',
     'const mapStateToProps = (state) => {isActive: state.isActive}',
+    `const mapStateToProps = (state, ownProps) => {};
+      connect(mapStateToProps, null)(Alert);`,
   ],
   invalid: [{
     code: 'const mapStateToProps = (state) => state',
@@ -92,5 +97,13 @@ ruleTester.run('mapStateToProps-no-store', rule, {
         message: 'mapStateToProps should not return complete store object',
       },
     ],
+  }, {
+    code: `const mapStateToProps = (state, ownProps) => state;
+      connect(mapStateToProps, null)(Alert);`,
+    errors: [
+      {
+        message: 'mapStateToProps should not return complete store object',
+      },
+    ],
   }],
 });