fix(eslint-plugin): fix autofixer for computed properties (#1662)

typescript-eslint · Mar 3, 2020 · ba22ea7 · ba22ea7
1 parent 608a750
commit ba22ea7
Show file tree

Hide file tree

Showing 2 changed files with 15 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
@@ -403,7 +403,7 @@ export default createRule<Options, MessageId>({
     function checkOptionalMemberExpression(
       node: TSESTree.OptionalMemberExpression,
     ): void {
-      checkOptionalChain(node, node.object, '.');
+      checkOptionalChain(node, node.object, node.computed ? '' : '.');
     }
 
     function checkOptionalCallExpression(

diff --git a/packages/eslint-plugin/tests/rules/no-unnecessary-condition.test.ts b/packages/eslint-plugin/tests/rules/no-unnecessary-condition.test.ts
@@ -222,6 +222,7 @@ anyValue?.();
 let unknownValue: unknown;
 unknownValue?.();
 `,
+    'const foo = [1, 2, 3][0];',
   ],
   invalid: [
     // Ensure that it's checking in all the right places
@@ -575,5 +576,18 @@ foo
         },
       ],
     },
+    {
+      code: 'const foo = [1, 2, 3]?.[0];',
+      output: 'const foo = [1, 2, 3][0];',
+      errors: [
+        {
+          messageId: 'neverOptionalChain',
+          line: 1,
+          endLine: 1,
+          column: 22,
+          endColumn: 24,
+        },
+      ],
+    },
   ],
 });