[Fix] no-unused-state: handle optional chaining

Fixes jsx-eslint#2515
golopot · Mar 8, 2020 · cb40c32 · cb40c32
1 parent 9861469
commit cb40c32
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/lib/rules/no-unused-state.js b/lib/rules/no-unused-state.js
@@ -415,7 +415,7 @@ module.exports = {
         handleAssignment(node.id, node.init);
       },
 
-      MemberExpression(node) {
+      'MemberExpression, OptionalMemberExpression'(node) {
         if (!classInfo) {
           return;
         }

diff --git a/tests/lib/rules/no-unused-state.js b/tests/lib/rules/no-unused-state.js
@@ -231,6 +231,28 @@ eslintTester.run('no-unused-state', rule, {
           }
         }`,
       parser: parsers.BABEL_ESLINT
+    }, {
+      code: `
+        class OptionalChaining extends React.Component {
+          constructor() {
+            this.state = { foo: 0 };
+          }
+          render() {
+            return <SomeComponent foo={this.state?.foo} />;
+          }
+        }`,
+      parser: parsers['@TYPESCRIPT_ESLINT']
+    }, {
+      code: `
+        class OptionalChaining extends React.Component {
+          constructor() {
+            this.state = { foo: 0 };
+          }
+          render() {
+            return <SomeComponent foo={this.state?.foo} />;
+          }
+        }`,
+      parser: parsers.BABEL_ESLINT
     },
     `class VariableDeclarationTest extends React.Component {
         constructor() {