Merge pull request #10 from eduzz/hotfix-object-ids

fix(id-rule): remove id with object as value from validation

Author: luanlmd

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@eduzz/eslint-config",
   "private": false,
-  "version": "2.5.3",
+  "version": "2.5.4",
   "keywords": [
     "eduzz",
     "eslint"

rules/requiredId.js

@@ -21,6 +21,8 @@ const isDefaultImportSourceSupported = source =>
     'antd/lib/checkbox/Checkbox'
   ].includes(source);
 
+const excludeNestedComponentsNameList = [['Checkbox', 'Group']];
+
 module.exports = {
   meta: {
     type: 'suggestion',
@@ -85,6 +87,13 @@ module.exports = {
           return;
         }
 
+        if (
+          node.name.type === 'JSXMemberExpression' &&
+          excludeNestedComponentsNameList.includes([node.name.object.name, node.name.property.name])
+        ) {
+          return;
+        }
+
         const [id] = node.attributes.filter(({ name }) => {
           if (!name) {
             return false;

rules/validId.js

@@ -46,7 +46,10 @@ module.exports = {
 
         const isJsxExpressionValueVariable =
           node.value.type === 'JSXExpressionContainer' &&
-          (node.value.expression.type === 'Identifier' || node.value.expression.type === 'MemberExpression');
+          (node.value.expression.type === 'Identifier' ||
+            node.value.expression.type === 'MemberExpression' ||
+            node.value.expression.type === 'ObjectExpression' ||
+            node.value.expression.type === 'ArrayExpression');
 
         if (isJsxExpressionValueVariable) {
           return;