fix(eslint-plugin): [no-unnecessary-condition] improve optional chain…

… handling (#2111)
typescript-eslint · May 30, 2020 · 9ee399b · 9ee399b
1 parent d7d4eeb
commit 9ee399b
Show file tree

Hide file tree

Showing 2 changed files with 63 additions and 1 deletion.
diff --git a/packages/eslint-plugin/src/rules/no-unnecessary-condition.ts b/packages/eslint-plugin/src/rules/no-unnecessary-condition.ts
@@ -20,6 +20,7 @@ import {
   isNullableType,
   nullThrows,
   NullThrowsReasons,
+  isMemberOrOptionalMemberExpression,
 } from '../util';
 
 const typeContainsFlag = (type: ts.Type, flag: ts.TypeFlags): boolean => {
@@ -439,7 +440,11 @@ export default createRule<Options, MessageId>({
         return;
       }
 
-      const type = getNodeType(node);
+      const nodeToCheck = isMemberOrOptionalMemberExpression(node)
+        ? node.object
+        : node;
+      const type = getNodeType(nodeToCheck);
+
       if (
         isTypeFlagSet(type, ts.TypeFlags.Any) ||
         isTypeFlagSet(type, ts.TypeFlags.Unknown) ||

diff --git a/packages/eslint-plugin/tests/rules/no-unnecessary-condition.test.ts b/packages/eslint-plugin/tests/rules/no-unnecessary-condition.test.ts
@@ -724,5 +724,62 @@ foo
         },
       ],
     },
+    {
+      code: `
+declare const x: { a?: { b: string } };
+x?.a?.b;
+      `,
+      output: `
+declare const x: { a?: { b: string } };
+x.a?.b;
+      `,
+      errors: [
+        {
+          messageId: 'neverOptionalChain',
+          line: 3,
+          endLine: 3,
+          column: 2,
+          endColumn: 4,
+        },
+      ],
+    },
+    {
+      code: `
+declare const x: { a: { b?: { c: string } } };
+x.a?.b?.c;
+      `,
+      output: `
+declare const x: { a: { b?: { c: string } } };
+x.a.b?.c;
+      `,
+      errors: [
+        {
+          messageId: 'neverOptionalChain',
+          line: 3,
+          endLine: 3,
+          column: 4,
+          endColumn: 6,
+        },
+      ],
+    },
+    {
+      code: `
+let x: { a?: string };
+x?.a;
+      `,
+      output: `
+let x: { a?: string };
+x.a;
+      `,
+      errors: [
+        {
+          messageId: 'neverOptionalChain',
+          line: 3,
+          endLine: 3,
+          column: 2,
+          endColumn: 4,
+        },
+      ],
+    },
   ],
 });