fix(eslint-plugin): [return-await] do not auto-fix when type is any/unknown

packages/eslint-plugin/src/rules/return-await.ts

@@ -1,7 +1,7 @@
 import {
   AST_NODE_TYPES,
-  TSESTree,
   TSESLint,
+  TSESTree,
 } from '@typescript-eslint/experimental-utils';
 import * as tsutils from 'tsutils';
 import * as ts from 'typescript';
@@ -169,11 +169,24 @@ export default util.createRule({
         return;
       }
 
+      // any could be thenable; do not auto-fix
+      const useAutoFix = !util.isTypeAnyType(type);
+      const removeFix = (fixer: TSESLint.RuleFixer): TSESLint.RuleFix | null =>
+        removeAwait(fixer, node);
       if (isAwait && !isThenable) {
         context.report({
           messageId: 'nonPromiseAwait',
           node,
-          fix: fixer => removeAwait(fixer, node),
+          ...(useAutoFix
+            ? { fix: removeFix }
+            : {
+                suggest: [
+                  {
+                    messageId: 'nonPromiseAwait',
+                    fix: removeFix,
+                  },
+                ],
+              }),
         });
         return;
       }

packages/eslint-plugin/tests/rules/return-await.test.ts

@@ -1,5 +1,5 @@
 import rule from '../../src/rules/return-await';
-import { getFixturesRootDir, RuleTester, noFormat } from '../RuleTester';
+import { getFixturesRootDir, noFormat, RuleTester } from '../RuleTester';
 
 const rootDir = getFixturesRootDir();
 
@@ -312,6 +312,27 @@ ruleTester.run('return-await', rule, {
         },
       ],
     },
+    {
+      code: `
+        const fn = (): any => null;
+        async function test() {
+          return await fn();
+        }
+      `,
+      // output matches input because return type any is suggestion only
+      output: `
+        const fn = (): any => null;
+        async function test() {
+          return await fn();
+        }
+      `,
+      errors: [
+        {
+          line: 4,
+          messageId: 'nonPromiseAwait',
+        },
+      ],
+    },
     {
       code: 'const test = async () => await Promise.resolve(1);',
       output: 'const test = async () => Promise.resolve(1);',