fix(eslint-plugin): case for destructuring

Author: denisbalyko

packages/eslint-plugin/lib/rules/no-classname-runtime.js

@@ -25,6 +25,13 @@ module.exports = {
           )
             return false
 
+          // Exception for props.x or state.x (case for destructuring)
+          if (
+            arg.object.type === 'Identifier' &&
+            (arg.object.name === 'props' || arg.object.name === 'state')
+          )
+            return false
+
           return canOptimize(arg.object) && canOptimize(arg.property)
         case 'Property':
           return canOptimize(arg.key) && canOptimize(arg.value)

packages/eslint-plugin/tests/lib/rules/no-classname-runtime.js

@@ -23,6 +23,8 @@ ruleTester.run('no-classname-runtime', rule, {
     '<Block className={isVisible ? "Visible" : Hidden}/>',
     '<Block className={cnTest(this.props.className)}/>',
     '<Block className={cnTest(undefined, [this.props.className])}/>',
+    '<Block className={cnTest({ [props.type]: true })}/>',
+    '<Block className={cnTest({ [props.type]: a.b.c.d })}/>',
   ].map((code) => ({
     code,
   })),
@@ -32,8 +34,6 @@ ruleTester.run('no-classname-runtime', rule, {
     '<Block className={cnTest("Test")}/>',
     '<Block className={cnTest("Test", "TestElem")}/>',
     '<Block className={cnTest({ theme })}/>',
-    '<Block className={cnTest({ [props.type]: true })}/>',
-    '<Block className={cnTest({ [props.type]: a.b.c.d })}/>',
     '<Block className={cnTest({ theme, colored })}/>',
     '<Block className={cnTest("Test", { theme })}/>',
     '<Block className={cnTest("Test", { params }, [mixClassName])}/>',