fix: issues reported by eslint-remote-tester (#60)

* fix: issues reported by eslint-remote-tester

Author: DianaSuvorova

lib/rules/no-unused-prop-types.js

@@ -10,7 +10,7 @@ const belongsToReduxReact = (node, objectName, destrArg) => {
     const secondArgumentName = secondArgument && secondArgument.type === 'Identifier' && secondArgument.name;
     return (secondArgumentName === objectName // ownProps.myProp
       || destrArg === secondArgument // {myProp} in fn argument
-      || (destrArg && destrArg.parent.type === 'VariableDeclarator' && destrArg.parent.init.name === secondArgumentName) // const {myProp} = ownProps;
+      || (destrArg && destrArg.parent.type === 'VariableDeclarator' && destrArg.parent.init && destrArg.parent.init.name === secondArgumentName) // const {myProp} = ownProps;
     );
   };
   let isReactRedux = false;

lib/rules/useSelector-prefer-selectors.js

@@ -28,7 +28,7 @@ module.exports = function (context) {
       ) {
         reportNoSelector(context, node);
       } else if (
-        selector.type === 'Identifier' &&
+        selector && selector.type === 'Identifier' &&
         config.matching &&
         !selector.name.match(new RegExp(config.matching))
       ) {

tests/code-sanity-samples.js

@@ -0,0 +1,9 @@
+module.exports = [
+  'expect(() => useSelector()).toThrow();',
+  `const rows = [];
+      function mapStateToProps(state, ownProps) {
+        for (const { value } of rows) {
+        }
+      }
+  `,
+];

tests/lib/rules/connect-prefer-minimum-two-arguments.js

@@ -2,6 +2,7 @@ require('babel-eslint');
 
 const rule = require('../../../lib/rules/connect-prefer-minimum-two-arguments');
 const RuleTester = require('eslint').RuleTester;
+const codeSamples = require('../../code-sanity-samples');
 
 const parserOptions = {
   ecmaVersion: 6,
@@ -15,6 +16,7 @@ const ruleTester = new RuleTester({ parserOptions });
 
 ruleTester.run('connect-prefer-minimum-two-arguments', rule, {
   valid: [
+    ...codeSamples,
     'connect(mapStateToProps, mapDispatchToProps, mergeProps, options)(Component)',
     'connect(mapStateToProps, mapDispatchToProps)(Component)',
     'connect({prop1, prop2}, {action1, action2})(Component)',

tests/lib/rules/connect-prefer-named-arguments.js

@@ -2,6 +2,7 @@ require('babel-eslint');
 
 const rule = require('../../../lib/rules/connect-prefer-named-arguments');
 const RuleTester = require('eslint').RuleTester;
+const codeSamples = require('../../code-sanity-samples');
 
 const parserOptions = {
   ecmaVersion: 6,
@@ -15,6 +16,7 @@ const ruleTester = new RuleTester({ parserOptions });
 
 ruleTester.run('connect-prefer-named-arguments', rule, {
   valid: [
+    ...codeSamples,
     'export default connect(null, mapDispatchToProps)(TodoApp)',
     'connect(mapStateToProps, mapDispatchToProps, mergeProps, options)(Component)',
     'connect(mapStateToProps, mapDispatchToProps)(Component)',

tests/lib/rules/mapDispatchToProps-prefer-parameters-names.js

@@ -2,6 +2,7 @@ require('babel-eslint');
 
 const rule = require('../../../lib/rules/mapDispatchToProps-prefer-parameters-names');
 const RuleTester = require('eslint').RuleTester;
+const codeSamples = require('../../code-sanity-samples');
 
 const parserOptions = {
   ecmaVersion: 6,
@@ -15,6 +16,7 @@ const ruleTester = new RuleTester({ parserOptions });
 
 ruleTester.run('mapDispatchToProps-prefer-parameters-names', rule, {
   valid: [
+    ...codeSamples,
     'const mapDispatchToProps = (dispatch, ownProps) => {}',
     'const mapDispatchToProps = (dispatch, {prop1, prop2}) => {}',
     'const mapDispatchToProps = (dispatch) => {}',

tests/lib/rules/mapDispatchToProps-prefer-shorthand.js

@@ -2,6 +2,8 @@ require('babel-eslint');
 
 const rule = require('../../../lib/rules/mapDispatchToProps-prefer-shorthand');
 const RuleTester = require('eslint').RuleTester;
+const codeSamples = require('../../code-sanity-samples');
+
 
 const parserOptions = {
   ecmaVersion: 6,
@@ -15,6 +17,7 @@ const ruleTester = new RuleTester({ parserOptions });
 
 ruleTester.run('mapDispatchToProps-prefer-shorthand', rule, {
   valid: [
+    ...codeSamples,
     'function mapDispatchToProps () {return {action}}',
     `const mapDispatchToProps = dispatch => ({
       onDoSomething: function() {return dispatch(toDo())},

tests/lib/rules/mapDispatchToProps-returns-object.js

@@ -2,6 +2,7 @@ require('babel-eslint');
 
 const rule = require('../../../lib/rules/mapDispatchToProps-returns-object');
 const RuleTester = require('eslint').RuleTester;
+const codeSamples = require('../../code-sanity-samples');
 
 const parserOptions = {
   ecmaVersion: 6,
@@ -15,6 +16,7 @@ const ruleTester = new RuleTester({ parserOptions });
 
 ruleTester.run('mapDispatchToProps-returns-object', rule, {
   valid: [
+    ...codeSamples,
     'const mapDispatchToProps = {}',
     'const mapDispatchToProps = null',
     'const mapDispatchToProps = actionsMap',

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

@@ -2,6 +2,7 @@ require('babel-eslint');
 
 const rule = require('../../../lib/rules/mapStateToProps-no-store');
 const RuleTester = require('eslint').RuleTester;
+const codeSamples = require('../../code-sanity-samples');
 
 const parserOptions = {
   ecmaVersion: 6,
@@ -15,6 +16,7 @@ const ruleTester = new RuleTester({ parserOptions });
 
 ruleTester.run('mapStateToProps-no-store', rule, {
   valid: [
+    ...codeSamples,
     ` const mapStateToProps = state => ({
        ...getSomeStateFromASelector(state),
        showDefaultHeader: showDefaultHeader(state),

tests/lib/rules/mapStateToProps-prefer-hoisted.js

@@ -2,6 +2,7 @@ require('babel-eslint');
 
 const rule = require('../../../lib/rules/mapStateToProps-prefer-hoisted');
 const RuleTester = require('eslint').RuleTester;
+const codeSamples = require('../../code-sanity-samples');
 
 const parserOptions = {
   ecmaVersion: 6,
@@ -17,6 +18,7 @@ const ruleTester = new RuleTester({ parserOptions });
 
 ruleTester.run('mapStateToProps-prefer-hoisted', rule, {
   valid: [
+    ...codeSamples,
     `function mapStateToProps(state) {
       return {};
     }`,